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

QoL: better logging for user-defined heating #143

Merged
merged 1 commit into from
May 23, 2023
Merged

Conversation

n-claes
Copy link
Owner

@n-claes n-claes commented May 12, 2023

PR description

Minor Quality-of-Life update, Legolas now throws a proper warning if a user-defined heating function is provided while forced thermal balance is enabled. This is done right before the equilibrium balance equations are checked, after info is logged to the console.

@n-claes n-claes added the QoL Quality of Life changes label May 12, 2023
@n-claes n-claes added this to the Legolas 2.1 milestone May 12, 2023
@codecov
Copy link

codecov bot commented May 12, 2023

Codecov Report

Merging #143 (0c3cb73) into develop (71f15ec) will decrease coverage by 0.01%.
The diff coverage is 66.66%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #143      +/-   ##
===========================================
- Coverage    90.79%   90.78%   -0.01%     
===========================================
  Files          159      159              
  Lines         9048     9050       +2     
===========================================
+ Hits          8215     8216       +1     
- Misses         833      834       +1     
Flag Coverage Δ
legolas 93.10% <66.66%> (-0.02%) ⬇️
pylbo 86.70% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@n-claes n-claes force-pushed the qol/usr_H_warning branch from acd1798 to 0c3cb73 Compare May 12, 2023 11:56
@n-claes n-claes merged commit 2a94ba9 into develop May 23, 2023
@n-claes n-claes deleted the qol/usr_H_warning branch May 23, 2023 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
QoL Quality of Life changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant