Skip to content

Remove code upload challenges#4458

Merged
gchhablani merged 5 commits intoCloud-CV:masterfrom
gchhablani:remove_code_upload
Mar 21, 2025
Merged

Remove code upload challenges#4458
gchhablani merged 5 commits intoCloud-CV:masterfrom
gchhablani:remove_code_upload

Conversation

@gchhablani
Copy link
Copy Markdown
Collaborator

@gchhablani gchhablani commented Feb 20, 2025 •
edited
Loading

This PR adds an error in challenge validation on creation of code-upload challenge via starter template.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Feb 20, 2025

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.27%. Comparing base (96968d6) to head (291a2f2).
Report is 1113 commits behind head on master.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files 
@@            Coverage Diff             @@
##           master    #4458      +/-   ##
==========================================
- Coverage   72.93%   69.27%   -3.66%     
==========================================
  Files          83       20      -63     
  Lines        5368     3574    -1794     
==========================================
- Hits         3915     2476    -1439     
+ Misses       1453     1098     -355

see 64 files with indirect coverage changes

see 64 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 913dd9f...291a2f2. Read the comment docs.

RishabhJain2018
RishabhJain2018 requested changes Mar 14, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@RishabhJain2018 RishabhJain2018 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @gchhablani: Can you please write the description for the PR.

RishabhJain2018
RishabhJain2018 requested changes Mar 15, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@RishabhJain2018 RishabhJain2018 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gchhablani Please check the comments!

Comment thread apps/challenges/challenge_config_utils.py Outdated
Comment thread docs/source/host_challenge.md Outdated
@gchhablani gchhablani merged commit c194f5e into Cloud-CV:master Mar 21, 2025
1 check failed
gchhablani added a commit that referenced this pull request Aug 24, 2025
@gchhablani
Revert "Remove code upload challenges (#4458)"
8644347 
This reverts commit c194f5e.
RishabhJain2018 pushed a commit that referenced this pull request Aug 25, 2025
@gchhablani
Revert "Remove code upload challenges (#4458)" (#4773)
0870651 
This reverts commit c194f5e.
RishabhJain2018 pushed a commit that referenced this pull request Mar 2, 2026
@gchhablani
Remove code upload challenges (#4458)
c2a3965 
* [Documentation] Fix host challenge doc

* Add validation for checking code-upload challenges

* Update challenge_config_utils.py

* Update host_challenge.md
RishabhJain2018 pushed a commit that referenced this pull request Mar 2, 2026
@gchhablani
Revert "Remove code upload challenges (#4458)" (#4773)
938dd35 
This reverts commit c2a3965.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@RishabhJain2018 RishabhJain2018 RishabhJain2018 requested changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@gchhablani @codecov-commenter @RishabhJain2018