search

Esri / distance-direction-addin-dotnet

Add-in provides the ability to easily and quickly create geodesy lines, circles, ellipses and range rings.
Apache License 2.0
17 stars 22 forks source link

Incorrect axes labels for ellipses in ArcGIS Pro #661

Closed ljuru closed 5 years ago

ljuru commented 5 years ago

Current Behavior

This bug was discovered while doing Certification Testing for the Distance and Direction add in for ArcGIS Pro 2.4.

Repro steps:

  1. Install ArcGIS Pro 2.4 + Distance and Direction Addin 4.1.2.68
  2. Using the Distance and Direction Tool to create an ellipse (either by using the map point tool or manually entering the values)
  3. Select Semi for the Ellipse Type drop-down box

image

a) Optional: Enter 3000 for the Major Axis box and 1500 for the Minor Axis box and 30 for Angle (though any angle will do because the angle label works correctly).

  1. Ellipsoid is created. The core measure tool is set to Geodesic and used to confirm the issue.

image

b) Selecting Full: image

image

Note: The labels being displayed for this feature show semi major and minor axis values instead of full major and minor axis values

Expected Behavior

The labels displayed when an ellipsoid is created should reflect major and minor axis values. The core measure tool set to Geodesic is used to confirm results.

image

Additional Notes:

• Observed in ArcGIS Pro 2.3 with released version of MT/DD • Did not observe on ArcMap 10.7.1

csmoore commented 5 years ago

Addressed in PR #664 - should behave the same in Pro and ArcMap now (i.e. the label shown is the "full" axes values)

Pro and ArcMap for same values used for both "full"(left) and "semi"(right) inputs: Pro: image

ArcMap: image

lfunkhouser commented 5 years ago

@ljuru @csmoore Was this just on Pro 2.4 and introduced new at MT 3.5? or was this an issue in previous versions of Pro/MT?

ljuru commented 5 years ago

@lfunkhouser I noticed the issue was in Pro 2.3 as well.

csmoore commented 5 years ago

I should have mentioned in my comments - it looked like this issue may have been present since we switched the graphics to features (June, 2018)

dfoll commented 5 years ago

Build 82 from this branch looks good

Tested on Pro: 2.4 ArcMap: 10.4.1

ArcGIS Pro

ArcMap

dfoll commented 5 years ago

Doc work:

Github doc work in this PR

dfoll commented 5 years ago
dfoll commented 5 years ago

removed G-qc verify because i forgot to do it when i made this comment

dfoll commented 5 years ago

Doc PR is still open

lfunkhouser commented 5 years ago

PR has been approved and merged.

lfunkhouser commented 5 years ago

Verified doc updates on solutionsdev for MT and DD and PDFs for MT (dev build 3.5.0.308) and DD (dev build (88).