-
Notifications
You must be signed in to change notification settings - Fork 17
Separate shield, vacuum vessel, and blanket calculations #4064
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
Open
chris-ashe
wants to merge
25
commits into
main
Choose a base branch
from
create_shield_class
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
2856654
Add VacuumVessel class and integrate into Caller model
chris-ashe c8f1ad4
Add calculate_dshaped_vessel_volumes method to VacuumVessel class
chris-ashe 223eaa8
Add calculate_elliptical_vessel_volumes method to VacuumVessel class
chris-ashe 134a43c
:fire: Remove now redundant vacuum calcs from blanket library
chris-ashe 25635e0
Refactor BlanketLibrary: remove vacuum vessel calculations and update…
chris-ashe 4dfb4a2
Add tests for DShaped vessel volume calculations in vacuum module
chris-ashe 3252aeb
Add Shield class with calculate_shield_half_height method
chris-ashe c56de6c
Add calculate_dshaped_shield_volumes method to Shield class
chris-ashe 06496a5
Add calculate_dshaped_shield_areas method to Shield class
chris-ashe 64e913d
Add calculate_elliptical_shield_volumes method to Shield class
chris-ashe dcfe308
Add calculate_elliptical_shield_areas method to Shield class
chris-ashe 9ec634c
Add Shield model integration to Caller and Models classes
chris-ashe fae93af
Implement shield area and volume calculations in run method of Shield…
chris-ashe d434e7b
Refactor BlanketLibrary to remove shield component calculations and s…
chris-ashe c03473b
Add unit test for elliptical shield volumes calculation
chris-ashe be1dc32
Add unit test for elliptical shield areas calculation
chris-ashe 2976730
Add unit test for D-shaped shield volumes calculation
chris-ashe e31d32a
Add unit test for D-shaped shield areas calculation
chris-ashe 9c4b508
Remove redundant D-shaped component parameters from test cases
chris-ashe 35f690c
Remove redundant EllipticalComponentParam test cases
chris-ashe 51f8c08
Post rebase changes
chris-ashe c6b1da3
Remove redundant shield surface parameters from ApplyCoverageFactorsP…
chris-ashe 472208a
Requested style changes
chris-ashe dde3844
Add vacuum vessel mass calculation to VacuumVessel class
chris-ashe 6280dc6
Add docstring to vacuum vessel volume calculation method
chris-ashe 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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.
Uh oh!
There was an error while loading. Please reload this page.