-
Notifications
You must be signed in to change notification settings - Fork 7
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
Read4D #43
base: main
Are you sure you want to change the base?
Read4D #43
Conversation
n.b. there are lots of commits here, as I made a bit of a meal of the implementation details |
transient_outputs = [] | ||
executable = './read.exe' | ||
|
||
@unittest.skipIf(os.environ.get('MVER', '') == 'XIOS/trunk@2252', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A comment here about why this is being skipped for r2252 would be useful
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there is a bug in XIOSr2252
which is causing problems with reading dimensionality from these 4D inputs
it's still not clear all of the conditions which lead to this, certainly some reads work, but this case fails.
it's a known error, and fixed in newer XIOS2
revisions
I've added a code comment detailing the failure mode within the code, as requested
exploring multi-dimensional read behaviour
this new example adds a 4D structured input file and simply reads data from that file into memory
This is designed to be extensible, enabling much larger files to be read, across multiple client and server ranks, and performance analyses undertaken