-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update develop after v2.16.2 #8265
Conversation
WalkthroughThe version 2.16.2 update significantly enhances user experience and application functionality. Key improvements include streamlined navigation by preventing page reloads from notification links, fixes to the copy/paste and undo functionalities for annotations, and critical enhancements to export processes. Usability improvements, like a cancellation button for deletions and better handling of job retries, further ensure a smoother interaction with the application. Changes
Poem
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? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
Outside diff range, codebase verification and nitpick comments (10)
CHANGELOG.md (10)
24-25
: Clarify the description for better readability.The description "Following the link in notification no longer reloads the page" could be more specific about the context or the type of notification.
- Following the link in notification no longer reloads the page + Following a link in a notification now navigates without reloading the page
29-30
: Clarify the description for better readability.The description "Copy/paste annotation guide with assets did not work, showing the message Asset is already related to another guide" could be more specific about the context.
- Copy/paste annotation guide with assets did not work, showing the message **Asset is already related to another guide** + Fixed an issue where copying/pasting an annotation guide with assets showed the message **Asset is already related to another guide**
32-33
: Clarify the description for better readability.The description "Undo can't be done when a shape is rotated" could be more specific about the context.
- Undo can't be done when a shape is rotated + Fixed an issue where undoing actions was not possible when a shape was rotated
35-37
: Clarify the description for better readability.The description "Exporting a skeleton track in a format defined for shapes raises error
operands could not be broadcast together with shapes (X, ) (Y, )
" could be more specific about the context.- Exporting a skeleton track in a format defined for shapes raises error `operands could not be broadcast together with shapes (X, ) (Y, )` + Fixed an error when exporting a skeleton track in a format defined for shapes, which raised the error `operands could not be broadcast together with shapes (X, ) (Y, )`
39-40
: Clarify the description for better readability.The description "Delete label modal window does not have cancellation button" could be more specific about the context.
- Delete label modal window does not have cancellation button + Added a cancellation button to the delete label modal window
42-43
: Clarify the description for better readability.The description "Export and export cache clean rq job retries' hangs" could be more specific about the context.
- Export and export cache clean rq job retries' hangs + Fixed an issue where export and export cache clean request job retries would hang
45-46
: Clarify the description for better readability.The description "The automatic annotation process failed for tasks from cloud data" could be more specific about the context.
- The automatic annotation process failed for tasks from cloud data + Fixed an issue where the automatic annotation process failed for tasks sourced from cloud data
51-52
: Clarify the description for better readability.The description "Annotations in a ground truth jobs marked as GT annotations after modifying" could be more specific about the context.
- Annotations in a ground truth jobs marked as GT annotations after modifying + Fixed an issue where annotations in ground truth jobs were not correctly marked as GT annotations after modifications
54-55
: Clarify the description for better readability.The description "API call to run automatic annotations fails on a model with attributes when mapping not provided in the request" could be more specific about the context.
- API call to run automatic annotations fails on a model with attributes when mapping not provided in the request + Fixed an issue where API calls to run automatic annotations failed on models with attributes when mapping was not provided in the request
57-59
: Clarify the description for better readability.The description "Fixed a label collision issue where labels with similar prefixes and numeric suffixes could conflict, causing error on export" could be more specific about the context.
- Fixed a label collision issue where labels with similar prefixes and numeric suffixes could conflict, causing error on export + Fixed a label collision issue where labels with similar prefixes and numeric suffixes could conflict, causing errors during export
- Request card was not disabed properly after downloading | ||
(<https://github.com/cvat-ai/cvat/pull/8197>) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix the typo.
The description "Request card was not disabed properly after downloading" contains a typo.
- Request card was not disabed properly after downloading
+ Request card was not disabled properly after downloading
Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
- Request card was not disabed properly after downloading | |
(<https://github.com/cvat-ai/cvat/pull/8197>) | |
- Request card was not disabed properly after downloading | |
+ Request card was not disabled properly after downloading |
Summary by CodeRabbit
New Features
Bug Fixes