Skip to content
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

A solution script to mistagged condos #97

Open
jschleuss opened this issue Jul 15, 2016 · 3 comments
Open

A solution script to mistagged condos #97

jschleuss opened this issue Jul 15, 2016 · 3 comments
Assignees
Labels

Comments

@jschleuss
Copy link
Contributor

God love 'em, condos have been coming in as building=house and building:units=1

They should be marked as building=apartments.

I've started a Python notebook to script to sort this out. By the end of the import we'll have about 18,500 or so condos incorrectly tagged. This is happening because condo is stored on the "SepcificUseDetail2" field, which we did not use. We pulled from "SpecificUseType" which labels these as "Single Family Residence."

I'm posting the start of this script to be transparent about the idea and seek any fixes. It will need to loop through all the condo AINs in the file also added to the repo. After an update we can start to run it and store the leftover AINs in a separate file to recheck once we're done importing the whole county.

The reason for all this madness is that condos, unlike apartment buildings, get individual parcel shapes. One condo building may have many parcels stacked underneath it. So, when the county joined the parcel shapefile to the building shapefile the top parcel number was sent to the building's "AIN" field.

One question: should we update the lacounty:ain tag to include any other possible numbers? Or delete it? Or leave it? I'm mixed. Not sure the limit of all the concatenated tags.

@maning maning added the cleanup label Jul 15, 2016
@jschleuss jschleuss self-assigned this Jul 29, 2016
@jschleuss
Copy link
Contributor Author

I've been testing this out using this jupyter notebook https://github.com/osmlab/labuildings/blob/master/find-and-fix-condos.ipynb

@jschleuss
Copy link
Contributor Author

I have few reservations about renaming "condos" like this from building=house to building=apartments

That example looks to be a small, single-family dwelling. However, it is on one parcel with the other nearby buildings. They are condos, but maybe not "apartments" as the OSM Wiki would define.

@jschleuss
Copy link
Contributor Author

jschleuss commented Aug 1, 2016

One bug: the list of AINs from the assessor listed as condos include buildings that appear to be single-family homes. Those should remain "house." Possible solution is to gather AINs of condos in the assessor file that have more than one with the same lat/lon. That would mean that parcels sit on top of each other and mean that more than one unit sits on the parcel, which would be appropriate for building=apartments.

Good: http://www.openstreetmap.org/way/430766661
Bad (maybe): http://www.openstreetmap.org/way/431197699

Shout if you have thoughts. I've only done about 40 parcels.

Those homes sit on top of 86 stacked parcels, which are quite large.
screen shot 2016-08-01 at 11 53 16 am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants