Skip to content

Conversation

@Eeshu-Yadav
Copy link

@Eeshu-Yadav Eeshu-Yadav commented Mar 18, 2025

Description

This PR adds a GitHub Actions workflow to automatically validate the challenge_config.yml file whenever a PR is opened or merged into the master branch. The validation ensures that the configuration file adheres to the required schema and that all referenced files (e.g., templates, annotations) exist.

Changes

  • Added .github/workflows/ci.yml:
    • Runs on PRs and pushes to master
    • Validates challenge_config.yml

Related Issues

Checklist

  • Added GitHub Actions workflow

from jsonschema import validate, ValidationError
from datetime import datetime

SCHEMA = {
Copy link
Member

Choose a reason for hiding this comment

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

@Eeshu-Yadav Can you please use the API from EvalAI to validate the challenge configuration on the server?

Copy link
Author

@Eeshu-Yadav Eeshu-Yadav Mar 18, 2025

Choose a reason for hiding this comment

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

Hi @RishabhJain2018,

Thank you for the feedback! I will update the PR to use the EvalAI API for server-side validation of the challenge_config.yml file.

@RishabhJain2018
Copy link
Member

- name: Checkout code
uses: actions/checkout@v3

- name: Set up Python 3.9
Copy link
Member

Choose a reason for hiding this comment

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

Can we use the latest python version?

Copy link
Author

@Eeshu-Yadav Eeshu-Yadav Mar 19, 2025

Choose a reason for hiding this comment

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

@RishabhJain2018 sorry for that , updated that.

@Eeshu-Yadav
Copy link
Author

Eeshu-Yadav commented Mar 20, 2025

@RishabhJain2018 even after putting the correct credentials , it is failing and showing logs in github actions like this : Eeshu-Yadav#3
any suggestions ?

@RishabhJain2018
Copy link
Member

Hey @Eeshu-Yadav , You need to only use this API: https://github.com/Eeshu-Yadav/EvalAI-Starters/blob/b3898846b718e00c938981995ea4b6b74516ad20/github/config.py#L7 and not use the other API for creating the challenge. We just need to validate if the challenge config is valid even when any PR is merged in the repository.

@RishabhJain2018
Copy link
Member

Hey @Eeshu-Yadav , Instead of changing the current python script, I would suggest you to add a new script. The current script is already being utilized in a feature.

@Eeshu-Yadav
Copy link
Author

okk @RishabhJain2018 sir , will work on that and raise a pr soon , Thanks

@RishabhJain2018
Copy link
Member

Hey @Eeshu-Yadav , Any updates on this?

@Eeshu-Yadav
Copy link
Author

@RishabhJain2018 extremely sorry for the delay , yesterday my exams finished , will raise a pr in the evening , working on that

@RishabhJain2018
Copy link
Member

No worries. Thanks for the update.

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.

Add CI script using Github Actions for each PR

2 participants