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

Rename ThermalResistance to ThermalInsulance #1460

Conversation

lipchev
Copy link
Collaborator

@lipchev lipchev commented Dec 16, 2024

As part of #1200

@lipchev
Copy link
Collaborator Author

lipchev commented Dec 16, 2024

Note that this is also part of the quantities with rounded expressions (see #1393):

    {
      "SingularName": "SquareCentimeterHourDegreeCelsiusPerKilocalorie",
      "PluralName": "SquareCentimeterHourDegreesCelsiusPerKilocalorie",
      "FromUnitToBaseFunc": "{x} * (0.0001 * 3600) / 4.184",
      "FromBaseToUnitFunc": "{x} * 4.184 / (0.0001 * 3600)",
      "Localization": [
        {
          "Culture": "en-US",
          "Abbreviations": [ "cm²Hr°C/kcal" ]
        }
      ]
    },
    {
      "SingularName": "HourSquareFeetDegreeFahrenheitPerBtu",
      "PluralName": "HourSquareFeetDegreesFahrenheitPerBtu",
      "FromUnitToBaseFunc": "{x} * (1000 * 0.3048 * 0.3048 * 3600) / (1055.05585262 * 1.8)",
      "FromBaseToUnitFunc": "{x} * (1055.05585262 * 1.8) / (1000 * 0.3048 * 0.3048 * 3600)",
      "Localization": [
        {
          "Culture": "en-US",
          "Abbreviations": [ "Hrft²°F/Btu" ]
        }
      ]

@lipchev
Copy link
Collaborator Author

lipchev commented Dec 16, 2024

Ah and by the way- we should consider switching the base unit to the SI unit: SquareMeterKelvinPerWatt (this is one of just several quantities which are don't match the SI base)

@angularsen angularsen merged commit 53e5b84 into angularsen:release/v6 Dec 17, 2024
1 check passed
@angularsen angularsen added this to the v6 milestone Dec 17, 2024
@angularsen angularsen mentioned this pull request Dec 17, 2024
19 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants