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

Update tests, CI and pre-commit #193

Merged
merged 15 commits into from
Jan 23, 2023
Merged

Update tests, CI and pre-commit #193

merged 15 commits into from
Jan 23, 2023

Conversation

Warvito
Copy link
Collaborator

@Warvito Warvito commented Jan 16, 2023

Signed-off-by: Walter Hugo Lopez Pinaya ianonimato@hotmail.com

@Warvito
Add initial runtests.sh
9dd9699 
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
@Warvito Warvito linked an issue Jan 16, 2023 that may be closed by this pull request
Docker container to run tests #185
Closed
@Warvito
Duplicate utils to remove necessity to clone MONAI in order to run tests
901eae5 
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
@Warvito
Add note
3c2fa35 
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
@Warvito
[WIP] Add tests runners
f1e196f 
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
@Warvito
Update min_tests list and reduce tests scope
9b602cb 
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
@Warvito
Use all environments tested in monai core
7629767 
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
@Warvito
Update pre-commit and requirements for tests
8435aec 
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
@Warvito
Remove duplicated tqdm and move lpips to dev
957ea60 
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
@Warvito
Remove test_latent_diffusion_inferer.py from min tests
1f4ec48 
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
@Warvito
Try to fix requirements for min tests
f3e78b8 
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
@Warvito
Fix dependencies
7120392 
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
@Warvito
Fix dependencies
3c30835 
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
@Warvito
Add test_diffusion_inferer to excluded min-tests
fac0b7e 
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
@Warvito
Update CONTRIBUTING.md
694b017 
Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
@Warvito
Fix typos, remove formatting from unit tests and add black and isort …
5ca8bac 
…to pre-commit

Signed-off-by: Walter Hugo Lopez Pinaya <ianonimato@hotmail.com>
@Warvito Warvito changed the title Add CI Update tests, CI and pre-commit Jan 20, 2023
@Warvito Warvito marked this pull request as ready for review January 20, 2023 19:52
@Warvito Warvito mentioned this pull request Jan 21, 2023
Closed
@Warvito Warvito merged commit 3d8bac6 into main Jan 23, 2023
@Warvito Warvito deleted the 185-docker-container-to-run-tests branch January 23, 2023 22:27
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.

Docker container to run tests
1 participant
@Warvito