-
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
Add radiation tools tests #3394
Add radiation tools tests #3394
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #3394 +/- ##
===========================================
- Coverage 74.57% 74.36% -0.21%
===========================================
Files 231 231
Lines 26810 26810
===========================================
- Hits 19993 19937 -56
- Misses 6817 6873 +56 ☔ View full report in Codecov by Sentry. |
|
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.
A little comment/documentation would help but code-wise, this looks good :)
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.
I agree with @OceanNuclear's comment.
Also this doesnt close #3330, there are some more tests to be written specifically something that calls FirstWallRadiationSolver. It should be a very simple case with only a couple of detectors.
fcc5d0c
to
465c428
Compare
Added test for FirstWallRadiationSolver (but the test requires a bit of time to run) |
465c428
to
bcbef15
Compare
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.
Overall nearly there just the one comment, I've added a few customisations to the Solver to allow us to customise it to make the test quicker.
This will also need a rebase for the tests to pass because of unrelated bugs that are now fixes
7142c09
into
Fusion-Power-Plant-Framework:develop
Linked Issues
Description
Added tests for the following
Interface Changes
Checklist
I confirm that I have completed the following checks:
pytest tests --reactor
pre-commit run --from-ref develop --to-ref HEAD
sphinx-build -W documentation/source documentation/build