agrc / DEQ-TankRisk

An ArcGIS Pro toolbox for DEQ to assess the risk of underground storage tanks
MIT License
1 stars 0 forks source link

FacilityID #31

Closed rsaathoff closed 2 years ago

rsaathoff commented 2 years ago

Something is different with the facilityid from the old tool to the new tool. One issue is that it needs to be named FACILITYID in all caps; however I tried just renaming the column heading and it still didn't like the file on that field.

image

steveoh commented 2 years ago

What are your inputs to the tool with their data sources?

rsaathoff commented 2 years ago

opensgid.environment.petroleum_storage_tank_facilities

steveoh commented 2 years ago

postgres has all lowercase field names. Why do you think it needs to be uppercase?

rsaathoff commented 2 years ago

The old tool or perhaps the source had uppercase and the tool I use to import is expecting uppercase for some reason. The programmer that designed the import tool has retired and nobody can work on it right now.

steveoh commented 2 years ago

Yeah, the SGID you used to connect to was upper cased. That isn't the case any longer. I suppose you could bring it local and rename it and use that as the source.

rsaathoff commented 2 years ago

if I change the view that supplies the opensgid so it is uppercase will that change it to uppercase in the opensgid layer or no?

steveoh commented 2 years ago

You have no edit rights in the open sgid. I don't think any view changes will make a difference. That may only be presentational or an alias for your project. If that works for you then great. But this isn't something we can fix. If you agree we can close this issue.

rsaathoff commented 2 years ago

I don't agree. How do we change the open sgid?

steveoh commented 2 years ago

You don't. It's a database convention. Every postgresql database is lowercase fields.

rsaathoff commented 2 years ago

hmm

steveoh commented 2 years ago

The programmer that designed the import tool has retired and nobody can work on it right now.

Gaining access and fixing the import tool seems like the correct solution

rsaathoff commented 2 years ago

k, I will see what I can do, thanks