Closed SpecLad closed 4 days ago
The primary change involves simplifying a function call within the ToolsControlComponent
in the CVAT UI. Specifically, a destructuring assignment was used to extract the convMaskToPoly
property from the body
object before making the call to core.lambda.call
, streamlining the code and improving readability.
File Path | Summary |
---|---|
cvat-ui/src/components/annotation-page/standard-workspace/controls-side-bar/tools-control.tsx |
Extracted convMaskToPoly from body and updated the call to core.lambda.call to use this extracted variable. |
No diagram needed as the changes are straightforward and merely involve code simplification with no modifications to control flow.
In code refined, a change so slight,
Extract the mask, set things right,
With clarity found and clutter gone,
Our tools now shine like the dawn.
🐇✨ CodeRabbit cheers, job well done!
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?
Motivation and context
The server doesn't use it; in the interactive case, the mask-to-polygon conversion is handled by the UI itself.
How has this been tested?
I verified that interactive detection still works (both with the "Convert masks to polygons" option and without).
Checklist
develop
branch[ ] I have created a changelog fragment[ ] I have updated the documentation accordingly[ ] I have added tests to cover my changesLicense
Summary by CodeRabbit