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

TestBPChangingShapeWithinStep without and with various compressors #3577

Merged
merged 1 commit into from
Apr 17, 2023

Conversation

pnorbert
Copy link
Contributor

@pnorbert pnorbert commented Apr 5, 2023

…on turned on. This is an openPMD use case that needs to be checked.

@pnorbert pnorbert requested a review from lizdulac April 5, 2023 18:21
anagainaru
anagainaru previously approved these changes Apr 6, 2023
Copy link
Contributor

@anagainaru anagainaru left a comment

Choose a reason for hiding this comment

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

Good to know we support this. Maybe we can test this for MGARD as well.

@pnorbert pnorbert requested a review from vicentebolea as a code owner April 7, 2023 16:02
@pnorbert pnorbert changed the title Added a test similar to TestBPChangingShape but with BLOSC2 compressi… TestBPChangingShapeWithinStep without and with various compressors Apr 7, 2023
@pnorbert
Copy link
Contributor Author

pnorbert commented Apr 7, 2023

New test now works with blosc, mgard and zfp (CPU-only) if they are available.

@pnorbert
Copy link
Contributor Author

pnorbert commented Apr 7, 2023

I did not dismiss your review, nor requested new review. Was automatic somehow

@vicentebolea
Copy link
Collaborator

I did not dismiss your review, nor requested new review. Was automatic somehow

Yes I am in the codeowner file for the cmake files. I think I have to limit it so I am not asked to review everywhere haha. It was good during the release but not needed anymore

@vicentebolea vicentebolea removed their request for review April 7, 2023 16:17
anagainaru
anagainaru previously approved these changes Apr 7, 2023
Copy link
Contributor

@anagainaru anagainaru left a comment

Choose a reason for hiding this comment

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

Haha yes, whenever you push a new commit, the reviews are voided :) Approving again, looks good to me.

@pnorbert
Copy link
Contributor Author

pnorbert commented Apr 7, 2023

rebased with master

@pnorbert pnorbert force-pushed the test-changing-shape-blosc2 branch from 47ec16c to 56b94d8 Compare April 7, 2023 18:45
@pnorbert
Copy link
Contributor Author

pnorbert commented Apr 7, 2023

Cleaned-up the commit history after fixing the build on windows. Ready for review and merge after passing the tests.

@pnorbert pnorbert force-pushed the test-changing-shape-blosc2 branch from 56b94d8 to 99dc7e0 Compare April 8, 2023 18:21
Copy link
Contributor

@anagainaru anagainaru left a comment

Choose a reason for hiding this comment

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

Approving it one last time

…ressors to it if they are available (blosc, mgard, zfp).

Tests: Engine.BP.*/BPChangingShapeWithinStep.MultiBlock/*.
@pnorbert pnorbert force-pushed the test-changing-shape-blosc2 branch from 99dc7e0 to 8bbdefa Compare April 17, 2023 02:20
@pnorbert pnorbert merged commit 9a2fcdd into ornladios:master Apr 17, 2023
@pnorbert pnorbert deleted the test-changing-shape-blosc2 branch April 17, 2023 05:13
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.

3 participants