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; } } }

arcpy.da.UpdateCursor fails to update geometry

BledarBirbo1

Hi.

I am having the following issue when i try to update an SDE Geodatabase featureclass geometry from arcpy

Using:

ArcGIS 10.3.1

SQL Server Geodatabase

Line featureclass with enabled: Archiving , Editor Tracking and Registered as Versioned.

The line featureclass also participates in a replica as the source layer.

The line fetureclass can be edited fine from arcmap.

The issue:

Because we do receive new updated line geometries (features) from another source i have bulid a python script that joins the 2 datasets based on an attribute key and and tries to update the geometry on the Line Featureclass.

I am using the following code tu update the geometry.

fieldNames <SPAN class="operator token">=</SPAN> <SPAN class="punctuation token">[</SPAN><SPAN class="string token">'ID'</SPAN><SPAN class="punctuation token">,</SPAN> <SPAN class="string token">'SHAPE@'</SPAN><SPAN class="punctuation token">,</SPAN><SPAN class="string token">'SHAPE@WKT'</SPAN><SPAN class="punctuation token">]</SPAN>
edit<SPAN class="punctuation token">.</SPAN>startEditing<SPAN class="punctuation token">(</SPAN><SPAN class="punctuation token">)</SPAN>
edit<SPAN class="punctuation token">.</SPAN>startOperation<SPAN class="punctuation token">(</SPAN><SPAN class="punctuation token">)</SPAN>
<SPAN class="keyword token">with</SPAN> arcpy<SPAN class="punctuation token">.</SPAN>da<SPAN class="punctuation token">.</SPAN>UpdateCursor<SPAN class="punctuation token">(</SPAN>line_layer<SPAN class="punctuation token">,</SPAN> fieldNames<SPAN class="punctuation token">)</SPAN> <SPAN class="keyword token">as</SPAN> updateCursor<SPAN class="punctuation token">:</SPAN>
 <SPAN class="keyword token">for</SPAN> tmpRow <SPAN class="keyword token">in</SPAN> updateCursor<SPAN class="punctuation token">:</SPAN>
&nbsp;&nbsp;&nbsp; tmpRow<SPAN class="punctuation token">[</SPAN><SPAN class="number token">1</SPAN><SPAN class="punctuation token">]</SPAN> <SPAN class="operator token">=</SPAN> <SPAN class="comment token">#newgeometry that was read previusly</SPAN>
&nbsp;&nbsp;&nbsp; updateCursor<SPAN class="punctuation token">.</SPAN>updateRow<SPAN class="punctuation token">(</SPAN>tmpRow<SPAN class="punctuation token">)</SPAN>
edit<SPAN class="punctuation token">.</SPAN>stopOperation<SPAN class="punctuation token">(</SPAN><SPAN class="punctuation token">)</SPAN>
edit<SPAN class="punctuation token">.</SPAN>stopEditing<SPAN class="punctuation token">(</SPAN><SPAN class="token boolean">True</SPAN><SPAN class="punctuation token">)</SPAN>‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍<SPAN class="line-numbers-rows"><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN></SPAN>

The code runs without any error but the geometry is not updated. I tested the code for other attributes and they were updated correctly.

Any ideas ?

Thanks.

Find more posts tagged with

Accepted answers

VinceAngelo

kimo wrote:
Do you need a geometry AND a wkt version? Surely one will do. You would have to be careful to get the one in the list.

This is the key point. By specifying an update of the same column in two different forms, but only updating the first referenced input, the code is likely updating the column with the new content, then updating it again with the previous contents.

Multiple references to the same geometry column in an UPDATE or INSERT cursor really ought to raise an error.

- V

All comments

KimOllivier

Do you need a geometry AND a wkt version? Surely one will do. You would have to be careful to get the one in the list.

Would it not be easier to just write row[1] = objectdict[row[0]] where the objectdict is a dictionary of shapes indexed by ID

I am trying to help you debug your code, so I would put in a lot of print statements that confirm you do have a geometry and it is

in the right row element by printing out a WKT representation as you go and then break after the first one.

If you only put up pseudocode you are hiding your problem with your assumptions on where the problem lies. It would be much better

if you posted your real code.

KimOllivier

Rather than a list of geometries a dictionary of geometries is more direct to insert.

KimOllivier

Is this too simple? If the geometry is WKT format then the insert should use tmpRow[2] not tmpRow[1] because the field spec you must have is SHAPE@WKT

BledarBirbo1

There are 300 new features that are beeing read from a SearchCursor.

Their geometry is then stored on a python list as Geometry object and as WKT plus the OBJECTID

During the UpdateCursor each feature is matched with the ones that were read before from the python list based on OBJECTID. If matched then the

tmpRow<SPAN class="" style="color: #999999; border: 0px; font-weight: inherit;">[</SPAN><SPAN class="" style="color: #990000; border: 0px; font-weight: inherit;">1</SPAN><SPAN class="" style="color: #999999; border: 0px; font-weight: inherit;">] = arcpy.FromKWT(objectlist[matchedindex] wkt)</SPAN><SPAN class="" style="color: #999999; border: 0px; font-weight: inherit;">

</SPAN><CODE> tmpRow<SPAN class="" style="color: #999999; border: 0px; font-weight: inherit;">[</SPAN><SPAN class="" style="color: #990000; border: 0px; font-weight: inherit;">1</SPAN><SPAN class="" style="color: #999999; border: 0px; font-weight: inherit;">] = objectlist[matchedindex] geometry.
#even constructing a new Geometry from the WKT like this does not work 
</SPAN>

KimOllivier

Ok, well how many features are in the layer definition?

Can you print out the ID and featurecount?

I assume you are using some sort of index based on the ID to get the respective geometry to add?

Why are you getting the geometry out twice in the updateCursor?

BledarBirbo1

The geometry that is beeing assigned is not the same and is fine.

KimOllivier

You have skipped the key item - what geometry are you adding? How did you get it? Is it really a geometry?

I don't think you can join two featureclasses and retrieve both geometries at once. You are probably just replacing the geometry with itself.

I would create a python dictionary of source geometries keyed by the id using a list comprehension and a ReadCursor.

Then close the cursor and open an UpdateCursor and iterate over the list. You could even speed it up

by using an SQL query to only select records to be updated with an IN sql clause of the keys.

BledarBirbo1

Hi.

There is an update command i just forgot to copy it in this post.

So that do not help.

bbrr.jpeg

432787844_122112381788241714_1171584705015624595_n.jpg

IanMurray

after you set the values with the cursor for the row, you need to have cursor.updateRow(tmpRow), otherwise it doesn't save the values.

Add the following to a new line after the current line 6 with the same indent as line 6:

cursor.updateRow(tmpRow)

See https://pro.arcgis.com/en/pro-app/arcpy/data-access/updatecursor-class.htm example 1 for another example if needed.

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.