Add sol outboard enum - #4514
Conversation
…ntegrate power decay length logic in ScrapeOffLayer model
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4514 +/- ##
==========================================
+ Coverage 49.13% 49.32% +0.19%
==========================================
Files 152 150 -2
Lines 29806 29802 -4
==========================================
+ Hits 14644 14700 +56
+ Misses 15162 15102 -60 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
012f5b0 to
d5257cb
Compare
…ayer logic for user input handling
d5257cb to
7dda017
Compare
| if ( | ||
| OutbordSOLPowerDecayLengthModel( | ||
| self.data.physics.i_len_sol_outboard_power_decay | ||
| ) | ||
| == OutbordSOLPowerDecayLengthModel.USER_INPUT | ||
| ): | ||
| self.data.physics.len_sol_outboard_power_decay = ( | ||
| self.data.physics.len_sol_outboard_power_decay | ||
| ) |
There was a problem hiding this comment.
This block of code is not doing anything. Maybe just add a comment saying that OutbordSOLPowerDecayLengthModel.USER_INPUT would mean that len_sol_outboard_power_decay is already set to the user-input value
There was a problem hiding this comment.
Would you say keep the condition check just for verbosity and add a pass statement with a comment or just remove the whole condition check and just have the comment?
There was a problem hiding this comment.
The latter, having 'useless' condition checks is a code smell. A comment will suffice
Description
Checklist
I confirm that I have completed the following checks: