Utilize PressureSlope unit in Media consistently

modelica / ModelicaStandardLibrary

Free (standard conforming) library to model mechanical (1D/3D), electrical (analog, digital, machines), magnetic, thermal, fluid, control systems and hierarchical state machines. Also numerical functions and functions for strings, files and streams are included.

https://doc.modelica.org

BSD 3-Clause "New" or "Revised" License

472 stars 169 forks source link

Utilize PressureSlope unit in Media consistently #4262

Closed beutlich closed 8 months ago

beutlich commented 8 months ago

This is a follow-up of the recently merged PRs #4115 (adding unit "Pa/s") and #4103 (introducing a new type PressureRate with unit "Pa/s").

HansOlsson commented 8 months ago

Based on #4236 shouldn't it be PressureSlopeinstead of PressureRate?

That would make Pressure and Temperature more consistent. Based on #4236 FlowRateis for connector variables, whereas Slope is for derivatives, (and plain Rate is used for completely different things).

beutlich commented 8 months ago

Based on #4236 shouldn't it be PressureSlopeinstead of PressureRate?

That would make Pressure and Temperature more consistent. Based on #4236 FlowRateis for connector variables, whereas Slope is for derivatives, (and plain Rate is used for completely different things).

Will do (in this PR).