-
Notifications
You must be signed in to change notification settings - Fork 55
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
Generate pipeline adaptation to data class #582
Closed
camillebrianceau
wants to merge
14
commits into
aramis-lab:dev
from
camillebrianceau:cb_generate_adapt
Closed
Generate pipeline adaptation to data class #582
camillebrianceau
wants to merge
14
commits into
aramis-lab:dev
from
camillebrianceau:cb_generate_adapt
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* first commit for creation of a Predict_manager
* creation of the trainer * remove trainer's methods from MAPSManager * creation of the trainer * introduce trainer in ClinicaDL's train and resume functions * small improvements in docstrings * omission * other omissions
* get trainer out of mapsmanager folder * base training config class * task specific config classes * unit test for config classes * changes in cli to have default values from task config objects * ranem and simplify build_train_dict * unit test for train_utils * small modification in training config toml template * rename build_train_dict in the other parts of the project * modify task_launcher to use config objects * Bump sqlparse from 0.4.4 to 0.5.0 (aramis-lab#558) Bumps [sqlparse](https://github.com/andialbrecht/sqlparse) from 0.4.4 to 0.5.0. - [Changelog](https://github.com/andialbrecht/sqlparse/blob/master/CHANGELOG) - [Commits](andialbrecht/sqlparse@0.4.4...0.5.0) --- updated-dependencies: - dependency-name: sqlparse dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * typo * change _network_task attribute * omissions * patches to match CLI data * small modifications * correction * correction reconstruction default loss * add architecture command specific to the task * add use_extracted_features parameter * add VAE parameters in reconstruction * add condition on whether cli arg is default or from user * correct wrong import in resume * validators on assignment * reformat * replace literal with enum * review on CLI options * convert enum to str for train function * correct track exp issue * test for ci --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Creation of GenerateConfig
* add predictConfig and InterpretConfig
* add Task class * rename task_launcher and put it in train_utils + add merge_cli_and_config_file_options function * remove task_utils * add unit tests * put train call in cli files * remove train function
* reorganize train folder
* first commirt to clean last PR * add enum file + adapt prepare-data
* create get_default_from_config_class and get_type_from_config_class * add enum for losses, transformations, metrics and optimizer * add PositiveInt, PositiveFloat, NonNegativeInt and NonNegtiveFloat typing
* create prepare data config
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.