ngageoint / hootenanny-ui

Hootenanny UI is a submodule of the Hootennany vector conflation project.
ISC License
29 stars 7 forks source link

Export hangs after resolving all reviews with cropped Haiti Port Au Prince data #290

Closed mikejeffe closed 8 years ago

mikejeffe commented 8 years ago

If you click the Export Data after resolving all reviews from the conflation of PortAuPrinceOsmPoiRoadBuilding.osm (ref) and MgcpCroppedPoiRoadBuilding.osm (secondary), the export job hangs for what seems to be an indefinite amount of time. I'm only seeing this on the develop branch not in the release. Appears to occur with all export file types.

Dev console reports: Uncaught TypeError: Cannot read property 'value' of null

ka.property @ d3.min.js:3 d3.selection.value @ d3.value.js:24 model_export.exportData @ Export.js:21 (anonymous function) @ tools.js:596 t @ d3.min.js:1 (anonymous function) @ Export.js:219 (anonymous function) @ d3.min.js:1

mikejeffe commented 8 years ago

@sisskind we would want to make sure that any recent commits from develop which caused this problem don't make it back into the release inadvertently but it's my understanding this would only get merged into develop if its a real bug.

sisskind commented 8 years ago

@jchoi1267 is this something you'll be able to look at in the next 24 hrs?

mikejeffe commented 8 years ago

occuring with the Latvia500Pois datasets as well so not specific to the original datasets listed above.

sisskind commented 8 years ago

@mjeffe98 what file are you exporting to (OSM, FGDB, etc.)?

jchoi1267 commented 8 years ago

@mjeffe98 Found the fix and about to commit.

sisskind commented 8 years ago

@jchoi1267 beat me to it!

jchoi1267 commented 8 years ago

@sisskind Looks like this will be my last ticket. :-) Now I am watching cat channel.. LOL

mikejeffe commented 8 years ago

still seeing an issue after rebuilding selma develop Hootenanny Core - Version: 0.2.22_337_gf6a5ad4_dirty - Built By: ubuntu

@jchoi1267 @sisskind I wasn't planning on testing tomorrow and not sure everyone's schedule but maybe @jchoi1267 can just verify that the commit is in this release or pending a submodule sync.

mikejeffe commented 8 years ago

Re-tested and appears to be working as expected.