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

Enable P3M for hybrid decomposition with one MPI node #4678

Conversation

RiccardoFrenner
Copy link
Contributor

Description of changes:

  • removed sanity checks preventing hybrid decomposition from being used with P3M/Dipolar P3M on a single MPI node
  • added tests to ensure forces/energies of the hybrid decomposition with P3M are equal to those calculated using the regular decomposition

@jngrad jngrad force-pushed the hybrid_decomp_enable_p3m_single_node branch from 98a522b to ea7861b Compare March 1, 2023 17:48
Copy link
Member

@jngrad jngrad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@jngrad jngrad added this to the Espresso 4.2.1 milestone Mar 1, 2023
@jngrad jngrad added Core automerge Merge with kodiak labels Mar 1, 2023
@kodiakhq kodiakhq bot merged commit e031659 into espressomd:python Mar 1, 2023
jngrad pushed a commit to jngrad/espresso that referenced this pull request Mar 2, 2023
Description of changes:
- removed sanity checks preventing hybrid decomposition from being used with P3M/Dipolar P3M on a single MPI node
- added tests to ensure forces/energies of the hybrid decomposition with P3M are equal to those calculated using the regular decomposition
jngrad pushed a commit to jngrad/espresso that referenced this pull request Mar 2, 2023
Description of changes:
- removed sanity checks preventing hybrid decomposition from being used with P3M/Dipolar P3M on a single MPI node
- added tests to ensure forces/energies of the hybrid decomposition with P3M are equal to those calculated using the regular decomposition
jngrad pushed a commit to jngrad/espresso that referenced this pull request Mar 2, 2023
Description of changes:
- removed sanity checks preventing hybrid decomposition from being used with P3M/Dipolar P3M on a single MPI node
- added tests to ensure forces/energies of the hybrid decomposition with P3M are equal to those calculated using the regular decomposition
jngrad pushed a commit to jngrad/espresso that referenced this pull request Mar 2, 2023
Description of changes:
- removed sanity checks preventing hybrid decomposition from being used with P3M/Dipolar P3M on a single MPI node
- added tests to ensure forces/energies of the hybrid decomposition with P3M are equal to those calculated using the regular decomposition
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge with kodiak Core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants