Op Generated code locally does not match master repository
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 928
- Forks
- 227
- PR merge metrics
- No merged PRs in 30d
Description
My local repository says it is 100% in sync with tensorflow/java master.
I do a mvn clean, mvn install, then all the generated Ops are updated and next it wants me to recommit them.
Either something is wrong on my end, or the master repository is old.
I was expecting the locally generated classes to be the same as the main repository, as I am only working in framework at the moment.
@rnett Would this be something you might have a clue about.
Here is a sample of the delta between the repository and the locally generated classes.
org.tensorflow.op.core.Abort
Stored on Master at tensorflow/java:
public static Abort create(Scope scope, Options... options) {
OperationBuilder opBuilder = scope.env().opBuilder(OP_NAME, scope.makeOpName("Abort"));
opBuilder = scope.apply(opBuilder);
if (options != null) {
Source generated locally:
public static Abort create(Scope scope, Options... options) {
OperationBuilder opBuilder = scope.opBuilder(OP_NAME, "Abort");
if (options != null) {
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Reproduce the mismatch with mvn clean and mvn install while working in framework, then compare the generated org.tensorflow.op.core.Abort class with the version in the tensorflow/java master repository. Trace the generation step that produces the Ops and determine whether the repository or local generation is outdated. Done means the generated classes match master or the discrepancy is documented with its cause.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- build-system
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100