NREL / floorspace.js

Other
66 stars 35 forks source link

Kyle merge two floorspace files #405

Closed kylem038 closed 2 years ago

kylem038 commented 2 years ago

This addresses #368 We've added a new icon to the toolbar which will accept a new floorspace json file and merge an incoming floorplan with the current floorplan in state. A few items that should be addressed before merging...

kylem038 commented 2 years ago

@bgschiller Thank you for taking the time to review. Did you have any thoughts on the first checkbox in the description? If that sounds good I can update each property to append "Original" and "Imported" to the names to denote what came from where.

bgschiller commented 2 years ago

@kylem038, "Original" and "Imported" work for me. I feel like it might be good to get @DavidGoldwasser 's input on it, if he's available. It's also easy to change later, so you could merge it and let him take a look once it's live.