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

[Job Launcher] - Fix manifest validation for CVAT type #690

Merged
merged 1 commit into from
Jul 15, 2023

Conversation

eugenvoronov
Copy link
Contributor

@eugenvoronov eugenvoronov commented Jul 14, 2023

Description

When the job was launched, the manifest validated Fortune type tasks, but did not correctly validate IMAGE_LABEL_BINARY type tasks.

Summary of changes

  • Updated validateManifest method.

How test the changes

POST /job/fortune- Wait a launch stage
POST /job/cvat - Wait a launch stage
yarn test

Related issues

[Job Launcher] - Fix manifest validation for CVAT type#689

@vercel vercel bot temporarily deployed to Preview – fortune-reputation-oracle July 14, 2023 15:45 Inactive
@vercel
Copy link

vercel bot commented Jul 14, 2023

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

Name Status Preview Comments Updated (UTC)
escrow-dashboard ✅ Ready (Inspect) Visit Preview Jul 14, 2023 3:47pm
faucet-server ✅ Ready (Inspect) Visit Preview Jul 14, 2023 3:47pm
fortune-exchange-oracle ✅ Ready (Inspect) Visit Preview Jul 14, 2023 3:47pm
fortune-exchange-oracle-server ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 14, 2023 3:47pm
fortune-job-launcher-client ✅ Ready (Inspect) Visit Preview Jul 14, 2023 3:47pm
fortune-job-launcher-server ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 14, 2023 3:47pm
fortune-recording-oracle ✅ Ready (Inspect) Visit Preview Jul 14, 2023 3:47pm
fortune-reputation-oracle ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 14, 2023 3:47pm

@vercel vercel bot temporarily deployed to Preview – fortune-job-launcher-server July 14, 2023 15:46 Inactive
@vercel vercel bot temporarily deployed to Preview – fortune-exchange-oracle-server July 14, 2023 15:47 Inactive
@eugenvoronov eugenvoronov self-assigned this Jul 14, 2023
@eugenvoronov eugenvoronov linked an issue Jul 14, 2023 that may be closed by this pull request
@eugenvoronov eugenvoronov merged commit cdc4410 into develop Jul 15, 2023
16 checks passed
@eugenvoronov eugenvoronov deleted the fix/manifest-validation-cvat branch July 15, 2023 08:11
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.

[Job Launcher] - Fix manifest validation for CVAT type
2 participants