Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enable full regression tests in the CI #44

Closed
wants to merge 10 commits into from
Closed

Enable full regression tests in the CI #44

wants to merge 10 commits into from

Conversation

lebrice
Copy link
Collaborator

@lebrice lebrice commented Aug 30, 2024 •
edited
Loading

  • Re-enables the reproducibility tests for algorithms and datamodules in the CI
  • Moves the regression files to a different folder (.regression_files) instead of placing them next to the source code.
  • Add an entry in the "files.exclude" entry of the vscode settings for this new ".regression_files" folder.

Signed-off-by: Fabrice Normandin [email protected]

@lebrice lebrice changed the title Enable-regression-tests Enable full regression tests in the CI Aug 30, 2024
@lebrice
Add regression files to new folder
56ce765 
Signed-off-by: Fabrice Normandin <[email protected]>
@lebrice
Tweaks to the hf example
4ab2451 
Signed-off-by: Fabrice Normandin <[email protected]>
@lebrice
Fix some regression files
7e65bb8 
Signed-off-by: Fabrice Normandin <[email protected]>
@lebrice
Slightly increase resources in SLURM CI step
2e48102 
Signed-off-by: Fabrice Normandin <[email protected]>
@lebrice
Make the job shorter, but unkillable
38ea1e5 
Signed-off-by: Fabrice Normandin <[email protected]>
@lebrice
Don't run 'slow' tests for now
8385927 
Signed-off-by: Fabrice Normandin <[email protected]>
@lebrice
Re-enable datamodule regression tests
815ae0f 
Signed-off-by: Fabrice Normandin <[email protected]>
@lebrice
Fix the regression tests for auto-schema feature
0e297c3 
Signed-off-by: Fabrice Normandin <[email protected]>
@lebrice lebrice closed this Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@lebrice