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

API ResultsReader to MATLAB #290

Merged
merged 7 commits into from
Mar 7, 2019

Conversation

drewejohnson
Copy link
Collaborator

Works to address #282 by adding a toMatlab method for exporting content of the ResultsReader to matlab.
The only thing that should be noted is the group constants are stored not as 2D matrices, as they would if you loaded the .mat file directly. Instead, the are constucted be burnup, universe index, <groups>, value/unc. Here, groups means either number of macro/micro energy groups, or a sub-2D matrix in the case of scattering

@drewejohnson drewejohnson added the API - Compatible Changes to our API that require no user actions label Mar 6, 2019
@drewejohnson drewejohnson added this to the 0.7.0 milestone Mar 6, 2019
drewejohnson and others added 6 commits March 7, 2019 08:41
Need to figure out the way to collect homogenized universe
data best...
Directly export the contents of a results reader to a
matlab file using scipy.io.savemat.

The group constant data will be presented as a multi-dimensional
array, rather than a stacked 2D matrix. The axis are ordered
``burnup, universeIndex, group, value/uncertainty``
Moving instead to two functions in s/p/results.py:
getMixedCaseName and getSerpentCaseName
Checks metadata, resdata, and universe data
uniquely and (hopefully) completely
Explain the ordering of universes and remove "object"
as first argument
Instead use MatlabConverter to perform the conversions.
@drewejohnson drewejohnson merged commit 943691b into CORE-GATECH-GROUP:develop Mar 7, 2019
@drewejohnson drewejohnson deleted the res-to-matlab branch March 7, 2019 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API - Compatible Changes to our API that require no user actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants