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

WIP: add SCR to HDF path #126

Open
wants to merge 6 commits into
base: raja
Choose a base branch
from
Open

Conversation

adammoody
Copy link

@adammoody adammoody commented Feb 11, 2022

This starts the work to integrate SCR into the (synchronous) HDF checkpoint path. Since the HDF path writes a single shared file, one would need to run SCR in bypass mode:

SCR_CACHE_BYPASS=1

This is not yet complete.

  • The string manipulation to compute the file suffix needs to be fixed.
  • The getDt path for HDF needs to include SCR calls. I updated getDt to show the SCR calls for HDF, but I dropped the existing SCR getDt code. That would need to be added back, but we'll need a new compile time guard.

Also updated this PR to include SCR in the non-HDF, single file path.

@adammoody adammoody changed the title add SCR to HDF path WIP: add SCR to HDF path Feb 12, 2022
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.

1 participant