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

Particle data #3251

Merged
merged 4 commits into from
Oct 15, 2019
Merged

Particle data #3251

merged 4 commits into from
Oct 15, 2019

Conversation

fweik
Copy link
Contributor

@fweik fweik commented Oct 15, 2019

Fixes #3157.

Description of changes:

  • Split definition of struct Particle from rest of particle data.

@fweik fweik requested a review from jngrad October 15, 2019 12:13
@codecov
Copy link

codecov bot commented Oct 15, 2019

Codecov Report

Merging #3251 into python will decrease coverage by <1%.
The diff coverage is 97%.

Impacted file tree graph

@@          Coverage Diff           @@
##           python   #3251   +/-   ##
======================================
- Coverage      85%     85%   -1%     
======================================
  Files         530     531    +1     
  Lines       25795   25796    +1     
======================================
  Hits        22168   22168           
- Misses       3627    3628    +1
Impacted Files Coverage Δ
src/core/cluster_analysis/Cluster.hpp 100% <ø> (ø) ⬆️
src/core/io/mpiio/mpiio.cpp 84% <ø> (ø) ⬆️
src/core/constraints/Constraint.hpp 100% <ø> (ø) ⬆️
src/core/reaction_ensemble.cpp 86% <ø> (ø) ⬆️
src/core/nonbonded_interactions/thole.hpp 100% <ø> (ø) ⬆️
src/core/electrostatics_magnetostatics/p3m.cpp 86% <ø> (ø) ⬆️
src/core/collision.cpp 79% <ø> (ø) ⬆️
src/core/constraints/HomogeneousMagneticField.hpp 100% <ø> (ø) ⬆️
src/core/communication.hpp 100% <ø> (ø) ⬆️
src/core/pair_criteria/pair_criteria.hpp 100% <ø> (ø) ⬆️
... and 56 more

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 94d51f5...aedadef. Read the comment docs.

@fweik fweik mentioned this pull request Oct 15, 2019
@jngrad
Copy link
Member

jngrad commented Oct 15, 2019

bors r+

bors bot added a commit that referenced this pull request Oct 15, 2019
3248: Fix cmake install on dev branch r=KaiSzuttor a=jngrad

Porting #3228 to the dev branch

Description of changes:
- fix the CMake build system to include libraries in the list of installed files and test it in a dedicated CI job
- fix the versioning of Cython .so shared objects
- simplify the CMake tests


3251: Particle data r=jngrad a=fweik

Fixes #3157.

Description of changes:
 - Split definition of `struct Particle` from rest of particle data.


Co-authored-by: Christoph Junghans <[email protected]>
Co-authored-by: Jean-Noël Grad <[email protected]>
Co-authored-by: Florian Weik <[email protected]>
@bors
Copy link
Contributor

bors bot commented Oct 15, 2019

Build succeeded

@bors bors bot merged commit aedadef into espressomd:python Oct 15, 2019
bors bot added a commit that referenced this pull request Oct 17, 2019
3252: Factor out ParticleList r=jngrad a=fweik

Follow up on #3251.

Description of changes:
 - Pulling `ParticleList` out of `particle_data.hpp` to get better
   header disentanglement.


3256: Remove tutorial 10 and unused LaTeX files r=fweik a=jngrad

Closes #3211

Description of changes:
- removed tutorial 10
- removed unused LaTeX preambles

3257: Remove PDB parser feature r=fweik a=jngrad

The consensus offline at the ICP and online in #1441 is to drop support of the PDB parser feature in favor of the dedicated python package MDAnalysis.

Co-authored-by: Florian Weik <[email protected]>
Co-authored-by: Jean-Noël Grad <[email protected]>
bors bot added a commit that referenced this pull request Oct 17, 2019
3252: Factor out ParticleList r=jngrad a=fweik

Follow up on #3251.

Description of changes:
 - Pulling `ParticleList` out of `particle_data.hpp` to get better
   header disentanglement.


3256: Remove tutorial 10 and unused LaTeX files r=fweik a=jngrad

Closes #3211

Description of changes:
- removed tutorial 10
- removed unused LaTeX preambles

3257: Remove PDB parser feature r=fweik a=jngrad

The consensus offline at the ICP and online in #1441 is to drop support of the PDB parser feature in favor of the dedicated python package MDAnalysis.

Co-authored-by: Florian Weik <[email protected]>
Co-authored-by: Jean-Noël Grad <[email protected]>
@jngrad jngrad added this to the Espresso 4.2 milestone Nov 22, 2019
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.

Split the Particle definition from the rest of particle_data.hpp
2 participants