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

[GSoc2024] Added labels in dataset_meta.json of widerface_dataset to solve import issue #7258 of opencv/cvat #36

Merged
merged 2 commits into from
Mar 29, 2024

Conversation

teja1412-hub
Copy link

Summary

To fix the Widerface import issue which is being caused due to the missing labels in the widerface_dataset of datumaro, dataset_meta.json file has been edited by adding the "face" label, which the CVAT is looking for during its import process. fix #7258 of opencv/cvat

How to test

This has been tested using the dataset wf_dataset1.zip

Checklist

  • I submit my changes into the develop branch
    - [ ] I have added description of my changes into CHANGELOG
    - [ ] I have updated the documentation accordingly
    - [ ] I have added tests to cover my changes
  • I have linked related issues

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) 2022 CVAT.ai Corporation
#
# SPDX-License-Identifier: MIT

@zhiltsov-max
Copy link
Collaborator

Hi, thank you for sending the PR! Please fix the failing tests.

@teja1412-hub teja1412-hub marked this pull request as draft March 28, 2024 14:23
@teja1412-hub teja1412-hub marked this pull request as ready for review March 28, 2024 20:27
@zhiltsov-max zhiltsov-max merged commit 3e33d6e into cvat-ai:develop Mar 29, 2024
19 checks passed
@zhiltsov-max
Copy link
Collaborator

Thank you for contributing into the project! The next step is to update CVAT dependency on Datumaro.

@teja1412-hub
Copy link
Author

@zhiltsov-max Could you please tell me what should I be doing next?

@zhiltsov-max
Copy link
Collaborator

Please send a PR in CVAT that updates the sever dependencies here.

@teja1412-hub
Copy link
Author

teja1412-hub commented Mar 29, 2024

@zhiltsov-max I've created a PR for the cvat repo that updates the cvat dependency on datumaro PR #7703. Could you please review it?

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.

WiderFace import does not work
2 participants