Skip to content

Conversation

@Didayolo
Copy link
Member

@Didayolo Didayolo commented Jun 1, 2023

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

Upon manual UI challenge creation, the leaderboard isn't saved.

Issues this PR resolves

A checklist for hand testing

  • Make a challenge using the form in the UI and don't forget to create a leaderboard.
  • Next once competition is created, go to edit and make sure the leaderboard you created still exists.
  • Also go to results tab and make sure you can see the leaderboard.

Any relevant files for testing

N/A

Misc. comments

I'm worried that this fix and the underlying code I copied or updating a challenge is not what the original creators wanted. See this comment above the code I copied:

# TODO - This is Temporary. Need to change Leaderboard to Phase connect to M2M and handle this correctly.
.

Checklist

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

@Didayolo Didayolo merged commit 1ce4c53 into develop Jun 1, 2023
@Didayolo Didayolo deleted the save-leaderboard branch June 1, 2023 15:41
Didayolo added a commit that referenced this pull request Jun 1, 2023
* html comment fixed

* repeated competitions in competitions api fixed

* uploading a submission, show asterick with required only answers

* do not show make private/public button to users who are not owner of the submission

* Remove console calls

* Resources -> Datasets : fiel sizes added in readable format

* console log removed

* Update management.tag

* pagination fixed

* Show competition name in Resources -> Datasets

* filename added in request, test updated

* Dataset detail tile changed to file name

* Use title and url of benchmark in job status page

* competition name is now a clickable link

* competition name clickable in dataset detail

* Update management.tag

* Update management.tag

* separate tab for submissions

* Add Dataset changed to Add Dataset/Program

* Add submission modal added

* After login redirect to `next` page (#900)

* after login redirect to next page

* trailing white space removed

---------

Co-authored-by: Ihsan Ullah <ihsanullah@Pixel-io.local>

* benchmark title instead of id in datasets detail

* Tasks tab: benchmark is now clickable link

* milliseconds -> seconds

* add option to show/hide public submissions/datasets

* new filters added to show/hide public submissions and datasets/programs

* default filter added

* unused function removed

* delete button is now shown for own submissions and datasets/programs only

* Save leaderboard during manual creation of benchmark (#908)

* Save leaderboard during manual creation of benchmark

* Update competitions.py

* Fix the test: precision should be a number

---------

Co-authored-by: Ihsan Ullah <ihsanullah@Pixel-io.local>
Co-authored-by: Ihsan Ullah <ihsan2131@gmail.com>
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