Closed klakhov closed 1 day ago
[!IMPORTANT]
Review skipped
Auto incremental reviews are disabled on this repository.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
This update introduces the Propagate Shapes
action in CVAT, allowing simultaneous propagation of filtered shapes across frames. Changes include adding the PropagateShapes
class, related types, and enums in annotations-actions.ts
, the propagateShapes
function in object-utils.ts
, and accommodating user interface adjustments in the annotations-actions-modal.tsx
. Updates also involved incrementing version numbers for cvat-core
and cvat-ui
.
File | Summary |
---|---|
changelog.d/..._propagate_shapes_simultaneously.md | Introduced a documentation file summarizing the new Propagate shapes action. |
cvat-core/package.json | Bumped version from 15.0.6 to 15.0.7 to indicate a new feature release. |
cvat-core/src/annotations-actions.ts | Added the PropagateShapes class, FrameSelectionType enum, and updated BaseSingleFrameAction class. |
cvat-core/src/object-utils.ts | Introduced the propagateShapes function to handle shape propagation between frames. |
cvat-ui/package.json | Incremented version from 1.63.11 to 1.63.12 for a minor UI update. |
cvat-ui/src/components/.../annotations-actions-modal.tsx | Enhanced the modal to support FrameSelectionType and updated related UI/logic components. |
cvat-ui/src/cvat-core-wrapper.ts | Added FrameSelectionType to BaseSingleFrameAction import and updated exported entities accordingly. |
sequenceDiagram
participant User
participant UI as CVAT UI
participant Core as CVAT Core
User->>UI: Opens annotations modal
UI->>Core: Initialize PropagateShapes action
Core->>UI: Provide frame selection options
User->>UI: Select frame range and shapes to propagate
UI->>Core: Call propagateShapes function with selected frames and shapes
Core->>UI: Return propagated shapes data
UI->>User: Display updated shapes on frames
🐇 In CVAT's world so bright and clear,
Frames now sync, there's no more fear,
Shapes propagate from frame to frame,
Simplifying tasks, enhancing the game.
With versions ticked up just a bit,
Shape your work with ease and wit!
🌟✨
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?
Default value for target frame "0" is useless as I am already on the same frame. I will propose to use the same default values as we use in default propagation functionality (+50 or -50 frames as I remember depending on possible propagation directions).
Also when I am on frame 0 and target frame is 0, running propagate action freezes the browser (infinite loop?)
Found a difference in behavior how masks are propagated using default propagation and actions. Example:
I am not sure, that copies in both cases are correct, but the second case creates more issues, not allowing to export task.
Also, please, add tests to cover found issues.
Attention: Patch coverage is 87.50000%
with 7 lines
in your changes missing coverage. Please review.
Project coverage is 83.63%. Comparing base (
78fa7cf
) to head (b56bbf7
). Report is 11 commits behind head on develop.
- Default value for target frame "0" is useless as I am already on the same frame.
In default propagate we use 0
and lastFrame
, so Ive added last frame as default
- Also when I am on frame 0 and target frame is 0, running propagate action freezes the browser (infinite loop?)
Fixed
- Found a difference in behavior how masks are propagated using default propagation and actions.
Fixed as discussed (cropMask if needed in MaskShape
constructor)
Also added tests for default propagate aciton, action on current frame and mask case
Extra file?
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
Motivation and context
The PR adds new action
Propagate shapes
that allows to propagate all filtered shapes on the current frame simultaneouslyHow has this been tested?
Manual testing
Checklist
develop
branchLicense
Summary by CodeRabbit
New Features
Updates
cvat-core
to15.0.7
andcvat-ui
to1.63.12
indicating a new release.Enhancements