-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make Topology Tool #63
Comments
Ok, I will take a look at these today. |
First, the Make Topology tool - I cannot recreate this error. The path to your gdb looks good and I confirmed the tool ran correctly even when the names of GeologicMap, ContactsAndFaults, and MapUnits had been changed to include a prefix. Can you include a screenshot from ArcCatalog of the contents of your gdb? And, try running the tools from ArcCatalog completely outside of an ArcMap session. With some tools - can't confirm this is one of them without digging way into the code - having layers in the map from different data sources that have the same names can cause the tools to point to the wrong feature class. |
Evan,
When I look at the Lainey’s error it is interestingly similar to the error with the search cursor that we “discussed” in October.
In rereading my notes from that:
# open search cursor on DMU sorted by HKey
### the sql clause is causing an error -- will try adding MapUnit to clause as per
### https://desktop.arcgis.com/en/arcmap/latest/analyze/arcpy-data-access/searchcursor-class.htm
# open search cursor on DMU sorted by HKey
sql = (None, 'ORDER BY HierarchyKey')
with arcpy.da.SearchCursor('DescriptionOfMapUnits',('MapUnit'),None,None,False,sql) as cursor:
From reading the ESRI Search Cursor page, I think that the field names must include HierarchyKey in order for the sql statement to work.
Linda
Linda Tedrow
Idaho Geological Survey
University of Idaho
3rd Floor Morrill Hall
875 Perimeter Drive, MS 3014
Moscow, Id 83844-3014
(208) 885-1579
From: Evan Thoms ***@***.***>
Sent: Thursday, April 8, 2021 11:23 AM
To: usgs/gems-tools-arcmap ***@***.***>
Cc: Subscribed ***@***.***>
Subject: Re: [usgs/gems-tools-arcmap] Make Topology Tool (#63)
Ok, I will take a look at these today.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub<https://urldefense.com/v3/__https:/github.com/usgs/gems-tools-arcmap/issues/63*issuecomment-816039845__;Iw!!JYXjzlvb!2UYC92B-CrUp6yZAJGJSjvZ-07feyYiByoCaUpveatznXAPS333pXQB1IFbbdl2q$>, or unsubscribe<https://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/AKYEVBZBMXVAUHEIUFMWDVLTHXYAZANCNFSM42Q7SM6A__;!!JYXjzlvb!2UYC92B-CrUp6yZAJGJSjvZ-07feyYiByoCaUpveatznXAPS333pXQB1ID6gqHqO$>.
|
@ethoms-usgs I thought I could fix the problem by flipping the names around, since the tools allows for prefixes, but maybe not suffixes? So good news, the make topology tool works now, but I'm still getting the same error message for the validation tool. I built the geodatabase from scratch rather than using the tool, so I could've messed up a few things along the way. Upon checking, I found that I allowed Null values in the DMU for HierarchyKey, could that be the problem? Also, somewhat related, in building the gdb from scratch I realized the attribute field in OrientationPoints is now "StationsID" in the publication, formerly was "StationID" in the manuscript. The validate tool will flag this as an error if someone was building their gdb based on the pub. Not a big deal, just pointing it out. And lastly, for what's it worth, I've experienced the issue you describe about having ArcMap open and the validate tool pulling from another open geodatabase than the one I directed it to. |
The Validate Database tool does expect to be able to find a 'DescriptionOfMapUnits' table, with no prefixes or suffixes, so try removing the prefix. The Make Topology tool works with feature classes with prefixes (only -- no suffixes) because the intent is for it to be used to build topology not only in GeologicMap feature dataset, but CrossSection datasets as well, where ContactsAndFaults and MapUnitPolys do get named with prefixes. In general, the tools do not expect to have to deal with prefixes like what you have used because of the idea that a GeMS database describes the geology of only one map, that is, a prefix that identifies a feature class as belonging to a particular gdb is redundant. We could consider re-writing the tools to always look for names in the way that Make Topology tool does, but the workaround for now is to not rename those required table names |
Changing the DMU back to its original name fixed the issue and the tool ran. I understand the need to keep the names as is, I did it for clarity since I'm compiling 4 GeMS geodatabases in one map. Changing the names is easy enough, and I should've just changed the alias (duh). Thanks for the help, Evan! |
Yeah, I wondered if you were doing something like that! |
@LindaTedrow Yes! It is similar isn't it? I am still curious about the error you reported. I have continued the discussion over at the original issue |
I'm using the most recent tools (feb. 2021), but I've tried running on the older versions and get the same result. Running in ArcCatalog produces the same result.
The text was updated successfully, but these errors were encountered: