search

NTNU-IndEcol / BuildME

6 stars 1 forks source link

Kamila changes merge - various bug fixes and improvements(2) #76

Closed kamilakrych closed 1 year ago

nheeren commented 1 year ago

Wow this is fantastic work! Thank you so much for solving so many problems at once ;)

I may partially revert some of afa09d1 at some point, because I made some enhancements to material.xlsx already.

I am a bit unclear what 92a3b4f does. Can you elaborate a bit or even document somewhere?

kamilakrych commented 1 year ago

Hi Niko! Thanks for merging :) https://github.com/nheeren/BuildME/commit/92a3b4f4ddf5a6e08853c1d1701bd55cb214dd6b does what we discussed in our last meeting - it includes internal mass into the material count. An internal mass object is basically equivalent to internal walls, but the object handling is a bit different, thus the changes in the commit.

kamilakrych commented 1 year ago

Where can I document it?