search

jconlon / LBWhite-heater-estimator

L.B. White Heater Estimator Web Application
0 stars 2 forks source link

Btu’hs s are not the same when showing Fahrenheit and Celsius for same volume room. #23

Closed jconlon closed 4 years ago

jconlon commented 5 years ago

Switching the temperature from Fahrenheit to Celsius, and the Btu/h calculation is not showing the correct calculation. I think it is taking the Celsius temp and calculating it as if it were Fahrenheit.

The Btu’hs should stay the same as long as you are showing the same temp in Fahrenheit and Celsius and the rooms are the same size.

jconlon commented 5 years ago

App is now only doing 2 calculations: Imperial or Metric. Calculation is determined by the Room Temperature setting: Fahrenheit or Celsius.

If the room volume is specified in Feet and the temp in Fahrenheit or the room volume is specified in Meters and the temp in Celsius. This works and probably most all your reps used it that way.

But if the volume is changed between Feet and Meters or Fahrenheit and Celsius the calculation is still only determined by the Fahrenheit and Celsius setting so an incorrect reading. Again most reps probably are just ignoring this and setting compatible volume and temp settings.

So to resolve this issue there are two approaches:

A. Constrain App to only allow only Imperial or Metric calculations with compatible volume and temp settings

  1. Feet and the temp in Fahrenheit
  2. Meters and the temp in Celsius

When either changes the UI changes the other: For example - changing from Meters to Feet will change the temp to Fahrenheit changing from Fahrenheit to Celsius will change the volume calculations from feet to meters.

B. Allow 4 different calculations:

  1. Feet and the temp in Fahrenheit
  2. Meters and the temp in Celsius
  3. Feet and the temp in Celsius
  4. Meters and the temp in Fahrenheit