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

Plotting improvements #55

Merged

Conversation

LiamPattinson
Copy link
Contributor

Upgraded TokamakSource plotting functions and expanded tests

  • Can now pass Axes into plotting functions, can be used when the user wishes to make use of subplots etc.
  • Operations converted to make better use of NumPy
  • Better error reporting from plotting functions
  • Plotting tests greatly expanded, now produces example plots. This currently creates about 25 pngs in the tests folder, which may not be appropriate behaviour.

- Can now pass Axes into plotting functions, can be used when the
  user wishes to make use of subplots etc.
- Operations converted to make better use of NumPy
- Better error reporting from plotting functions
- Plotting tests greatly expanded, now produces example plots
@codecov
Copy link

codecov bot commented Feb 11, 2022

Codecov Report

Merging #55 (af1fa5e) into develop (6944d70) will increase coverage by 0.26%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #55      +/-   ##
===========================================
+ Coverage    96.07%   96.34%   +0.26%     
===========================================
  Files            7        7              
  Lines          204      219      +15     
===========================================
+ Hits           196      211      +15     
  Misses           8        8              
Impacted Files Coverage Δ
...enmc_plasma_source/plotting/plot_tokamak_source.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6944d70...af1fa5e. Read the comment docs.

incorrect quantity testing for scatter_tokamak_source was
shadowed by the equivalent test for plot_tokamak_source_3D
Copy link
Member

@RemDelaporteMathurin RemDelaporteMathurin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

very nice refactor here!

Thanks for also extending the tests!

@RemDelaporteMathurin RemDelaporteMathurin merged commit 8d4c3c7 into fusion-energy:develop Feb 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants