Skip to content

MIL training #49

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
Open
wants to merge 88 commits into
base: main
Choose a base branch
from
Open

MIL training #49

wants to merge 88 commits into from

Conversation

jcohenadad
Copy link
Member

TODO

Thomas Dagonneau and others added 30 commits November 21, 2024 15:21
add a gitignore
6a68533
remove of undesired files
69357c0
training code for sbs added
4778473
add a code to train for nfn with both left and right, rnfn with only …
1cd1ecc 
…right images and lnfn with only left images
Realised the crop and resampling were not done on the good coordinate…
9cb7783 
…s....
tried to change a few things for nfn
3132064
Modification of nfn to take into account the fact the images are not …
f10b2ca 
…resampled
minor adjustment
0b930b0
the nfn takes only T1 scans
bbed1e6
Tested a few variations with data augmentation in the transform function
3d7dfb1
added a weight decay for training to reduce overfitting
5be05ea
created a first version of the code for sas and modifications on nfn
346ffe9
try to change from Adam optimizer to SGD for scs
ffb619b
Changed the classic block to bottelneck
ee44d98
before trying to do patch extraction in preprocessing
39e032e
@abelsalm
add a file to train using volume and data augmentation
158c9c1
added train,ing with data augmentation, see issue associated
249a73d
first data augmentation plots added in models (.pth still in .gitignore)
cea2e05
gitignore changed
36f8ad4
@abelsalm
nfn training with volumes and data aug
0dafdff
nfn training with data aug works, also added the curves of the trainings
b994273
nfn data aug working
719f67a
sas training with volumes and data aug
fccfbfb
training with splits among subjects, and data aug
84e6dc1
added a code to train nfn based on the patient split
9106ce9
The code for train_nfn_patient_split didn t work but I fixed it
be313fb
wandb configured! see in the weight and biases folder...
379760c
Merge remote-tracking branch 'origin/ResNetTraining' into ResNetTraining
90685ae 
Merging branches cause Thomas and I were working on two different tings and commits started being different
modif split
75456bd
Merge branch 'ResNetTraining' of https://github.com/ivadomed/lumbar-c…
6470608 
…lassification-rsna-challenge-2024 into ResNetTraining
abelsalm and others added 30 commits March 17, 2025 15:33
@abelsalm
tested data aug, works well !
a43b024
gitignore
492ed64
version with RNN working
f508204
adding different schedule for encoder, including freezing it...
480cc4d
up to date
71b2aa1
removed CustomDataset
2364237
insertion of regular_transforms in the gat_transforms and added the n…
5e24670 
…fn part.
added the training for nfn
371b60e
added the versions of training for nfn
dc419e8
minor modif
8f231b9
made a loss without weights for training the encoder
38294e7
final push before checkout
6595a9d
last changes on MIL, to be cleaned still
ce91439
@abelsalm
Merge branch 'MILtraining' of github.com:ivadomed/lumbar-classificati…
acab887 
…on-rsna-challenge-2024 into MILtraining
@abelsalm
nfn training
4cbd427
@abelsalm
cleaning bien commencé, réorganisation et readme, plus retirer des fo…
c2f562c 
…nctions inutiles...
@abelsalm
rdme
f4519f3
@abelsalm
rdme
490541c
@abelsalm
everything works
23f63d4
removed old files
c3e1fcf
put everything related to mil training in MIL_training. Also standard…
1b5a2a6 
…ized the data augmentation according to the choices made for RESNET
@tomDag25
removed duplicated files
b16a454
@tomDag25
removed the distinction between left and right
977f128
@tomDag25
harmonization of the code between the three pathologies
19bbe23
@tomDag25
Code cleaning
d32f0a1
changed the padding moment to not have only noise slices
d0cd246
test
25a19b4
validate merge
4c601d4
standardization of the training code based on nfn
d210b81
hyperparameters modification
579031e
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.

3 participants

@jcohenadad @abelsalm @tomDag25