Skip to content
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

sync with upstream #80

Merged
merged 55 commits into from
Feb 13, 2025
Merged

sync with upstream #80

merged 55 commits into from
Feb 13, 2025

Conversation

Eldies
Copy link

@Eldies Eldies commented Feb 7, 2025

Summary

Syncing

  • util/mask_tools.py
  • util/image.py
  • components/media.py
  • components/importer.py
  • util/meta_file_util.py
  • components/exporter.py
  • components/hl_ops.py
  • components/dataset.py

Also

  • moving Registry and PluginRegistry classes to components/registry.py
  • renaming cli/contexts/project/diff.py to cli/util/compare.py

How to test

Checklist

  • I have added unit tests to cover my changes.​
  • I have added integration tests to cover my changes.​
  • I have added the description of my changes into CHANGELOG.​
  • I have updated the documentation accordingly

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.
  • I have updated the license header for each file (see an example below).
# Copyright (C) 2025 CVAT.ai Corporation
#
# SPDX-License-Identifier: MIT

@Eldies Eldies force-pushed the dl/sync-with-upstream branch 2 times, most recently from cb16095 to 240196f Compare February 7, 2025 12:16
@Eldies Eldies assigned Eldies and unassigned Eldies Feb 7, 2025
@Eldies
Copy link
Author

Eldies commented Feb 7, 2025

I am still investigating this one test which fails on windows, but I dont think there will be a lot of changes

@Eldies Eldies mentioned this pull request Feb 10, 2025
6 tasks
@zhiltsov-max
Copy link
Collaborator

always use exif info of images

Please extract this into a separate PR.

@Eldies
Copy link
Author

Eldies commented Feb 10, 2025

always use exif info of images

Please extract this into a separate PR.

#82

@zhiltsov-max
Copy link
Collaborator

If there are any tests for the Crypter component, it's better to copy them as well.

@Eldies
Copy link
Author

Eldies commented Feb 13, 2025

If there are any tests for the Crypter component, it's better to copy them as well.

done

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
9.4% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@zhiltsov-max zhiltsov-max linked an issue Feb 13, 2025 that may be closed by this pull request
@zhiltsov-max zhiltsov-max merged commit 8e9d47b into develop Feb 13, 2025
5 of 6 checks passed
@zhiltsov-max zhiltsov-max deleted the dl/sync-with-upstream branch February 13, 2025 16:06
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.

allow opencv-python>=4.11
2 participants