Skip to content

Adds setting to Annotation UI to change quality of the chunk request for both image and video#10840

Open
Casper-lu wants to merge 7 commits into
cvat-ai:developfrom
Casper-lu:smart-res-2-dev
Open

Adds setting to Annotation UI to change quality of the chunk request for both image and video#10840
Casper-lu wants to merge 7 commits into
cvat-ai:developfrom
Casper-lu:smart-res-2-dev

Conversation

@Casper-lu

Copy link
Copy Markdown

Motivation and context

Adds an option for users to select data quality, based on a closed PR #5460 and requested in #3303 and #3824. Additionally support for video has been added as mentioned in the closed PR.

Additionally there have been some discussions regarding 4k video annotation and loading times, #1507, for use cases where one wants to annotation large objects or quickly review annotations this feature could help.

Functionality

If toggle is switched, the compressed version will be loaded instead of the original. Allows users to upload a lower quality compressed version and switch to the original if needed. Caching is adapted to allow for prefetching different sizes depending on the difference in data size between the two modes.

Improvements

  • Would personally like to move to button from settings into the UI for quicker access during annotation.
  • Additionally I would like to see the canvas update directly when the setting is updated but found no neat way of doing it quickly so will have to be looked into.
  • Since video differs a bit from how the images version was implemented this might not be the most optimal solution, if there are any other ideas this could definitely be rewritten, both it does its job as of now!

How has this been tested?

  • Automated tests added (might have to be extended)
  • General manual testing through the UI, loading chunks, prefetching etc with local CVAT dev env

Checklist

  • I submit my changes into the develop branch
  • I have created a changelog fragment
  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • I have linked related issues (see GitHub docs)

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant