Skip to content

Conversation

iqbalcodes6602
Copy link
Collaborator

Fixes: #8

  • Implement generateresources.py to generate test plans and images for problems.
  • Create lint script for linting checks in a Docker container.
  • Introduce problems.py to manage problem loading and validation.
  • Develop runtests.py for running tests on problems with concurrent execution.
  • Add a sample configuration file sample-config.yaml for problem setup.
  • Implement upload.py to handle problem uploads to omegaUp, including zip creation and API interactions.

- Implement `generateresources.py` to generate test plans and images for problems.
- Create `lint` script for linting checks in a Docker container.
- Introduce `problems.py` to manage problem loading and validation.
- Develop `runtests.py` for running tests on problems with concurrent execution.
- Add a sample configuration file `sample-config.yaml` for problem setup.
- Implement `upload.py` to handle problem uploads to omegaUp, including zip creation and API interactions.
@iqbalcodes6602 iqbalcodes6602 requested a review from pabo99 July 14, 2025 17:05
Copy link
Collaborator

@pabo99 pabo99 left a comment

Choose a reason for hiding this comment

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

The content of this PR is too extensive. Even so, we can try to address the suggestions so we can merge it and complete the tasks for the midterm evaluation.

@iqbalcodes6602
Copy link
Collaborator Author

iqbalcodes6602 commented Jul 16, 2025

The content of this PR is too extensive. Even so, we can try to address the suggestions so we can merge it and complete the tasks for the midterm evaluation.

@pabo99 ,
Sorry for the inconvinience, as you know these files were not created by me omegaup-deploy, but we had to use them.
We could have used it as submodule, the thing was i needed to update many of the funcitons and also depencies and all were outdated. So i didn't had any other options except to commit them the whole folder.
That's why this PR is so entensive.

Also I have made all the changes you have asked for.

@iqbalcodes6602 iqbalcodes6602 requested a review from pabo99 July 16, 2025 16:03
@pabo99
Copy link
Collaborator

pabo99 commented Jul 16, 2025

The content of this PR is too extensive. Even so, we can try to address the suggestions so we can merge it and complete the tasks for the midterm evaluation.

@pabo99 , Sorry for the inconvinience, as you know these files were not created by me omegaup-deploy, but we had to use them. We could have used it as submodule, the thing was i needed to update many of the funcitons and also depencies and all were outdated. So i didn't had any other options except to commit them the whole folder. That's why this PR is so entensive.

Also I have made all the changes you have asked for.

No problem, the ideal approach would’ve been for the library to be up-to-date so we could’ve saved time setting everything up. Thanks for applying the fixes!

Copy link
Collaborator

@pabo99 pabo99 left a comment

Choose a reason for hiding this comment

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

Great!

@pabo99 pabo99 merged commit 8fe9245 into omegaup:main Jul 16, 2025
1 check passed
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.

Integrate automatic deployment (similar to OMI repo)

2 participants