-
Notifications
You must be signed in to change notification settings - Fork 12
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
Update OAT sensitivity analysis #144
Merged
Merged
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
201c304
change parallelization scheme:
abachma2 82acfbf
Merge branch 'master' into sensitivity-analysis
abachma2 fe4ad7d
Merge branch 'master' into sensitivity-analysis
abachma2 2d2a168
create files to vary xe100 and mmr burnup
abachma2 23d844f
add files for MMR definition of different burnups
abachma2 46e729c
correct file names, update variable values to sample
abachma2 8d97dcd
Merge branch 'sensitivity-analysis' of github.com:abachma2/transition…
abachma2 e398617
set up files and file names to run analysis
abachma2 5b3d7d8
Merge branch 'sensitivity-analysis' of github.com:abachma2/transition…
abachma2 85bb910
update bunrup values, correct file paths
abachma2 278fc63
update notebook with burnup results
abachma2 4501c65
Merge branch 'master' into sensitivity-analysis
abachma2 d2f4176
update driver to new function inputs
abachma2 523404b
Merge branch 'master' into sensitivity-analysis
abachma2 cb0ab57
update share drivers to new function input format
abachma2 0ee7fe3
update burnup change drivers for new functions
abachma2 f92b8b8
update analysis for OAT re-runs
abachma2 041cf1c
fix mosaic plots to read labels
abachma2 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -20,10 +20,10 @@ | |
|
||
# Edit Cyclus input file | ||
cyclus_template = 'mmr_burnup_input.xml.in' | ||
scenario_name = 'mmr_burnup' + str(round(params['mmr_burnup'])) | ||
scenario_name = 'mmr_burnup_' + str(int(params['mmr_burnup'])) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. good naming scheme |
||
variable_dict = {'handle': scenario_name, | ||
'mmr_burnup': str(int(params['mmr_burnup']))} | ||
output_xml = './cyclus-files/mmr_burnup' + str(params['mmr_burnup']) + '.xml' | ||
output_xml = './cyclus-files/' + scenario_name + '.xml' | ||
output_sqlite = './cyclus-files/' + scenario_name + '.sqlite' | ||
|
||
inp.render_input(cyclus_template, variable_dict, output_xml) | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Can you explain what this change is? I'm not understanding from the commit message. It could also be my lack of familiarity with dakota.
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.
For the other variations (transition start time, lwr lifetimes, and build shares) I am sweeping over a grid of evenly spaced points. However, the points I want to use for the burnup variations (applied to the MMR or Xe-100) are not evenly spaced. So I am telling Dakota to use these specific values instead of an even grid like I do when varying the parameters.