A quick fix to ensure we no longer include "misc" in the acitivity field for Idaho parcels. The root cause of this was rows in Miscellaneous.shp and Easements.shp with NULL values for the Type and EasementPu columns, respectively. In the case of NULL values, we compute an activity_name of None and fallback to the original activity.name in the final return, which in this case is "misc".
The fix here is to actually correct the data, using the string "Miscellaneous" for NULL values in the Type column of Miscellaneous.shp and "Easements" for NULL values in the EasementPu column for Easements.shp.
One last thing: I noticed we were using the column name Sale_Status as the activity column for Timber.shp, but that column is missing from that dataset. I assume we mean the column Sale_Statu? This PR includes that update as well.
A quick fix to ensure we no longer include
"misc"in theacitivityfield for Idaho parcels. The root cause of this was rows inMiscellaneous.shpandEasements.shpwithNULLvalues for theTypeandEasementPucolumns, respectively. In the case ofNULLvalues, we compute anactivity_nameofNoneand fallback to the originalactivity.namein the finalreturn, which in this case is"misc".The fix here is to actually correct the data, using the string
"Miscellaneous"forNULLvalues in theTypecolumn ofMiscellaneous.shpand"Easements"forNULLvalues in theEasementPucolumn forEasements.shp.One last thing: I noticed we were using the column name
Sale_Statusas the activity column forTimber.shp, but that column is missing from that dataset. I assume we mean the columnSale_Statu? This PR includes that update as well.