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

Monitor Memory Usage in Tests to figure out heavy misuse #321

Open
SteffenSeckler opened this issue Jul 31, 2019 · 2 comments
Open

Monitor Memory Usage in Tests to figure out heavy misuse #321

SteffenSeckler opened this issue Jul 31, 2019 · 2 comments
Labels
CI GitHub actions

Comments

@SteffenSeckler
Copy link
Member

Is your feature request related to a problem? Please describe.
Some verlet-type containers use too much memory. This can only partially be checked beforehand.

Describe the solution you'd like
Add a check for different numbers of particles and add some sort of memory limit.

Additional context
Problematic configuration:

  • VarVerletLists, SoA, disabled Newton3
@HobbyProgrammer
Copy link
Contributor

The configuration you mentioned uses way to much memory only when using many threads. Might be helpful to add that to the configuration description. See #322 for the reason.

@FG-TUM FG-TUM added the CI GitHub actions label Apr 18, 2024
@FG-TUM
Copy link
Member

FG-TUM commented Apr 18, 2024

Alternative: Compare memory usage of master vs PR. If delta > some limit fail.

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

No branches or pull requests

3 participants