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: Adios io #2739

Closed
wants to merge 21 commits into from
Closed

WIP: Adios io #2739

wants to merge 21 commits into from

Conversation

pnorbert
Copy link
Collaborator

Just showing diff for discussion.

pnorbert and others added 12 commits August 23, 2023 12:27
In general file I/O maps a logically rectangular subset of
the local (nx, ny, nz) array onto a part of a global array.
That global array may be sparse i.e. there might be holes in it.

The mapping does not cover the entire local array because we
insert communication guard cells around the outside.

To perform this mapping we use ADIOS Put() function to get
a span of a buffer, and then iterate over fields to copy data
into those buffers.
ADIOS2 and array index mapping
…). Add helper functions to avoid redefining variables in an adios2::IO object.
@bendudson bendudson changed the base branch from master to next September 13, 2023 00:01
@bendudson bendudson changed the base branch from next to master September 13, 2023 00:05
…for restart files when using ADIOS. Replaced the use of Span and manual copy for memory selections to write data without ghost cells.
@bendudson
Copy link
Contributor

This will need to go into the next branch rather than master, but first we need to sort out merging master into next. Waiting for #2720

The Options tree passed to this function contains the data to be
written to file, not necessarily the input settings.

The output file name should be set from the Options::root() input
options, not the input options tree.
Using autoconf this should be defined, but is always "no"
Haven't yet regenerated configure script
Generated new configure script, updated autoconf_build_defines.hxx.in
Needs to be added to the makefile to be built.
Add --has-adios option to bout-config, and require in
test-options-adios integrated test.
@bendudson
Copy link
Contributor

Failures now seem unrelated to changes in this PR

@bendudson bendudson mentioned this pull request Oct 12, 2023
@bendudson
Copy link
Contributor

Replaced by #2766

@bendudson bendudson closed this Oct 12, 2023
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.

2 participants