Esri / coordinate-conversion-addin-dotnet

Addin for ArcMap and Pro for convenient coordinate conversion in Desktop.
Apache License 2.0
45 stars 20 forks source link

Exporting to feature class when using a custom Display Coordinate causes Pro to become unresponsive #555

Closed dfoll closed 5 years ago

dfoll commented 5 years ago

Exporting to feature class when using a custom Display Coordinate (Input and List) causes the app to become unresponsive.

Repro steps

  1. In Coordinate Conversion pane, set the Display Coordinate to DD
  2. Use the Map Point Tool to collect a few points from the map
  3. Click Export
  4. Use the Feature class radio button.
  5. Click OK
  6. Name fc.
  7. FC is added to TOC
  8. Change the Display Coordinate to a custom version of one the coordinate formats, for example, format DD to be Y0.0#####N/\X0.0#####E
  9. Click Export either ArcGIS Pro just hangs or CC gets stuck in a state where the Map Point Tool is active but it does not do anything

Notes

MT 189

BobBooth commented 5 years ago

I can repro using DD and the /\ chars together as a separator.

lfunkhouser commented 5 years ago

More testing needed. Test from a build from Jan 7 to see if it was introduced with fix from #551

BobBooth commented 5 years ago

It seems to write out the feature classes correctly, even when it freezes. image MGRS with spaces, MGRS with /\ separator, etc...

kgonzago commented 5 years ago

Here are the builds I tested with:

  1. Build 42 (CTLocalGovTeam-dev branch) from current pre-release
  2. Build 20 from December 2018 release
  3. Build 28 from August 2018 release Issue is still there in all three builds. Also, I confirmed @BobBooth's comment: feature class was still created when Pro was non-responsive.
kgonzago commented 5 years ago

We can dig more into this but given the tight time frame, I suggest push this issue to the next release.

dfoll commented 5 years ago

Testing with ArcGIS Pro 2.2 CC build 89 The repro steps listed above no longer cause ArcGIS Pro to hang.

dfoll commented 5 years ago

Doc ready for review at: