Closed brimoor closed 4 weeks ago
[!WARNING]
Rate limit exceeded
@brimoor has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 19 minutes and 2 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between 2350e7420791ae9b8120b7dec1c97f877cdc94bb and 74cc85d5bfa6cadaa8673bc0e8511a05e12025ec.
The recent changes enhance FiftyOne's functionality, especially in handling datasets, 3D scenes, and video processing. Key updates include new parameters for dataset persistence and generation, refined asset path handling, and improved metadata processing for 3D scenes. Additionally, new test methods ensure robustness, and various refinements streamline CVAT integration and dataset export logic.
File(s) | Change Summary |
---|---|
docs/source/teams/cloud_media.rst |
Added methods for downloading scenes, including 3D assets, with configurable options. |
fiftyone/core/clips.py , fiftyone/core/patches.py , fiftyone/core/video.py |
Added persistent and _generated parameters to dataset creation functions for better control over persistence and generation. |
fiftyone/core/dataset.py |
Corrected a comment related to sample_id association with frames in clips datasets. |
fiftyone/core/metadata.py |
Enhanced SceneMetadata class with new methods for handling 3D scene metadata and asset parsing. |
fiftyone/core/stages.py |
Added _generated=True parameter to several function calls within the load_view method. |
fiftyone/core/storage.py |
Renamed resolve to realpath and updated docstrings for clarity. Modified abspath to handle relative path indicators better. |
fiftyone/core/threed/object_3d.py , fiftyone/core/threed/scene_3d.py |
Improved handling of asset paths within Object3D and Scene classes, including logic refinements and efficient storage. |
fiftyone/utils/cvat.py , fiftyone/utils/data/exporters.py , fiftyone/utils/utils3d.py |
Refined parameter descriptions, improved logic for handling annotations, and adjusted path resolution for 3D assets. |
tests/intensive/cvat_tests.py , tests/unittests/metadata_tests.py , tests/unittests/patches_tests.py , tests/unittests/threed/scene_3d_tests.py , tests/unittests/utils3d_tests.py , tests/unittests/video_tests.py |
Added new test methods and adjusted existing ones to ensure robustness and proper functionality of new features. |
In the land of code, where datasets grow,
New methods bloom, like flowers in snow.
With assets in 3D, and scenes to download,
Persistent and generated, a story unfolds.
Tests now ensure, all runs smooth and bright,
As FiftyOne evolves, in the digital light.
🌟
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?
These PRs were included in the release notes for 0.24.0 but somehow appear to have not actually been released (I've moved their associated release notes into v0.24.1 in https://github.com/voxel51/fiftyone/pull/4456):
Also cherry-picks these bug fixes:
And the following PR which just missed the cutoff from v0.24.0 but has tests:
Summary by CodeRabbit
New Features
Enhancements
Bug Fixes
Tests