cstb / citygml-energy

CityGML Energy ADE
40 stars 9 forks source link

Change cardinality of relation ThermalOpening-->_Opening #133

Closed gioagu closed 6 years ago

gioagu commented 7 years ago

The current relation "relatesTo" has cardinality 0..1.

We should extend it to 0..* in order to be able to also define _ThermalOpenings (e.g. just in terms of global area in m^2) that actually correspond to (the sum of) multiple _Openings.

Agree?

JoachimBenner commented 7 years ago

I principally agree, but we must be careful. The restriction is that all these _Opening objects have the same orientation, which normally means that they belong to the same _BoundarySurface. This should be mentioned in the documentation.

JoachimBenner commented 7 years ago

Can we agree on this proposal

RomainNouvel commented 7 years ago

ok

gioagu commented 7 years ago

ok