-
Notifications
You must be signed in to change notification settings - Fork 16
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
Cleanup Depcode
class
#172
Merged
Merged
Changes from all commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
4ea7e4f
read_depcode_info() -> read_step_metadata(); sim_info -> step_metadata
yardasol 152806f
update depcode unit test filenames
yardasol 4947cd7
read_depcode_step_param() -> read_neutronics_parameters(); param -> n…
yardasol 94b2647
read_dep_comp() -> read_depleted_materials(); variable name adjustments
yardasol 053b64a
create_nuclide_name_map_zam_to_serpent() -> map_nuclide_code_zam_to_s…
yardasol 93c0a1c
run_depcode() -> run_depletion_step()
yardasol d823203
write_mat_file() -> update_depletable_materials()
yardasol 94f1479
cleanup nuclide code handling functions
yardasol d4f19e8
Merge remote-tracking branch 'upstream/master' into cleanup-depcode
yardasol 475ff1f
change_sim_par() -> apply_neutron_settings(); cleanup read_plaintext_…
yardasol 9ab1823
create_iter_matfile() -> create_runtime_matfile()
yardasol abc61f3
replace_burnup_parameters() -> set_power_load()
yardasol 69ee50d
cleanup switch_to_next_geometry()
yardasol be59037
cleanup insert_path_to_geometry()
yardasol f5aec4e
write_depcode_input() -> write_depletion_step_input()
yardasol 87fed81
attribute name change: iter_* -> runtime_*
yardasol 5d98386
write_depletion_step_input() -> write_runtime_input()
yardasol 0d6aff4
remove default path values in depcode classes
yardasol f6962a6
remove npop, active_cycles, inactive_cycles
yardasol 7d41cd3
fix redudnant path specifier for runtime filenames
yardasol accd451
add npop, active_cycles, inactive_cycles back in; fix some path-relat…
yardasol c2f0852
remove cruft commented code in app.py
yardasol 623cda3
cleanup main docstrings for depcode classes
yardasol ec023d8
cleanup run_depletion_step() in OpenMCDepcode
yardasol 213f498
cleanup write_depletion_settings() and write_runtime_input() in OpenM…
yardasol c690927
cleanup set_power_load() in SerpentDepcode
yardasol 84d12b5
add missing comma
yardasol 692c418
cleanup create_runtime_matfile()
yardasol ac78eec
fix integration tests
yardasol 4a38449
Add test for get_neutron_settings()
yardasol 2cb688a
Merge remote-tracking branch 'upstream/master' into cleanup-depcode
yardasol 4db68bf
Path.as_posix() -> str(Path)
yardasol 660e88f
set uniqueItems=false for geo_file_paths
yardasol 8396c2b
fix typo in file_interface_openmc
yardasol 4a49c11
Apply suggestions from @abachma2 code review
yardasol ae1a740
Apply suggestions from @abachma2 code review
yardasol 647a1d1
Add @samgdotson suggestions.
yardasol 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
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
Oops, something went wrong.
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.
Any reason for this change? Is it because you've added the output path as an input parameter to the Depcode class?
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.
This change just cleans up the path itself. The
./
is not necessary.