-
Notifications
You must be signed in to change notification settings - Fork 2
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
Change "thermal" to "heat" (in most places) #21
Conversation
You need to change the directory and file names from "Thermal" to "Heat" too, in "ThermalBoundaryConditions" at least |
thank you! |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #21 +/- ##
=======================================
Coverage 22.80% 22.80%
=======================================
Files 11 11
Lines 307 307
=======================================
Hits 70 70
Misses 237 237
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you want to change the name of IceWaterThermalEquilibrium as well?
I thought "thermal equilibrium" made sense to keep since that's the typical expression. For fluxes though we usually say "heat flux" not "thermal flux", that motivates the changes we've put in so far. What do you think? |
I agree, that's how I've usually heard fluxes / equilibrium referred to as well, just wanted to make sure. |
great! |
After a bit of rumination I think we should use the term "heat flux" not "thermal flux". It's more standard. Sorry for any pain this causes...
@jlk9