Skip to content

Conversation

franzpoeschel
Copy link
Contributor

@franzpoeschel franzpoeschel commented Mar 28, 2025

Some assorted performance optimizations notices when developing the openPMD plugin for CP2K. These mostly affect the performance of flushing Iterations.

TODO:

  • Check if we should check for dirtyRecursive() in other places, too, or when reading. Update: No need in reading, Iterations are not reparsed.
  • Merge HDF5 filters (compression) #1644 first

@franzpoeschel franzpoeschel changed the title Performance optimizations for interacting with many meshes / species types [WIP] Performance optimizations for interacting with many meshes / species types Mar 28, 2025
@franzpoeschel franzpoeschel force-pushed the performance-optimizations-cp2k branch from b62f3fd to a4df35d Compare March 31, 2025 16:47
@franzpoeschel franzpoeschel force-pushed the performance-optimizations-cp2k branch from a4df35d to 410a82a Compare April 22, 2025 09:35
@franzpoeschel franzpoeschel added this to the 0.16.2 milestone Jun 6, 2025
@franzpoeschel franzpoeschel force-pushed the performance-optimizations-cp2k branch 2 times, most recently from 6f1d04e to e73adad Compare July 21, 2025 09:10
@franzpoeschel franzpoeschel changed the title [WIP] Performance optimizations for interacting with many meshes / species types Performance optimizations for interacting with many meshes / species types Aug 1, 2025
@franzpoeschel franzpoeschel requested a review from ax3l August 1, 2025 12:18
@franzpoeschel franzpoeschel force-pushed the performance-optimizations-cp2k branch 3 times, most recently from 561a76a to fc13164 Compare August 8, 2025 13:41
ByID,
// H5Pset_deflate
Zlib>;

Check warning

Code scanning / CodeQL

Poorly documented large function Warning

Poorly documented function: fewer than 2% comments for a function of 110 lines.
@franzpoeschel franzpoeschel force-pushed the performance-optimizations-cp2k branch from fc13164 to 2e8de04 Compare August 11, 2025 15:08
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