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

More stable run_from_experiment test #2419

Merged
merged 2 commits into from
May 8, 2024

Conversation

Ericgig
Copy link
Member

@Ericgig Ericgig commented May 7, 2024

Description
The test for the new method run_from_experiment has a time step too long for the euler method. This would cause fail once a few hundreds time as in https://github.com/qutip/qutip/actions/runs/8979789558/job/24662362047.
Reducing the system size with smaller steps makes the test more consistent (no failure in 20000 tries.)

Also reduced test_feedback step size to improve that test consistency.

@coveralls
Copy link

Coverage Status

coverage: 86.09%. first build
when pulling 8304a3a on Ericgig:misc.better_stoc_test
into 94f7392 on qutip:master.

@Ericgig Ericgig merged commit d8c6584 into qutip:master May 8, 2024
11 of 12 checks passed
@Ericgig Ericgig deleted the misc.better_stoc_test branch May 8, 2024 13:35
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.

None yet

3 participants