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

Feature/refactor tests #67

Merged
merged 49 commits into from
Sep 2, 2022
Merged

Feature/refactor tests #67

merged 49 commits into from
Sep 2, 2022

Conversation

sdahdah
Copy link
Member

@sdahdah sdahdah commented Sep 2, 2022 •
edited
Loading

Fixes #57 and #59

Proposed Changes

  • Refactor unit tests so it's easy to add new ones.
  • Re-implement regression tests with pytest-regressions plugin.
  • Allow MOSEK tests to be run remotely with --remote option. These tests are marked with @pytest.mark.mosek.
  • Rename predict_state() to predict_trajectory(). predict_state() ever made it into a release, so there will be no deprecation cycle for it.
  • Remove MOSEK dependence from examples and doctests so they can be run in CI.

sdahdah added 30 commits August 27, 2022 13:48
@sdahdah
Remove unneeded import of numpy in examples
3512ef5
@sdahdah
Add pytest-regressions to requirements
ff5cd70
@sdahdah
Update intersphinx URLs
2186547
@sdahdah
Stop plots from appearing when run from pytest, but keep them when ru…
8841adb 
…nning script
@sdahdah
Refactor sklearn compatibility tests
450abf4
@sdahdah
Start splitting unit tests by module. Created temp folders for now
9b68a8b
@sdahdah
Remove duplicate tests from temporary folder
bb438e2
@sdahdah
Move MOSEK solver parameter setting to a fixture
83b235f
@sdahdah
Move angle preprocessor test to utils
30308ab
@sdahdah
Adjust parameterization of angle feature test
c8223e1
@sdahdah
Refactor angle preprocessor tests to parameterize by class
61157be
@sdahdah
Add doc for class attribute
d7d152d
@sdahdah
Refactor delay lifting function tests
9f90485
@sdahdah
Fully refactor lifting function unit tests
a84f857
@sdahdah
Refactor SplitPipeline and lift/retract testing
697d41c
@sdahdah
Add :mod: to module docstrings
0c8f3a7
@sdahdah
Refactor Koopman pipeline transformer tests
94b5ca0
@sdahdah
Add more info to conftest
dc63ad0
@sdahdah
Refactor multistep prediction and add fixture
5648d0f
@sdahdah
Refactor episode manipulation tests
4c60705
@sdahdah
Correct Koopman matrix in fixture
137af57
@sdahdah
Fully refactor Koopman pipeline
2e4046e
@sdahdah
Rename pre-commit Git hook
bea159a
@sdahdah
Refactor HinfZpkMeta tests
49bf5be
@sdahdah
Tweak formatting of pre-commit hook
5dab59a
@sdahdah
Update test names
7495962
@sdahdah
Refactor LMI regressions with known answers
08eb284
@sdahdah
Refactor LMI Tikhonvov regressor tests
74fd2e0
@sdahdah
Refactor exact tests of EDMD, DMDc, DMD
7cef7a8
@sdahdah
Explicitly highlight prediction
0035f2f
@sdahdah sdahdah self-assigned this Sep 2, 2022
@sdahdah sdahdah linked an issue Sep 2, 2022 that may be closed by this pull request
Fix remote MOSEK OptServer tests #59
Closed
@sdahdah sdahdah merged commit 0c2d473 into main Sep 2, 2022
@sdahdah sdahdah deleted the feature/refactor-tests branch September 2, 2022 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@sdahdah sdahdah

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Fix remote MOSEK OptServer tests Refactor unit tests
1 participant
@sdahdah