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

Fix breaking tests on 4.1.1 #3358

Merged
merged 2 commits into from
Dec 5, 2019
Merged

Fix breaking tests on 4.1.1 #3358

merged 2 commits into from
Dec 5, 2019

Conversation

jngrad
Copy link
Member

@jngrad jngrad commented Dec 5, 2019

Description of changes:

The P3M benchmark test is fragile and breaks regularly in CI, but
only in tutorial-sample jobs. Multiple attempts at fixing the tuning
function did not resolve the issue. Disabling the test for now.
This test fails on openSUSE Tumbleweed i586 due to precision error.
@jngrad jngrad added the DevOps label Dec 5, 2019
@jngrad jngrad added this to the Espresso 4.1.2 milestone Dec 5, 2019
@jngrad jngrad requested a review from fweik December 5, 2019 12:40
@codecov
Copy link

codecov bot commented Dec 5, 2019

Codecov Report

Merging #3358 into python will increase coverage by <1%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           python   #3358    +/-   ##
=======================================
+ Coverage      86%     86%   +<1%     
=======================================
  Files         537     537            
  Lines       25283   25283            
=======================================
+ Hits        21754   21755     +1     
+ Misses       3529    3528     -1
Impacted Files Coverage Δ
src/core/unit_tests/field_coupling_fields_test.cpp 100% <ø> (ø) ⬆️
src/core/electrostatics_magnetostatics/p3m.cpp 85% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d7dab86...7e773e9. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

P3M benchmark randomly fails CI
2 participants