nav[aria-label="Primary Navigation"] { padding: 0; & ul { list-style: none; width: 100%; display: flex; flex-direction: row; justify-content: start; align-items: start; gap: 30px; padding: 0; & li { margin: 0; } & ul li { list-style: none; } } }

Can someone please explain this ConstructedOnWrongThreadException?

DanNarsavage_IDWR

I had this method (simplified but I think it provides all the necessary context) . . .

public override Envelope GetExtent()
{
    var geometry = GetGeometry();
    return GeometryEngine.Instance.Expand(geometry.Extent, 1.5, 1.5, true);
}

But when I used that inside Pro, I was treated to an exception with this stack trace . . .

ArcGIS.Core.ConstructedOnWrongThreadException: This object must be created within the lambda passed to QueuedTask.Run, or on a compatible STA thread.
   at ArcGIS.Core.CoreObjectsBase.ValidateThread()
   at ArcGIS.Core.Internal.Geometry.EnvelopeBuilder..ctor(Envelope envelope, Boolean getHandle)
   at ArcGIS.Core.Internal.Geometry.EnvelopeBuilder..ctor(Envelope envelope)
   at ArcGIS.Core.Geometry.GeometryEngine.Expand(Envelope envelope, Double dx, Double dy, Boolean asRatio)
   at My.Namespace.MyClass.GetExtent() in D:\Whatever\MyClass.cs:line 106
   at My.Namespace.MyOtherClass.AnotherMethod() in D:\Whatever\MyOtherClass.cs:line 167

I found that this avoids that exception . . .

public override Envelope GetExtent()
{
    var geometry = GetGeometry();
    var extent = geometry.Extent;
    var geometryEngine = GeometryEngine.Instance;
    var expandTask = QueuedTask.Run(() => geometryEngine.Expand(extent, 1.5, 1.5, true);)
    return expandTask.Result;
}

But I'm not constructing anything within that QueuedTask. I think this is just a leaky abstraction in the API, but I'm still somewhat new to async and fear I'm missing something, so I'd appreciate anyone pointing out what I'm constructing or why I've gotta run that method (whose doc doesn't call out the need to call it on the MCT) in a QueuedTask.

Find more posts tagged with

Accepted answers

AnnetteLocke

It is a bug in GeometryEngine, and the fix will be in the next release. Fortunately, you have the workaround in the meantime. Thank you for reporting this issue.

Annette

GKmieliauskas

We have proverb in my native language: "Butter buttered". So, the same thing is with Envelope and Expand method. Envelope has own Expand method:

            var sr = SpatialReferenceBuilder.CreateSpatialReference(8826);
            MapPoint minPt = MapPointBuilderEx.CreateMapPoint(2456480, 1307526, sr);
            MapPoint maxPt = MapPointBuilderEx.CreateMapPoint(2456895, 1307790, sr);
            EnvelopeBuilderEx builderEx = new EnvelopeBuilderEx(minPt, maxPt, sr);
            Envelope envelope = builderEx.ToGeometry();
            var expandedExtent = envelope.Expand(1.5, 1.5, true);

It works as expected without QueuedTask.Run. It could be a bug in GeometryEngine method.

All comments

AnnetteLocke

It is a bug in GeometryEngine, and the fix will be in the next release. Fortunately, you have the workaround in the meantime. Thank you for reporting this issue.

Annette

DanNarsavage_IDWR

Thank you, that did the trick. Not sure how I didn't notice that method in the first place.

GKmieliauskas

@DanNarsavage_IDWR Yes, you are right. Expand methos doesn't need to be run within QueuedTask.Run.

I think issue is with your returned geometry (from GetGeometry method) extent. Try to check extent for null and for IsEmpty.

DanNarsavage_IDWR

Thank you, GKmieliauskas. But as usual, I wrote too much and obfuscated my own question. My brief "NB FYI" comment indicates that we already plan to refactor our GetExtent method exactly as your second example shows. I'll restate my question here . . .

Why does the GeometryEngine class's Expand(...) method need to be called in QueuedTask.Run? Neither its doc nor the SDK via VisualStudio's intellisense says anything about that need. And why don't I get a CalledOnWrongThreadException for that? I get a ConstructedOnWrongThreadException (we've caused & addressed the former exception many times, but this is my first time seeing the latter) despite my code not constructing anything.

GKmieliauskas

Hi,

Take a look at these:

https://github.com/esri/arcgis-pro-sdk/wiki/ProConcepts-Asynchronous-Programming-in-ArcGIS-Pro

ArcGIS Pro SDK for .NET: Synchronous and Asynchronous Custom Method Design - YouTube

Shortly. If you want something return from asynchronous method you need to use Task<return_data_type>.

That method could contain QueuedTask.Run or orther .NET methos which returns Task.

You can make synchronous method and call QueuedTask.Run outside method. Below two samples with your mehod implemetation:

public Envelope GetExtent()
{
    var geometry = GetGeometry();
    var extent = geometry.Extent;
    var geometryEngine = GeometryEngine.Instance;
    return geometryEngine.Expand(extent, 1.5, 1.5, true);
}

var newExtent = await QueuedTask.Run(() =>
{
   return GetExtent();
});

public Task<Envelope> GetExtentAsync()
{
    var geometry = GetGeometry();
    var extent = geometry.Extent;
    var geometryEngine = GeometryEngine.Instance;
    return QueuedTask.Run(() =>
    {
       return geometryEngine.Expand(extent, 1.5, 1.5, true);
    });
}

var newExtent = await GetExtentAsync();

DanNarsavage_IDWR

NB FYI we will eventually modify that method to `public override async Task<Envelope> GetExtent()`, but we need other parts of our solution to stabilize first--this is a migration from ArcObjects.

Join the Conversation

Signed in members can post, follow updates, and more. New here? Register a free account.

Quick Links

Community Help
Find useful guides, FAQs, and documents to help you navigate and make the most of Esri Community.