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

Options for a deterministic behaviour #174

Closed
wants to merge 28 commits into from

Conversation

14thibea
Copy link
Contributor

@14thibea 14thibea commented Aug 2, 2021

To be merged after #171 .

Elina Thibeau Sutre and others added 27 commits June 25, 2021 17:25
* Major refactoring

* Fix test_cli

* Fix tsvtools

* Fix test_tsvtools

* Remove broken import

* Fix train

* Fix random_search
Introduction of the Maps manager.

* Major refactoring

* Fix test_cli

* Fix tsvtools

* Fix test_tsvtools

* Remove broken import

* Fix train

* Fix random_search

* Add MapsManager

* First draft

* Functional version for single CNN, train / resume / predict

* Add prediction log

* Add prediction.log reader

* Add split manager

* add multi flag

* use MapsManager in command line

* Remove previous tests if failed

* Fix train

* Fix test outputs

* Fix resnet18

* Fix random search

* Fix imports

* Fix interpretability

* Fix predict

* Remove mixed slice dataset

* Addition of task manager

* Move sampler generation in task_manager

* Fix trainers with hard-coded example

* Add docstrings

* Fix CNN training

* Fix AE test

* Remove redundancy and fix tests

* Remove legacy

* Evaluate data management with split manager

* Fix regression test

* Fix command line errors in train

* Fix interpretation

* Fix random search

* Fix import

* Edit soft link in the CI pipeline to find new models

* Fix symbolic link to new models in CI script

* Refactoring + minor fixes

* Fix cli in tests

* Add dataset used in predict test, in the CI script

* Fix interpretation

* Adapt resume detection to new MAPS

Co-authored-by: Mauricio DIAZ <[email protected]>
@pep8speaks
Copy link

pep8speaks commented Aug 2, 2021

Hello @14thibea! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 19:1: E402 module level import not at top of file

Line 1089:86: W291 trailing whitespace

Line 9:1: W391 blank line at end of file

Line 41:1: E122 continuation line missing indentation or outdented
Line 51:1: E122 continuation line missing indentation or outdented
Line 56:1: E122 continuation line missing indentation or outdented

Comment last updated at 2021-08-05 08:19:52 UTC

@14thibea 14thibea closed this Aug 6, 2021
@14thibea 14thibea deleted the random_seed branch August 25, 2021 08:36
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.

3 participants