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

Add tests #24

Merged
merged 19 commits into from
Jan 9, 2022
Merged

Add tests #24

merged 19 commits into from
Jan 9, 2022

Conversation

lucmos
Copy link
Member

@lucmos lucmos commented Jan 9, 2022 •
edited

@lucmos lucmos mentioned this pull request Jan 9, 2022
Merged
@lucmos lucmos marked this pull request as ready for review January 9, 2022 15:10
@lucmos
Disable pytest coverage uncompatible with PyCharm
bc7332c 
See:
- https://stackoverflow.com/questions/40718760/unable-to-debug-in-pycharm-with-pytest
- https://youtrack.jetbrains.com/issue/PY-20186
@lucmos
Update coverage configuration to ignore unreachable code
9cc7610
@lucmos
Avoid executing the model test in fast_dev_run mode
e2b91fa
@lucmos
Add conftest to define basic configurations and training
4608483
@lucmos
Add tests to validate configuration
d95cada
@lucmos
Add train loop tests
6e8bb3d
@lucmos
Optimize fixture scoping
183fda2
@lucmos
Update template to be compatible with the tests
35494e9
@lucmos
Reorganize conftest fixtures
4762185
@lucmos
Test the configuration is correctly stored on the storage folder
b7bb1ec
@lucmos
Test the checkpoint are correctly saved in the storage
ec7e71e
@lucmos
Test the checkpoint can be loaded
6d3e08a
@lucmos
Test the configuration is stored in the checkpoint
325e750
@lucmos
Rename test_resume into test_checkpoint
bbda6a7
@lucmos
Test on save checkpoint monkey patching
30995a0
@lucmos
Test PROJECT_ROOT resolution
c71923f
@lucmos
Test train loop in different regimens
16aed0d
@Flegyas Flegyas merged commit 560314f into develop Jan 9, 2022
@Flegyas Flegyas deleted the feature/tests branch January 9, 2022 16:26
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.

None yet
2 participants
@lucmos @Flegyas