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.

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

ENH: Add command line arg for custom SSL checkpoint #560

Merged
merged 12 commits into from
Aug 9, 2022

Conversation

mebristo
Copy link
Member

@mebristo mebristo commented Aug 8, 2022

Merge in changes from main to add custom SSL checkpoint flag and a smoke test for TcgaCrckSSLMIL

fepegar and others added 9 commits August 3, 2022 13:31
* Relax TorchVision requirement version

* Update TorchVision versions interval
* MNT: Prepare package for PyPI

* Update authors field

As suggested by @javier-alvarez.

* Remove keywords and unused path variable

* Fix import
* Add hubconf file

* Refactor to minimise hubconf dependencies

* Pin hubconf dependencies

* Revert "Pin hubconf dependencies"

This reverts commit bc904a9 as it
didn't seem to work.

* Add support for newer versions of torch

* Add only the model folder to path

* Remove unnecessary try-except block

* Avoid duplicate definition of Hugging Face strings

* Import from a more appropriate module

* Add test to compare package and PyTorch Hub models

* Add version number to package __init__

* Remove branch name from PyTorch Hub repo string

* Check only fields from package model

* Remove unnecessary zip wrap

Co-authored-by: Shruthi42 <13177030+Shruthi42@users.noreply.github.com>

Co-authored-by: Shruthi42 <13177030+Shruthi42@users.noreply.github.com>
Reset Accuracy metric after calling compute
@codecov
Copy link

codecov bot commented Aug 8, 2022

Codecov Report

Merging #560 (519af35) into transfer_main (5060e28) will decrease coverage by 0.00%.
The diff coverage is 60.00%.

Impacted file tree graph

Flag Coverage Δ
hi-ml-cpath 78.07% <60.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...alth_cpath/configs/classification/DeepSMILECrck.py 68.75% <0.00%> (ø)
...gs/classification/DeepSMILESlidesPandaBenchmark.py 63.79% <ø> (ø)
...src/health_cpath/configs/classification/BaseMIL.py 79.25% <50.00%> (-0.16%) ⬇️
...lth_cpath/configs/classification/DeepSMILEPanda.py 83.56% <100.00%> (ø)

@mebristo mebristo changed the title [DRAFT] Add TCGACRCKSSLMIL smoke test to workflow ENH: Add TCGACRCKSSLMIL smoke test to workflow Aug 8, 2022
@mebristo mebristo changed the title ENH: Add TCGACRCKSSLMIL smoke test to workflow ENH: Add command line arg for custom SSL checkpoint Aug 8, 2022
@mebristo mebristo marked this pull request as ready for review August 9, 2022 07:23
@mebristo mebristo merged commit 6c9e4c9 into transfer_main Aug 9, 2022
@mebristo mebristo deleted the mebristo/update_workflow branch August 9, 2022 07:25
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.

4 participants