Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

fixing PandaInnereyeSSLMIL #625

Merged
merged 12 commits into from
Jan 11, 2022
Merged

fixing PandaInnereyeSSLMIL #625

merged 12 commits into from
Jan 11, 2022

Conversation

vale-salvatelli
Copy link
Contributor

This PR contains:

  • an updated committed to hi-ml where the checkpoint downloader is fixed
  • some updates to PandaDeepMIL to enable the use of a SSL pre-trained checkpoint on PANDA
  • downloading checkpoint for CRCk has been moved to the child class for clarity

@vale-salvatelli vale-salvatelli changed the title WIP: fixing PandaInnereyeSSLMIL fixing PandaInnereyeSSLMIL Jan 11, 2022
dccastro
dccastro previously approved these changes Jan 11, 2022
View reviewed changes
InnerEye/Common/fixed_paths.py Outdated Show resolved Hide resolved
InnerEye/ML/Histopathology/models/deepmil.py Outdated Show resolved Hide resolved
InnerEye/ML/configs/histo_configs/classification/DeepSMILECrck.py Show resolved Hide resolved
InnerEye/ML/configs/histo_configs/run_ids.py Show resolved Hide resolved
InnerEye/ML/configs/histo_configs/classification/DeepSMILEPanda.py Outdated Show resolved Hide resolved
@vale-salvatelli vale-salvatelli merged commit 25db288 into main Jan 11, 2022
@vale-salvatelli vale-salvatelli deleted the vsalva/use_panda_checkpoint branch January 11, 2022 17:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@javier-alvarez javier-alvarez javier-alvarez approved these changes

@dccastro dccastro dccastro approved these changes

@mebristo mebristo Awaiting requested review from mebristo

@ant0nsc ant0nsc Awaiting requested review from ant0nsc

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@vale-salvatelli @javier-alvarez @ant0nsc @dccastro