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

Add radiation tools tests #3394

Merged

Conversation

athoynilimanew
Copy link
Contributor

@athoynilimanew athoynilimanew commented Jun 25, 2024

Linked Issues

Description

Added tests for the following

  • interpolated_field_values
  • filtering_in_or_out
  • pfr_filter
  • make_wall_detectors
  • FirstWallRadiationSolver

Interface Changes

Checklist

I confirm that I have completed the following checks:

  • Tests run locally and pass pytest tests --reactor
  • Code quality checks run locally and pass pre-commit run --from-ref develop --to-ref HEAD
  • Documentation built locally and checked sphinx-build -W documentation/source documentation/build

@athoynilimanew athoynilimanew requested a review from a team as a code owner June 25, 2024 10:54
Copy link

codecov bot commented Jun 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.36%. Comparing base (b1cf23b) to head (3aff10a).
Report is 57 commits behind head on develop.

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.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

github-actions bot commented Jun 25, 2024

⚠️ Warning Report

Found 0 new warnings, 0 fixed warnings. 🎉

All warnings (5)

On runtest

  • /home/runner/miniconda3/envs/bluemira/lib/python3.10/site-packages/ffcx/element_interface.py:23: DeprecationWarning: Use of elements created by UFL is deprecated. You should create elements directly using Basix.
  • /home/runner/miniconda3/envs/bluemira/lib/python3.10/site-packages/ffcx/element_interface.py:26: DeprecationWarning: Converting elements created in UFL to Basix elements is deprecated. You should create the elements directly using basix.ufl.element instead
  • /home/runner/miniconda3/envs/bluemira/lib/python3.10/site-packages/basix/ufl.py:1909: DeprecationWarning: Converting elements created in UFL to Basix elements is deprecated. You should create the elements directly using basix.ufl.element instead
  • /home/runner/miniconda3/envs/bluemira/lib/python3.10/site-packages/ufl/core/expr.py:275: DeprecationWarning: Expr.ufl_domain() is deprecated, please use extract_unique_domain(expr) instead.

On collect

  • /home/runner/miniconda3/envs/bluemira/lib/python3.10/site-packages/ufl/core/ufl_type.py:56: DeprecationWarning: attach_operators_from_hash_data deprecated, please use UFLObject instead.

@athoynilimanew athoynilimanew changed the title WIP: Papercut Add few tests Papercut add few tests Jun 25, 2024
Copy link
Contributor

@OceanNuclear OceanNuclear left a 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 :)

@je-cook je-cook changed the title Papercut add few tests Add radiation tools tests Jun 26, 2024
@je-cook je-cook self-assigned this Jun 26, 2024
@je-cook je-cook added the radiation_transport Tasks relating to radiation transport label Jun 26, 2024
Copy link
Contributor

@je-cook je-cook left a 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.

@athoynilimanew
Copy link
Contributor Author

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.

Added test for FirstWallRadiationSolver (but the test requires a bit of time to run)

Copy link
Contributor

@je-cook je-cook left a 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

@je-cook je-cook enabled auto-merge (squash) July 11, 2024 14:28
@je-cook je-cook merged commit 7142c09 into Fusion-Power-Plant-Framework:develop Jul 11, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
radiation_transport Tasks relating to radiation transport
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants