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

[CVAT integration] Add Recording Oracle example #1005

Merged
merged 4 commits into from
Oct 11, 2023

Conversation

zhiltsov-max
Copy link
Collaborator

@zhiltsov-max zhiltsov-max commented Oct 5, 2023

Description

This PR replaces/adds the old CVAT integration example with a new one, that allows to annotate bounding boxes and points.

Summary of changes

  • examples/cvat/recording-oracle is completely replaced with a new implementation
    • Added support for bbox and point annotations
    • Added validation against user-provided ground truth annotations
    • Removed support for tags
  • Enabled black / isort formatting for the component

See also #1004

How test the changes

No unit tests added yet, it needs to be done. Please contact me about instructions if you want to test the changes.

Related issues

Keywords for linking issues

Operational checklist

  • All new functionality is covered by tests
  • Any related documentation has been changed or added

@CLAassistant
Copy link

CLAassistant commented Oct 5, 2023

CLA assistant check
All committers have signed the CLA.

@vercel
Copy link

vercel bot commented Oct 5, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Ignored Deployments
Name Status Preview Comments Updated (UTC)
fortune-exchange-oracle-server ⬜️ Ignored (Inspect) Visit Preview Oct 11, 2023 8:27am
human-protocol-typescript-sdk-docs ⬜️ Ignored (Inspect) Oct 11, 2023 8:27am

@vercel
Copy link

vercel bot commented Oct 5, 2023

@zhiltsov-max is attempting to deploy a commit to the HUMAN Protocol Team on Vercel.

A member of the Team first needs to authorize it.

@vercel vercel bot temporarily deployed to Preview – human-protocol-typescript-sdk-docs October 5, 2023 12:30 Inactive
@Dzeranov Dzeranov changed the base branch from develop to cvat-oracles October 10, 2023 10:42
eugenvoronov and others added 3 commits October 10, 2023 16:26
* Added refund payment logic

* Updated cancel escrot and refund logic

* Updated migration

* Updated job service and unit tests

* Removed TO_REFUND status

* Removed comment unit tests
@Dzeranov Dzeranov merged commit cc90fce into humanprotocol:cvat-oracles Oct 11, 2023
Dzeranov added a commit that referenced this pull request Oct 26, 2023
* [CVAT integration] Add Exchange Oracle example (#1004)

* Add exchange oracle example for CVAT annotation
---------

Co-authored-by: eugenvoronov <[email protected]>

* [CVAT integration] Add Recording Oracle example (#1005)

* Add recording oracle example for CVAT annotation

---------

Co-authored-by: eugenvoronov <[email protected]>

* [CVAT] Update SDK integration (#1100)

* Update exchange oracle to use newer humanprotocol sdk

* Update sdk and launcher integrations

* Remove extraneous CVAT_ADMIN_USER_ID (#1110)

* Revert unintentional changes in Job Launcher

* An explicit constraint to the python version to be able to update `scipy` package.

---------

Co-authored-by: Maxim Zhiltsov <[email protected]>
Co-authored-by: eugenvoronov <[email protected]>
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.

4 participants