You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
If I understand correctly, current version does not take into account orientation of walls: it should affect power and energy calculation by a multiplying factor which, according to this document, for Italy varies from 0 to 20% (I don't know if it's the same everywhere):
Possible structure to use (add in config structure):
Thanks @jumpjack that's great! I've added a link to your fork on the front page readme, see https://github.com/TrystanLea/heatlossjs/blob/master/readme.md. Perhaps it would be worth starting an issue on this repository called heatlossjsITA with updates on the changes that you are making in your fork so that others can see your progress and if it makes sense build on your repository if that's the direction this should take..
If I understand correctly, current version does not take into account orientation of walls: it should affect power and energy calculation by a multiplying factor which, according to this document, for Italy varies from 0 to 20% (I don't know if it's the same everywhere):
Possible structure to use (add in config structure):
Current calculation:
Updated:
(note: e.orientation contents should be changed from "South" to "S", from "North" to "N" and so on:
Additionally, this line in model.js:
should be changed to:
I cannot add a pull request because I performed too many other changes to the original source to fit in just on PR.
The text was updated successfully, but these errors were encountered: