search

cstb / citygml-energy

CityGML Energy ADE
40 stars 9 forks source link

Delete relations isConsumedBy and isProcudecBy #112

Closed mlauster closed 8 years ago

mlauster commented 8 years ago

Purpose is to link from EnergySource to EnergyConversionSystem, but is there a real use case where somebody uses this way instead of linking from an EnergyConversionSystem to an EnergySource using consumes and produces? If not, I vote for deleting them.

RomainNouvel commented 8 years ago

The use of the different relations must be demonstrated with new examples...

JoachimBenner commented 8 years ago

@mlauster, @RomainNouvel can we come to a decision with this issue?

mlauster commented 8 years ago

We have still no use case for it, so I still vote for remaining it.

RomainNouvel commented 8 years ago

:+1:

JoachimBenner commented 8 years ago

O. k., let's decide to remove the relations

RomainNouvel commented 8 years ago

@JoachimBenner : Morith and I voted to keep the relations, not removing them :-)

JoachimBenner commented 8 years ago

Sorry, I am a little bit confused because Moritz at the beginning voted for "deleting", and then for "remaining". @mlauster can you clarify this?

mlauster commented 8 years ago

Oh, sorry, I meant removing. But if @RomainNouvel would like to keep them, I'm also fine with it.

RomainNouvel commented 8 years ago

Since we can't come to an agreement without having tested it, Let's leave it as it is, and test it in the next months...