-
Notifications
You must be signed in to change notification settings - Fork 8
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
Commissioning #19
base: master
Are you sure you want to change the base?
Commissioning #19
Conversation
Respects joint limits if provided.
Codecov Report
@@ Coverage Diff @@
## master #19 +/- ##
==========================================
- Coverage 91.66% 85.19% -6.48%
==========================================
Files 6 6
Lines 396 412 +16
==========================================
- Hits 363 351 -12
- Misses 33 61 +28
Continue to review full report at Codecov.
|
tests/test_icrestimator.py
Outdated
@@ -82,6 +102,79 @@ def test_estimate_lambda(): | |||
# assert np.allclose(desired_lmda, lmda_e.T, atol=tolerance) | |||
|
|||
|
|||
@pytest.mark.skip("Closeness calculations not quite working yet") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pytest.mark.xfail instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It only fails some of the time, because sometimes the random values are bad.
It would be better to change them to fixed "errors" to get deterministic behaviour.
Do you still thing xpass is more appropriate in this case?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, I see. The pytest docs seem to suggest that marking as xfail with strict=False is an appropriate way of marking a flaky test. An xfail mark will still allow the test to run.
…r different time steps)
…es error per module
No description provided.