Closed SpecLad closed 3 days ago
A small adjustment was made to the ToolsControlComponent
class in the file tools-control.tsx
by modifying the deconstruction of the body
object to include the cleanup
parameter. This refactor is focused on improving code clarity and potentially preparing for future uses of the cleanup
property.
File Path | Change Summary |
---|---|
cvat-ui/src/components/annotation-page/standard-workspace/controls-side-bar/tools-control.tsx |
Addition of the cleanup parameter in the deconstruction of the body object within ToolsControlComponent . |
In a realm of code, with keys and clicks,
A new tweak emerges, nifty and slick.
With acleanup
tag, now in the flow,
Our functions shine, and efficiency grows.
🐇✨🌟
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 support it, and it wouldn't make sense anyway. Potentially it could be handled on the client side, but right now it isn't.
This should've been a part of #8074, but I didn't realize there was a similar issue with
cleanup
as withconvMaskToPoly
.How has this been tested?
Manual testing.
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
cleanup
parameter was not correctly deconstructed in the annotation tools control, improving the reliability of theconvMaskToPoly
feature.