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

Cb tsvtools finish #374

Merged
merged 65 commits into from
Feb 13, 2023
Merged

Cb tsvtools finish #374

merged 65 commits into from
Feb 13, 2023

Conversation

camillebrianceau
Copy link
Collaborator

This PR is the continuation of the #338
It aims to adapt the training part of clinicaDL to the new tsvtools.

  • change append to np.concat to avoid Warnings !
  • add a new tool adapt. I don't know if we need to keep it, I've done it at the beginning of this PR to convert tsvtools directory from the old way (with one TSV file for each label) to the new way. Maybe it can be useful ?
  • I removed some useless options and added some needed one in some tsvtools functions
  • I add a multi-diagnoses option in the split pipeline (Lisa needed it and it can be useful)
  • I adapt the training part of clinicaDL to the new tsvtools (all the utilsfolder)
    do not worry about the quality-check tests, it will be fix with PR Cb pet qc #361 and PR Cb qc #357

camillebrianceau and others added 30 commits October 11, 2022 16:56
@camillebrianceau
update train for new tsvtools
93e3a36
@camillebrianceau
modification split manager
8f760f0
@camillebrianceau
Merge branch 'aramis-lab:dev' into cb_tsvtools_finish
0adab43
@camillebrianceau
new functionadapt
3ff8cb4
@camillebrianceau
adapt
6a90436
@camillebrianceau
maps manager
65069a0
@camillebrianceau
adapt
01219dd
@camillebrianceau
Merge branch 'aramis-lab:dev' into cb_tsvtools_finish
bb08856
@camillebrianceau
test
9a90a7b
@camillebrianceau
Merge branch 'aramis-lab:dev' into cb_tsvtools_finish
13796eb
@camillebrianceau
maps
a516a3e
@camillebrianceau
black pre commit
1773e9a
@camillebrianceau
black pre commit
6cc0d99
@camillebrianceau
update
fb46b30
@camillebrianceau
print help
1975a1d
@camillebrianceau
Merge branch 'aramis-lab:dev' into cb_tsvtools_finish
4dd0e4f
@camillebrianceau
Merge branch 'aramis-lab:dev' into cb_tsvtools_finish
cdb9c5d
@camillebrianceau
test train with new tsvtools
b32a412
@camillebrianceau
test data ci
495f98c
@camillebrianceau
test data ci train
9bbbf42
@camillebrianceau
test train
4f83241
@camillebrianceau
test train
f5b51c7
@camillebrianceau
test train
7ebb8cb
@camillebrianceau
train test
0278eee
@camillebrianceau
test train
587d7d4
@camillebrianceau
tests
74f5881
@camillebrianceau
test train
4b20f58
@camillebrianceau
test train
211d494
@camillebrianceau
clean code
8ab6297
@camillebrianceau
fix problem with caps option in get_labels
cefe001
@camillebrianceau camillebrianceau merged commit 823391b into aramis-lab:dev Feb 13, 2023
@camillebrianceau camillebrianceau deleted the cb_tsvtools_finish branch February 15, 2023 15:02
camillebrianceau added a commit to camillebrianceau/clinicadl that referenced this pull request Apr 26, 2023
@camillebrianceau
Cb tsvtools finish (aramis-lab#374)
19e3531 
* update train, randomSearch, predict, interpret for new tsvtools

* modification split /maps manager

* new function adapt

* fix warning : use pd.concat instead of append

* add old get-labels
camillebrianceau added a commit to camillebrianceau/clinicadl that referenced this pull request Apr 26, 2023
@camillebrianceau
Cb tsvtools finish (aramis-lab#374)
337f59a 
* update train, randomSearch, predict, interpret for new tsvtools

* modification split /maps manager

* new function adapt

* fix warning : use pd.concat instead of append

* add old get-labels
camillebrianceau added a commit to camillebrianceau/clinicadl that referenced this pull request Apr 26, 2023
@camillebrianceau
Cb tsvtools finish (aramis-lab#374)
ae87465 
* update train, randomSearch, predict, interpret for new tsvtools

* modification split /maps manager

* new function adapt

* fix warning : use pd.concat instead of append

* add old get-labels
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
@camillebrianceau