fusion-energy / openmc-plasma-source

Creates a plasma source as an openmc.source object from input parameters that describe the plasma

MIT License

27 stars 11 forks source link

Fix test randomly failing #67

Closed RemDelaporteMathurin closed 2 years ago

RemDelaporteMathurin commented 2 years ago

This PR fixes #64 by making sure ion_density_centre and ion_density_pedestal are different.

I don't understand why this didn't show up before though!

codecov[bot] commented 2 years ago

Codecov Report

Merging #67 (7146054) into develop (f5a24b6) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #67   +/-   ##
========================================
  Coverage    96.29%   96.29%           
========================================
  Files            7        7           
  Lines          216      216           
========================================
  Hits           208      208           
  Misses           8        8

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

RemDelaporteMathurin commented 2 years ago

@shimwell can we merge this?

shimwell commented 2 years ago

looks good to me, thanks for fixing the tests