Skip to content

Conversation

@ihsaan-ullah
Copy link
Collaborator

@ihsaan-ullah ihsaan-ullah commented Oct 25, 2024

@ mention of reviewers

@Didayolo

A brief description of the purpose of the changes contained in this PR.

This new feature is for uploading a task. A zip file with task.yaml and zips of datasets/programs are required to upload a task

You can use already existing datasets/programs by using their keys in your yaml. Check the following two valid tasks:

You can also use a mix of keys and files

Screenshots:

Click the upload task button in the tasks tab in the resource interface to start uploading a task zip
Screenshot 2024-10-31 at 7 31 16 PM

Select a zip file in the filed field and click upload to finish uploading
Screenshot 2024-11-01 at 12 49 00 PM

Documentation in wiki:

I renewed the resource interface documentation including task upload here:
https://github.com/codalab/codabench/wiki/Resource-Management

Issues this PR resolves

A checklist for hand testing

  • use the provided zip files to upload a task
  • try to break the uploading by planting errors either in files, keys or the yaml file

Any relevant files for testing

You can use the following files to test that uploading fails when there are problems:

Checklist

  • Code review by me
  • Hand tested by me
  • I'm proud of my work
  • Code review by reviewer
  • Hand tested by reviewer
  • CircleCi tests are passing
  • Ready to merge

…s and quota refreshed on successful task upload, task upload testing files created
@ihsaan-ullah ihsaan-ullah linked an issue Oct 31, 2024 that may be closed by this pull request
@Didayolo Didayolo self-assigned this Dec 3, 2024
@Didayolo
Copy link
Member

Very nice work. Thank you for the detailed documentation, tests, and explanations.

@Didayolo Didayolo merged commit cf90bc6 into develop Dec 10, 2024
@Didayolo Didayolo deleted the upload_task branch December 10, 2024 14: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.

Task upload

3 participants