Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Additional units for sensor #212

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

RaphaelGebhart
Copy link
Contributor

@RaphaelGebhart RaphaelGebhart commented Sep 30, 2024

Might not be necessary due to #215

I suggest to add:

  • h in kJ/kg
  • s in kJ/(kg.K)
  • M in kg/kmal
  • H_flow in kW
  • H_flow in MW

because this will yield 'nicer' values for the sensor display, e.g. the specific enthalpy of vaporization of water is always given as 2257 kJ/kg and never as 2257000 J/kg

Actually i would prefer a version, where we differentiate between display of the sensor (in non SI units) and output of a sensor (always in SI units) but this would cause major changes.

…H_flow in MW since e.g. h = 2200 kJ/kg is nicer for the sensor display than h = 2200000 J/kg
@RaphaelGebhart RaphaelGebhart self-assigned this Oct 14, 2024
@RaphaelGebhart RaphaelGebhart marked this pull request as draft October 14, 2024 16:23
@RaphaelGebhart
Copy link
Contributor Author

Might not be necessary due to #215

@tobolar tobolar removed their request for review October 21, 2024 09:26
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant