emoncms / MyHomeEnergyPlanner

My Home Energy Planner - Open Source home energy assessment software based on emoncms framework + openbem
GNU Affero General Public License v3.0
22 stars 27 forks source link

NaN value shown for B' in floor u-value calculator for slab on ground. #370

Closed ghost closed 6 years ago

ghost commented 6 years ago

Not sure if this is an error in the calc (from those I've tested, don't think so) or just a display error.

cagabi commented 6 years ago

This is an error in the calculation. NaN stands for Not A Number. So don't trust the u-value calculated in those cases as B is not a number!!!!

Anyway B calculation is such a simple thing that I cannot think what the error is (as far as the area and perimeter has been inputted)

Can you please try to replicate the problem and:

That is the object that the user interface passes to the calculator. Now you know what to do when something looks dodgy with the calculator ;)

Thank you

cagabi commented 6 years ago

Wait I think I got it, When perimeter and area already in MHEP and you open the calculator NaN is shown. As you soon as you change any of them, the Nan goes. Even if you don't change them the calculation is properly made, so no need to worry for now, I think.

I'll fix it now

cagabi commented 6 years ago

So if the problem is what I say in previous message then I have fixed it. Can you please check and if everything ok close this issue

ghost commented 6 years ago

Yes, that seems to have solved the issue. Thanks Carlos.