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

Adds HDF5 option to use subfiling #644

Merged
merged 6 commits into from
Jan 12, 2024
Merged

Conversation

brtnfld
Copy link
Contributor

@brtnfld brtnfld commented Jun 28, 2023

@brtnfld
Copy link
Contributor Author

brtnfld commented Aug 15, 2023

@streeve, I've finished the advanced HDF5 example.

@codecov
Copy link

codecov bot commented Aug 15, 2023

Codecov Report

Merging #644 (d672f07) into master (092aa5d) will increase coverage by 0.0%.
The diff coverage is 100.0%.

@@          Coverage Diff           @@
##           master    #644   +/-   ##
======================================
  Coverage    95.2%   95.2%           
======================================
  Files          51      51           
  Lines        6297    6309   +12     
======================================
+ Hits         5997    6009   +12     
  Misses        300     300           
Files Changed Coverage Δ
core/src/Cabana_HDF5ParticleOutput.hpp 99.3% <100.0%> (+<0.1%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@brtnfld
Copy link
Contributor Author

brtnfld commented Jan 9, 2024

I don't think CI will test these PR features since it needs HDF5 > =1.14, and I don't think any distros have it by default.

@brtnfld
Copy link
Contributor Author

brtnfld commented Jan 10, 2024

Added to CI: building HDF5 version 1.14.3 for testing with subfiling.

@streeve
Copy link
Member

streeve commented Jan 12, 2024

Just rebased and pushed (I kept a backup copy of your branch as well) - one quick question for when subfiling isn't available and I think it's ready to merge. Thanks for all your time on this!

@brtnfld
Copy link
Contributor Author

brtnfld commented Jan 12, 2024

What was the question about if subfiling is not available?

Copy link
Member

@streeve streeve left a comment

Choose a reason for hiding this comment

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

Sorry, this comment got left in "pending" - looks like the CI worked, so this is ok with me

example/core_tutorial/CMakeLists.txt Show resolved Hide resolved
@streeve
Copy link
Member

streeve commented Jan 12, 2024

HIP timeout unrelated

@streeve streeve merged commit 9e099a6 into ECP-copa:master Jan 12, 2024
33 of 34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants