search

NTNU-IndEcol / BuildME

6 stars 1 forks source link

Move settings.odym_materials to data/materials.xlsx #66

Closed nheeren closed 8 months ago

nheeren commented 2 years ago

Related to #52. @shnkn and I discussed today that the most reasonable place for settings.odym_materials would in data/materials.xlsx in a new column mat_category.

Does this conflict with our ambitions to move to YAML?

Any takers? ;)

nheeren commented 2 years ago

So this idea is not as simple / smart as I first thought. settings.odym_materials contains also materials that are not contained in data/materials.xlsx.

shnkn commented 2 years ago

yes, but did not we say that for this reason, we would have an additional sheet for all settings.odym_materials in the same data/materials.xlsx file? So I do not think there would be a problem.

But I do not know the case with YAML. We might need two different YAML files maybe instead of one single excel file.

kamilakrych commented 8 months ago

In BuildME2.0, the materials for aggregation are included in the xlsx configuration file