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

parcels.existing_units does not have number of units for residential units #170

Open
stefancoe opened this issue Jan 15, 2019 · 2 comments

Comments

@stefancoe
Copy link
Contributor

This variable almost always returns square footage and not units for buildings with residential units. This is because parcels.unit_name has no records with 'residential_units'.

@stefancoe stefancoe changed the title parcels.existing_units parcels.existing_units does not have number of units for residential units Jan 15, 2019
@stefancoe
Copy link
Contributor Author

@hanase should change logic so parcels.existing_units == parcels.residential_units when parcels.residential_units is greater than 0?

@hanase
Copy link
Collaborator

hanase commented Jan 16, 2019

@stefancoe It really depends where and how it is used. If a parcel has both, res units and non-res sqft, does the usage of this variable assumes that it gives total sqft?

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

No branches or pull requests

2 participants