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

Test testsuite/python/script_interface.py is flaky #4488

Closed
espresso-ci opened this issue Apr 4, 2022 · 1 comment · Fixed by #4489
Closed

Test testsuite/python/script_interface.py is flaky #4488

espresso-ci opened this issue Apr 4, 2022 · 1 comment · Fixed by #4489
Assignees

Comments

@espresso-ci
Copy link

https://gitlab.icp.uni-stuttgart.de/espressomd/espresso/pipelines/17386

@jngrad jngrad changed the title CI build failed for merged PR Test testsuite/python/script_interface.py is flaky Apr 4, 2022
@jngrad
Copy link
Member

jngrad commented Apr 4, 2022

@jngrad jngrad reopened this Apr 4, 2022
@jngrad jngrad self-assigned this Apr 4, 2022
@kodiakhq kodiakhq bot closed this as completed in #4489 Apr 6, 2022
kodiakhq bot added a commit that referenced this issue Apr 6, 2022
Fixes #4488, fixes  #4490, fixes #4491, fixes #4477

Description of changes:
- require Python 3.8+ (as per #3421)
- use `contextlib` in the testsuite to better handle `ImportError` from missing dependencies
- use `pathlib` in the testsuite and scripts for portability on non-Posix environments
- modernize python testsuite, fix broken test cases, improve code coverage
- implement initial position offset for LEbc protocol `OscillatoryShear`
- use new CI infrastructure: run jobs on 4 cores (reduce test runtime by 30%), update labels to better utilize GPU runners
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 a pull request may close this issue.

2 participants