-
Notifications
You must be signed in to change notification settings - Fork 133
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: Xarray DataObjects #183
Conversation
…t it is unit tested
This can be used if the user does not have access to change /opt
Script update
It was pointed out, we should wait on this merge request until the DataObject is close to needing it, since it forces the users to update their libraries. |
Job New Windows install procedure |
Test checklist:
|
We determined that cached XArray was not sufficiently fast for our needs, and instead will use one structure for collecting data, then convert that data into an XArray. This is no longer needed. |
Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
First step in addressing #182.
What are the significant changes in functionality due to this change request?
This merge request wraps the powerful
XArray.Dataset
high-density field data container with a caching system to alleviate the cost of "appending" to the data object.To do this, the new libraries
xarray
andpandas
are added as RAVEN dependencies, which leads to creating a script for the moose_build regression test recipe that will set up the appropriate RAVEN conda environments, rather than force the existing conda package to expand.Lastly, a small change is added that will allow conda to exist both in a home directory as well as
/opt
.For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.
<internalParallel>
to True.