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

Support for HDF5 I/O format in E3SM #6046

Merged
merged 2 commits into from
Dec 6, 2023
Merged

Support for HDF5 I/O format in E3SM #6046

merged 2 commits into from
Dec 6, 2023

Conversation

dqwu
Copy link
Contributor

@dqwu dqwu commented Nov 2, 2023

Adding support for the HDF5 I/O format which is already available
in SCORPIO.

Also using HDF5_ROOT in buildlib.spio to turn on HDF5 support.

[BFB]

@dqwu dqwu added BFB PR leaves answers BFB SCORPIO The E3SM I/O library (derived from PIO) labels Nov 2, 2023
@dqwu dqwu requested review from jgfouca and jayeshkrishna November 2, 2023 20:19
Copy link

github-actions bot commented Nov 2, 2023

PR Preview Action v1.4.4
🚀 Deployed preview to https://E3SM-Project.github.io/E3SM/pr-preview/pr-6046/
on branch gh-pages at 2023-12-04 19:21 UTC

@dqwu
Copy link
Contributor Author

dqwu commented Nov 2, 2023

@jayeshkrishna Maybe we can add support for HDF5 in MPAS with another PR later, similar to #5490.

jgfouca added a commit that referenced this pull request Nov 16, 2023
Support for HDF5 I/O format in E3SM

Adding support for the HDF5 I/O format which is already available
in SCORPIO.

Also using HDF5_ROOT in buildlib.spio to turn on HDF5 support.

[BFB]

* origin/dqwu/add_hdf5_iotype:
  Removing redundant env var WITH_HDF5_SCORPIO in buildlib.spio
  Adding support for HDF5 iotype
@dqwu
Copy link
Contributor Author

dqwu commented Nov 16, 2023

@jgfouca
It seems that you have merged this PR to next branch. Maybe you need revert that merge for now. There is a potential merge conflict with PR #6069. I will rebase this PR after PR 6069 is merged.

@dqwu
Copy link
Contributor Author

dqwu commented Nov 16, 2023

This PR will be rebased and re-merged to next branch after PR #6069 is merged to E3SM master.

@jgfouca
Copy link
Member

jgfouca commented Dec 1, 2023

@dqwu , OK, #6069 is on master now.

dqwu added 2 commits December 4, 2023 13:17
Add HDF5 typename to valid values of PIO_TYPENAME and support
conversion of HDF5 typename to PIO type.
If HDF5_ROOT is already set as an env var, it is sufficient to turn
on the WITH_HDF5 CMake option for scorpio. Introducing a new env var
like WITH_HDF5_SCORPIO may lead to unnecessary confusion.

Additionally, for now, maintain the WITH_HDF5 CMake option as OFF by
default to prevent potential build errors with the default scorpio
1.4.2 submodule.
@dqwu dqwu force-pushed the dqwu/add_hdf5_iotype branch from b0d9e51 to 34b0875 Compare December 4, 2023 19:20
@dqwu
Copy link
Contributor Author

dqwu commented Dec 4, 2023

@dqwu , OK, #6069 is on master now.

@jgfouca Now you can re-merge this rebased PR to next branch, thanks.

jgfouca added a commit that referenced this pull request Dec 4, 2023
Adding support for the HDF5 I/O format which is already available
in SCORPIO.

Also using HDF5_ROOT in buildlib.spio to turn on HDF5 support.

[BFB]
@jgfouca
Copy link
Member

jgfouca commented Dec 4, 2023

Merged to next.

@jgfouca jgfouca merged commit d1e4786 into master Dec 6, 2023
@jgfouca jgfouca deleted the dqwu/add_hdf5_iotype branch December 6, 2023 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BFB PR leaves answers BFB SCORPIO The E3SM I/O library (derived from PIO)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants