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] Update SDK integration #1100

Merged

Conversation

zhiltsov-max
Copy link
Collaborator

Description

This PR updates CVAT example oracles to use the latest Human Protocol SDK features.

Summary of changes

  • Escrow operations in oracles are replaced with optimized versions
  • Accepted escrow status is limited to only Pending for task creation. Handling a Launched escrow could result in missing escrow fields

How 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

@vercel
Copy link

vercel bot commented Oct 19, 2023

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

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
fortune-exchange-oracle-server ⬜️ Ignored (Inspect) Visit Preview Oct 19, 2023 2:56pm

@vercel
Copy link

vercel bot commented Oct 19, 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.

@Dzeranov Dzeranov self-requested a review October 20, 2023 09:13
@Dzeranov Dzeranov merged commit 653ce7d into humanprotocol:cvat-oracles Oct 20, 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.

2 participants