Closed ehofesmann closed 2 months ago
The recent updates enhance the CVAT
class by introducing a method to generate URLs for job annotations and refining the annotation download process. Additionally, the test suite for detection labeling has been updated by adjusting the segment_size
parameter, ensuring more precise unit testing.
File Path | Change Summary |
---|---|
fiftyone/utils/cvat.py |
Added job_annotation_url , modified download_annotations , added _get_job_ids in CVAT class. |
tests/intensive/... |
Updated test_detection_labelling method by adding a segment_size parameter. |
🐇✨ In the realm of code, where logic is king, A rabbit hopped in, making changes with a swing. URLs for jobs, a tweak in the test, Now everything runs just at its best. Hop, hop, hooray, let the data flow, With every line, our project will grow! 🌱🚀 🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Implementation LGTM 💪
@ehofesmann if you retarget this at
release/v0.24.0
we can include in the release this week 🤓
Thanks @brimoor ! Just getting back to this now, I assume I missed the window on this. I do still need to get it into teams too. It's OK if it doesn't make it into v0.24.0.
@benjaminpkane I see you changed the base back to develop
, is it good to merge into there? If so, can I get a rereview?
What changes are proposed in this pull request?
Optimized loading annotations from the CVAT backend. Annotations are now loaded from individual jobs instead of entire tasks which allows for importing annotations from much larger task sizes. There is one task in the internal CVAT deployment with 10k samples, in 200 jobs of 50 samples. Previously, trying to load this task would make a single request to the CVAT server to load all annotations from the task at once, this crashes the CVAT server. Now, annotations from each job are loaded sequentially which resolves this problem.
How is this patch tested? If it is not, please explain why.
Unit tests pass:
Also task 159 on the internal CVAT test deployment containing bdd100k-validation now imports properly. It is recommended you have bdd100k validation images available locally on disk as it makes this easier:
Release Notes
Is this a user-facing change that should be mentioned in the release notes?
Optimized loading annotations from the CVAT backend. Annotations are now loaded from individual jobs instead of entire tasks which allows for importing annotations from much larger task sizes.
What areas of FiftyOne does this PR affect?
fiftyone
Python library changesSummary by CodeRabbit