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

Cleanup myconfig features #3740

Merged
merged 8 commits into from
May 28, 2020
Merged

Conversation

jngrad
Copy link
Member

@jngrad jngrad commented May 28, 2020

Description of changes:

  • remove unused features
  • improve documentation of interaction features

@jngrad jngrad added this to the Espresso 4.1.3 milestone May 28, 2020
@jngrad jngrad requested a review from fweik May 28, 2020 20:34
@codecov
Copy link

codecov bot commented May 28, 2020

Codecov Report

Merging #3740 into python will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           python   #3740   +/-   ##
======================================
  Coverage      89%     89%           
======================================
  Files         552     552           
  Lines       24513   24513           
======================================
  Hits        21836   21836           
  Misses       2677    2677           
Impacted Files Coverage Δ
src/core/energy_inline.hpp 84% <ø> (ø)
src/core/forces_inline.hpp 91% <ø> (ø)
src/core/particle_data.cpp 96% <0%> (-1%) ⬇️
src/core/electrostatics_magnetostatics/p3m.cpp 87% <0%> (+<1%) ⬆️

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 b17a8d5...da19c22. Read the comment docs.

@jngrad jngrad added the automerge Merge with kodiak label May 28, 2020
@kodiakhq kodiakhq bot merged commit 158732a into espressomd:python May 28, 2020
@jngrad jngrad deleted the remove-unused-features branch January 18, 2022 12:13
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.

2 participants