meta-pytorch / meta-pytorch/torchcodec
Remove `get_json_metadata()`
Open
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 1.2k
- Forks
- 125
- Avg merge
- 22h 47m
- Merged PRs (30d)
- 54
Description
The old get_json_metadata() core API is superseded by other APIs and we should remove it along with some of its tests.
To remove them, we have to migrate some of the fbcode users.
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
Start in src/torchcodec/decoders/_core/video_decoder_ops.py at get_json_metadata() and inspect the related tests in test/decoders/test_video_decoder_ops.py around lines 258-289. Identify the fbcode users that must migrate before removing the API and its tests. Done means those users are migrated and the obsolete API and tests are removed.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- audio-video-rtc
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100