tensorflow / tensorflow/tensorboard
Write test against TensorFlow proto and graph parser.ts to check for divergence
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 7.2k
- Forks
- 1.7k
- Avg merge
- 4d 22h
- Merged PRs (30d)
- 1
Description
TensorBoard graph parser rolled its own prototxt parser and it requires to know name of the repeated field upfront. Although this problem would go away if we use jspb, since we want to keep the parser for foreseeable future, we might want to write a test that automatically checks for divergence.
Good starting point can be found here: https://github.com/tensorflow/tensorboard/pull/2068#issuecomment-477413961
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
Review the graph parser.ts implementation and the TensorFlow proto format, then read the linked pull request comment for the suggested starting point. Add a test that compares the parser against the proto and detects divergence in repeated-field handling.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- tensorflow, typescript
- Domain
- machine-learning, testing
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100