search

mezerotm / cmv

CMV is a citySDK based map visualizatoin tool
5 stars 5 forks source link

Location input #37

Closed NJTuley closed 7 years ago

NJTuley commented 7 years ago

Wanted to get everyone to see the code that implemented the dynamic location input being integrated with the citysdk request, and answer any questions about it, and/or talk about any issues anyone sees in the code

mezerotm commented 7 years ago

@NJTuley I remember you telling me you were still having issues with pulling information. Can you please better document these issues and how I can replicate them. I don't expect to fix them overnight but I would like to put some fresh eyes on the issue (me)

hardnett commented 7 years ago

@NJTuley and @mezerotm If there is an issue with pulling information, then go ahead and create a bug issue for the Alpha Release and tag it also as a research issue. If this is the issue that is required collaboration with Logan and so its semi-blocked then make a note of that and we may decided to let float to the beta release since it may require cooperation with the CitySDK folks and in the meantime we can develop a way to guard against the bad data pulls by notify the user of the problem. This does not need to hold up this PR.

hardnett commented 7 years ago

Hi @mezerotm. Before I make this PR merge, I want to make sure that my assertion is correct that this PR can be merged and that bad data issues can be resolved via an additional issue. Thanks, and so if you are okay, just approve.