topic Re: populate x, y on mouse click in Python Questions

populate x, y on mouse click

CCWeedcontrol — Sat, 11 Dec 2021 07:24:42 GMT

I am trying to populate the new created point with X, Y coordinates but i have not been successful. The fields it needs to populate on the point feature class is "Point_X" and "Pont_Y". it seems as only "Point_Y" is the only one that gets populated. I also need to take two fields from the parcels ("SiteStreet" & StreetType") and combine them to populate a field from on the point feature layer (StreetName) i am unsure on how to do that.

#import modules  
import arcpy  
""" 
tool box parameters 
point param = feature set 
targetpoint = featurelayer 
parcel = featurelayer 
"""
arcpy.env.qualifiedFieldNames = False

point = arcpy.GetParameterAsText(0)  #point boolen
targetpoint = "CCAP1" #target point feature class 
parcel = "parcels"  #target feature class
parcel_lyr = 'parcel_lyr'  
   
for prow in arcpy.da.SearchCursor(point,'SHAPE@XY'):  
    x,y = prow[0]
del prow  
   
point1 = arcpy.Point(x, y)  
ptGeometry = arcpy.PointGeometry(point1)

CC_list = []
with arcpy.da.SearchCursor(targetpoint, ["AddressID"]) as cursor:
    for row in cursor:
        try:
            if "CC" in row[0]:
                CC_list.append(int(row[0].strip("CC")))   
        except TypeError:
            pass        
del cursor

CC_list.sort()
AddressID = CC_list[-1] + 1
AddressID = 'CC' + str(AddressID)
   
arcpy.MakeFeatureLayer_management(parcel,parcel_lyr)  
arcpy.SelectLayerByLocation_management(parcel_lyr,"INTERSECT",ptGeometry)  
  
fldList = ['ACCOUNT','OwnerName','SiteAddres','SiteNum','siteNumSfx','Predir','SiteStreet', 'StreetType', 'Postdir', 'SiteCity', 'SiteZip']  
fldDict ={}  
  
if int(arcpy.GetCount_management(parcel_lyr).getOutput(0))==1:  
    for parrow in arcpy.da.SearchCursor(parcel_lyr,fldList):  
        for x in range(len(fldList)):  
            fldDict[fldList]=parrow  
    del parrow  
    fldAttList = []  
    for x in range(len(fldList)):  
        fldAttList.append(fldDict[fldList])  
    manualFields =  ["SiteState","FacltyType", "StructType","GIS_STEW", "UpdateBy","Verified", "Status", "StructCat", "APA_CODE","AddressID", "POINT_X", "POINT_Y"]  
    for fld in manualFields:  
        fldList.append(fld)  
    fixedAtts = ["ID","Single Family Home","Primary, Private","CanyonCo", "TA", "Yes, GRM, TA","Active","Residential","1110", AddressID, x, y] 
    for att in fixedAtts:  
        fldAttList.append(att)  
    fldAttList.append(point1)  
    fldtuple = (fldAttList)  
    fldList.append( 'SHAPE@XY')  
    tprow = arcpy.da.InsertCursor(targetpoint, fldList) 
    tprow.insertRow(fldtuple)  
     
del tprow

Re: populate x, y on mouse click

WesMiller — Mon, 20 Jul 2015 20:31:53 GMT

The x isn't populating correctly because it was reused in line 45 and 49 change those to different letters and that should fix that part

edit are the "SiteStreet" & StreetType" fields both being used in the target point feature class or are they going to another field?

Re: populate x, y on mouse click

DarrenWiens2 — Sat, 11 Dec 2021 07:24:44 GMT

You set 'x' on line 17, but then give it new values on lines 45 and 49. Use a different variable name in your loops if you want to maintain the value of x.

See below for a very simple example of what's going on with your variable 'x':

>>> x = 15
... for x in range(100):
...    pass
... print x
...
99

Re: populate x, y on mouse click

CCWeedcontrol — Mon, 20 Jul 2015 21:13:40 GMT

I change the following x to w and get this error.

Traceback (most recent call last):

File "C:\GIS\Python\UpdateAddressPoints\CreatePointUdateAttributesTest2.py", line 46, in <module>

fldDict[fldList]=parrow

TypeError: tuple indices must be integers, not float

Failed to execute (Pointtest).

the "SiteStreet" & StreetType" fields both being in the point feature class.

Re: populate x, y on mouse click

CCWeedcontrol — Mon, 20 Jul 2015 21:28:20 GMT

Weird, I closed out of arcmap and reopen it and it is now populating the X, Y. . but in feet. I have sent my coordinate system to NAD1983 but it still populates feet not Lat and Log (long: -116.5988, Lat: 43.77097) .

Re: populate x, y on mouse click

WesMiller — Tue, 21 Jul 2015 01:01:38 GMT

Because you also have to change it in the for loops. So it's lines 45,46 and 49,50. You should also be getting the coordinated of the data frame even though it's not set explicitly

Re: populate x, y on mouse click

WesMiller — Sat, 11 Dec 2021 07:24:47 GMT

See the notes in the code below

#import modules    
import arcpy    
"""  
tool box parameters  
point param = feature set  
targetpoint = featurelayer  
parcel = featurelayer  
"""  
arcpy.env.qualifiedFieldNames = False  
  
point = arcpy.GetParameterAsText(0)  #point boolen  
targetpoint = "CCAP1" #target point feature class   
parcel = "parcels"  #target feature class  
parcel_lyr = 'parcel_lyr'    
    
    
for prow in arcpy.da.SearchCursor(point,'SHAPE@XY'):    
    x,y = prow[0]    
del prow    
     
point1 = arcpy.Point(x, y)    
ptGeometry = arcpy.PointGeometry(point1)  
  
CC_list = []  
with arcpy.da.SearchCursor(targetpoint, ["AddressID"]) as cursor:  
    for row in cursor:  
        try:  
            if "CC" in row[0]:  
                CC_list.append(int(row[0].strip("CC")))     
        except TypeError:  
            pass          
del cursor  
  
CC_list.sort()  
AddressID = CC_list[-1] + 1  
AddressID = 'CC' + str(AddressID)  
  
     
arcpy.MakeFeatureLayer_management(parcel,parcel_lyr)    
arcpy.SelectLayerByLocation_management(parcel_lyr,"INTERSECT",ptGeometry)    
    
fldList = ['ACCOUNT','OwnerName','SiteAddres','SiteNum','siteNumSfx','Predir','SiteStreet', 'StreetType', 'Postdir', 'SiteCity', 'SiteZip']    
fldDict ={}    
    
if int(arcpy.GetCount_management(parcel_lyr).getOutput(0))==1:    
    for parrow in arcpy.da.SearchCursor(parcel_lyr,fldList):    
        for w in range(len(fldList)):    
            fldDict[fldList]=parrow    
    del parrow    
    fldAttList = []
    #You Can Change this to a name better suited for your concatenated field 
    newfld = str(fldDict["SiteStreet"]) +" "+str(fldDict["StreetType"])
    #This Takes them out of the loop
    del fldDict["SiteStreet"]
    del (fldDict["StreetType"]
    for w in range(len(fldList)):    
        fldAttList.append(fldDict[fldList])    
    manualFields =  ["SiteState","FacltyType", "StructType","GIS_STEW", "UpdateBy","Verified", "Status", "StructCat", "APA_CODE","AddressID", "POINT_X", "POINT_Y",PutYourConcatenatedFieldHere]    
    for fld in manualFields:    
        fldList.append(fld)
    #if you change newfld change it here too
    fixedAtts = ["ID","Single Family Home","Primary, Private","CanyonCo", "TA", "Yes, GRM, TA","Active","Residential","1110",AddressID,x,y,newfld ]   
    for att in fixedAtts:    
        fldAttList.append(att)    
    fldAttList.append(point1)    
    fldtuple = (fldAttList)    
    fldList.append( 'SHAPE@XY')    
    tprow = arcpy.da.InsertCursor(targetpoint,fldList)    
    tprow.insertRow(fldtuple)
    del tprow
else:
    arcpy.AddMessage("Number of parcels selected incorrect for this process")

Re: populate x, y on mouse click

CCWeedcontrol — Sat, 11 Dec 2021 07:24:50 GMT

I made the suggested changes but i am getting an error:

Traceback (most recent call last):

File "C:\GIS\Python\UpdateAddressPoints\CreatePointUdateAttributesTest3.py", line 57, in <module>

fldAttList.append(fldDict[fldList])

KeyError: 'SiteStreet'

Current code

#import modules  
import arcpy  
""" 
tool box parameters 
point param = feature set 
targetpoint = featurelayer 
parcel = featurelayer 
"""
arcpy.env.qualifiedFieldNames = False

point = arcpy.GetParameterAsText(0)  #point boolen
targetpoint = "CCAP1" #target point feature class 
parcel = "parcels"  #target feature class
parcel_lyr = 'parcel_lyr'  
   
for prow in arcpy.da.SearchCursor(point,'SHAPE@XY'):  
    x,y = prow[0]
del prow  
   
point1 = arcpy.Point(x, y)  
ptGeometry = arcpy.PointGeometry(point1)

CC_list = []
with arcpy.da.SearchCursor(targetpoint, ["AddressID"]) as cursor:
    for row in cursor:
        try:
            if "CC" in row[0]:
                CC_list.append(int(row[0].strip("CC")))   
        except TypeError:
            pass        
del cursor

CC_list.sort()
AddressID = CC_list[-1] + 1
AddressID = 'CC' + str(AddressID)
   
arcpy.MakeFeatureLayer_management(parcel,parcel_lyr)  
arcpy.SelectLayerByLocation_management(parcel_lyr,"INTERSECT",ptGeometry)  
  
fldList = ['ACCOUNT','OwnerName','SiteAddres','SiteNum','siteNumSfx','Predir','SiteStreet', 'StreetType', 'Postdir', 'SiteCity', 'SiteZip']  
fldDict ={}  

#setvalue = ('SiteStreet'+ 'StreetType')  
if int(arcpy.GetCount_management(parcel_lyr).getOutput(0))==1:  
    for parrow in arcpy.da.SearchCursor(parcel_lyr,fldList):  
        for w in range(len(fldList)):  
            fldDict[fldList]=parrow  
    del parrow  
    fldAttList = []
    #You Can Change this to a name better suited for your concatenated field   
    newfld = str(fldDict["SiteStreet"]) +" "+str(fldDict["StreetType"])  
    #This Takes them out of the loop  
    del fldDict["SiteStreet"]  
    del (fldDict["StreetType"])
    for w in range(len(fldList)):  
        fldAttList.append(fldDict[fldList])  
    manualFields =  ["SiteState","FacltyType", "StructType","GIS_STEW", "UpdateBy","Verified", "Status", "StructCat", "APA_CODE","AddressID", "POINT_X", "POINT_Y","StreetName"]  
    for fld in manualFields:  
        fldList.append(fld)  
    fixedAtts = ["ID","Single Family Home","Primary, Private","CanyonCo", "TA", "Yes, GRM, TA","Active","Residential","1110", AddressID, x, y,newfld] 
    for att in fixedAtts:  
        fldAttList.append(att)  
    fldAttList.append(point1)  
    fldtuple = (fldAttList)  
    fldList.append( 'SHAPE@XY')  
    tprow = arcpy.da.InsertCursor(targetpoint, fldList) 
    tprow.insertRow(fldtuple)

     
del tprow

Re: populate x, y on mouse click

WesMiller — Sat, 11 Dec 2021 07:24:52 GMT

I forgot to remove them from the fldList try this

#import modules    
import arcpy    
"""  
tool box parameters  
point param = feature set  
targetpoint = featurelayer  
parcel = featurelayer  
"""  
arcpy.env.qualifiedFieldNames = False  
  
point = arcpy.GetParameterAsText(0)  #point boolen  
targetpoint = "CCAP1" #target point feature class   
parcel = "parcels"  #target feature class  
parcel_lyr = 'parcel_lyr'    
     
for prow in arcpy.da.SearchCursor(point,'SHAPE@XY'):    
    x,y = prow[0]  
del prow    
     
point1 = arcpy.Point(x, y)    
ptGeometry = arcpy.PointGeometry(point1)  
  
CC_list = []  
with arcpy.da.SearchCursor(targetpoint, ["AddressID"]) as cursor:  
    for row in cursor:  
        try:  
            if "CC" in row[0]:  
                CC_list.append(int(row[0].strip("CC")))     
        except TypeError:  
            pass          
del cursor  
  
CC_list.sort()  
AddressID = CC_list[-1] + 1  
AddressID = 'CC' + str(AddressID)  
     
arcpy.MakeFeatureLayer_management(parcel,parcel_lyr)    
arcpy.SelectLayerByLocation_management(parcel_lyr,"INTERSECT",ptGeometry)    
    
fldList = ['ACCOUNT','OwnerName','SiteAddres','SiteNum','siteNumSfx','Predir','SiteStreet', 'StreetType', 'Postdir', 'SiteCity', 'SiteZip']    
fldDict ={}    
  
#setvalue = ('SiteStreet'+ 'StreetType')    
if int(arcpy.GetCount_management(parcel_lyr).getOutput(0))==1:    
    for parrow in arcpy.da.SearchCursor(parcel_lyr,fldList):    
        for w in range(len(fldList)):    
            fldDict[fldList]=parrow    
    del parrow    
    fldAttList = []  
    #You Can Change this to a name better suited for your concatenated field     
    newfld = str(fldDict["SiteStreet"]) +" "+str(fldDict["StreetType"])    
    #This Takes them out of the loop    
    del fldDict["SiteStreet"]    
    del (fldDict["StreetType"])
    fldList.remove("SiteStreet")
    fldList.remove("StreetType")
    for w in range(len(fldList)):    
        fldAttList.append(fldDict[fldList])    
    manualFields =  ["SiteState","FacltyType", "StructType","GIS_STEW", "UpdateBy","Verified", "Status", "StructCat", "APA_CODE","AddressID", "POINT_X", "POINT_Y","StreetName"]    
    for fld in manualFields:    
        fldList.append(fld)    
    fixedAtts = ["ID","Single Family Home","Primary, Private","CanyonCo", "TA", "Yes, GRM, TA","Active","Residential","1110", AddressID, x, y,newfld]   
    for att in fixedAtts:    
        fldAttList.append(att)    
    fldAttList.append(point1)    
    fldtuple = (fldAttList)    
    fldList.append( 'SHAPE@XY')    
    tprow = arcpy.da.InsertCursor(targetpoint, fldList)   
    tprow.insertRow(fldtuple)  
  
       
del tprow

Re: populate x, y on mouse click

CCWeedcontrol — Tue, 21 Jul 2015 14:34:56 GMT

So the fldList.remove removes the fields from list fldList.

Awesome! thanks.

Re: populate x, y on mouse click

CCWeedcontrol — Tue, 21 Jul 2015 14:37:38 GMT

sorry, forgive my dumbness but i am not sure i understand?

Re: populate x, y on mouse click

WesMiller — Tue, 21 Jul 2015 14:46:25 GMT

Would be willing to upload a sample of the parcel file(4 or 5 parcels) and a sample of the target point file?

Re: populate x, y on mouse click

CCWeedcontrol — Tue, 21 Jul 2015 14:53:02 GMT

ya of course.

Re: populate x, y on mouse click

WesMiller — Tue, 21 Jul 2015 14:58:19 GMT

I'll get back with you tommorrow

Re: populate x, y on mouse click

WesMiller — Sat, 11 Dec 2021 07:24:55 GMT

I tweaked the code a little to make it easier to read and tested and everything appears to be working.

#import modules    
import arcpy    
"""  
tool box parameters  
point param = feature set  
targetpoint = featurelayer  
parcel = featurelayer  
"""  
arcpy.env.qualifiedFieldNames = False  
  
point = arcpy.GetParameterAsText(0)  #point feature set  
targetpoint = "PointsTest" #target point feature class   
parcel = "ParcelsTest"  #parcel feature class  
parcel_lyr = 'parcel_lyr'    
     
for prow in arcpy.da.SearchCursor(point,'SHAPE@XY'):    
    x,y = prow[0]  
del prow    
     
point1 = arcpy.Point(x, y)    
ptGeometry = arcpy.PointGeometry(point1)  
  
CC_list = []  
with arcpy.da.SearchCursor(targetpoint, ["AddressID"]) as cursor:  
    for row in cursor:  
        try:  
            if "CC" in row[0]:  
                CC_list.append(int(row[0].strip("CC")))     
        except TypeError:  
            pass          
del cursor  
  
CC_list.sort()  
AddressID = CC_list[-1] + 1  
AddressID = 'CC' + str(AddressID)  
     
arcpy.MakeFeatureLayer_management(parcel,parcel_lyr)    
arcpy.SelectLayerByLocation_management(parcel_lyr,"INTERSECT",ptGeometry)    
    
fldList = ['ACCOUNT','OwnerName','SiteAddres','SiteNum','siteNumSfx','Predir','SiteStreet', 'StreetType', 'Postdir', 'SiteCity', 'SiteZip']    
fldDict ={}    
  
#Check that we only have one parcel and get the attributes from it  
if int(arcpy.GetCount_management(parcel_lyr).getOutput(0))==1:    
    for parrow in arcpy.da.SearchCursor(parcel_lyr,fldList):    
        for w in range(len(fldList)):    
            fldDict[fldList]=parrow    
    del parrow    
       
    targetFields = ['ACCOUNT', 'OwnerName', 'SiteAddres', 'SiteNum', 'siteNumSfx', 'Predir', 'Postdir', 'SiteCity', 'SiteZip',
                    'SiteState', 'FacltyType', 'StructType', 'GIS_STEW', 'UpdateBy', 'Verified', 'Status', 'StructCat', 'APA_CODE',
                    'AddressID', 'POINT_X', 'POINT_Y', 'StreetName'] 
        
    tprows = arcpy.da.InsertCursor(targetpoint, targetFields)
    row = []
    #Attributes from parcels
    row.append(fldDict['ACCOUNT'])
    row.append(fldDict['OwnerName'])
    row.append(fldDict['SiteAddres'])
    row.append(fldDict['SiteNum'])
    row.append(fldDict['siteNumSfx'])
    row.append(fldDict['Predir'])
    row.append(fldDict['Postdir'])
    row.append(fldDict['SiteCity'])
    row.append(fldDict['SiteZip'])
    #Preset attributes
    row.append('ID')
    row.append('Single Family Home')
    row.append('Primary, Private')
    row.append('CanyonCo')
    row.append('TA')
    row.append("Yes, GRM, TA")
    row.append('Active')
    row.append('Residential')
    row.append('1110')
    row.append(AddressID)
    row.append(x)
    row.append(y)
    row.append(fldDict['SiteStreet'] + " " + fldDict['StreetType'])
    tupleRow = (row)
    tprows.insertRow(tupleRow)  
  
       
del tprows

Re: populate x, y on mouse click

CCWeedcontrol — Wed, 22 Jul 2015 14:39:12 GMT

Interesting, on the row.append can you explain what the benefit is to this versus List.append

I have added the code to a script and runs but no point is actually created. I get no errors.

Re: populate x, y on mouse click

WesMiller — Wed, 22 Jul 2015 14:51:48 GMT

For me it's easier to see what's going on and looks cleaner

Re: populate x, y on mouse click

CCWeedcontrol — Wed, 22 Jul 2015 15:09:04 GMT

ok, i wasn't sure if there was an advantage to it or not.

Any ideas on why the point does not get created after i run the script?

Re: populate x, y on mouse click

WesMiller — Sat, 11 Dec 2021 07:24:58 GMT

Yes i forgot to add the field back, here is the working script.

#import modules    
import arcpy    
"""  
tool box parameters  
point param = feature set  
targetpoint = featurelayer  
parcel = featurelayer  
"""  
arcpy.env.qualifiedFieldNames = False  
  
point = arcpy.GetParameterAsText(0)  #point feature set  
targetpoint = "PointsTest" #target point feature class   
parcel = "ParcelsTest"  #parcel feature class  
parcel_lyr = 'parcel_lyr'    
     
for prow in arcpy.da.SearchCursor(point,'SHAPE@XY'):    
    x,y = prow[0]  
del prow    
     
point1 = arcpy.Point(x, y)    
ptGeometry = arcpy.PointGeometry(point1)  
  
CC_list = []  
with arcpy.da.SearchCursor(targetpoint, ["AddressID"]) as cursor:  
    for row in cursor:  
        try:  
            if "CC" in row[0]:  
                CC_list.append(int(row[0].strip("CC")))     
        except TypeError:  
            pass          
del cursor  
  
CC_list.sort()  
AddressID = CC_list[-1] + 1  
AddressID = 'CC' + str(AddressID)  
     
arcpy.MakeFeatureLayer_management(parcel,parcel_lyr)    
arcpy.SelectLayerByLocation_management(parcel_lyr,"INTERSECT",ptGeometry)    
    
fldList = ['ACCOUNT','OwnerName','SiteAddres','SiteNum','siteNumSfx','Predir','SiteStreet', 'StreetType', 'Postdir', 'SiteCity', 'SiteZip']    
fldDict ={}    
  
#Check that we only have one parcel and get the attributes from it  
if int(arcpy.GetCount_management(parcel_lyr).getOutput(0))==1:    
    for parrow in arcpy.da.SearchCursor(parcel_lyr,fldList):    
        for w in range(len(fldList)):    
            fldDict[fldList]=parrow    
    del parrow    
       
    targetFields = ['ACCOUNT', 'OwnerName', 'SiteAddres', 'SiteNum', 'siteNumSfx', 'Predir', 'Postdir', 'SiteCity', 'SiteZip',
                    'SiteState', 'FacltyType', 'StructType', 'GIS_STEW', 'UpdateBy', 'Verified', 'Status', 'StructCat', 'APA_CODE',
                    'AddressID', 'POINT_X', 'POINT_Y', 'StreetName','SHAPE@XY'] 
        
    tprows = arcpy.da.InsertCursor(targetpoint, targetFields)
    row = []
    #Attributes from parcels
    row.append(fldDict['ACCOUNT'])
    row.append(fldDict['OwnerName'])
    row.append(fldDict['SiteAddres'])
    row.append(fldDict['SiteNum'])
    row.append(fldDict['siteNumSfx'])
    row.append(fldDict['Predir'])
    row.append(fldDict['Postdir'])
    row.append(fldDict['SiteCity'])
    row.append(fldDict['SiteZip'])
    #Preset attributes
    row.append('ID')
    row.append('Single Family Home')
    row.append('Primary, Private')
    row.append('CanyonCo')
    row.append('TA')
    row.append("Yes, GRM, TA")
    row.append('Active')
    row.append('Residential')
    row.append('1110')
    row.append(AddressID)
    row.append(x)
    row.append(y)
    row.append(fldDict['SiteStreet'] + " " + fldDict['StreetType'])
    row.append(point1)
    tprows.insertRow(row)  
  
       
del tprows

Re: populate x, y on mouse click

CCWeedcontrol — Wed, 22 Jul 2015 15:34:32 GMT

Got it, thanks for all your help