NREL / floorspace.js

Other
66 stars 35 forks source link

Broken Geometry Bug #302

Closed fsudaman closed 6 years ago

fsudaman commented 6 years ago

It appears that making any geometry change and merging will result in the object state being corrupted. This issue can be reproduced by the following steps (OSM file attached) FloorSpace Broken Geometry Bug - Reproduction.zip

All three, once merged, resulted in the bugs such as below appearing (checked by creating a few new floors and trying to draw new spaces)

floorspace broken geometry bug screenshot

floorspace broken geometry bug screenshot2

macumber commented 6 years ago

Moved to https://github.com/NREL/OpenStudio/issues/3008

macumber commented 6 years ago

@fsudaman thank you for the very detailed report. Please note, that issues related to the FloorspaceJS widget embedded inside the OpenStudio Application (including translation to OSM and merging with OSM) should be reported in the OpenStudio repository. This repository is for issues with the JavaScript FloorspaceJS widget only

rwadhwa910 commented 3 years ago

@macumber is there a solution for this I see it was opened in 2018.

macumber commented 3 years ago

I'm not sure which bug you are referring to @rwadhwa910? It looks like a issue with file change counters might have been fixed recently with:

https://github.com/NREL/floorspace.js/pull/379

That change is not yet incorporated with the OpenStudio Application, I may see if it is something we can include in our next release:

https://github.com/openstudiocoalition/OpenStudioApplication/issues/327

This bug was moved to OpenStudio and is still open: https://github.com/NREL/OpenStudio/issues/3008

Right now it is not tagged for the next release. If you are seeing specific issues with the import/merge of Floorspace JSON files in the OpenStudio Application please add them to the bug in the OpenStudio Application repo.