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

Change the interpretation of zoning_heights in feasibility model #168

Open
hanase opened this issue Jan 14, 2019 · 2 comments
Open

Change the interpretation of zoning_heights in feasibility model #168

hanase opened this issue Jan 14, 2019 · 2 comments

Comments

@hanase
Copy link
Collaborator

hanase commented Jan 14, 2019

The UDST feasibility model determines the max FAR for mix-use parcels using the zoning_heights table as follows:

  1. FAR from the heights column is computed (max_far_from_heights).
  2. If the table contains the column max_dua, it calculates the corresponding FAR (max_far_from_dua) and takes the minimum of max_far, max_far_from_heights, and max_far_from_dua.
  3. Otherwise, it takes the minimum of max_far and max_far_from_heights.

The last two steps are implemented in the _min_max_fars function.

Our interpretation of the table is that max_far limits the non-residential part whereas max_dua limits the residential part of the mixed development. Thus, taking the minimum over these is wrong.

@hanase
Copy link
Collaborator Author

hanase commented Jan 14, 2019

We now have a child class that overwrites the _min_max_fars function. Step 2. above is replaced by taking the sum of max_far_from_dua and max_far capped by the max_far_from_heights. In Step 3., max_far_from_heights is only considered if max_far is not given.

@hanase hanase closed this as completed Jan 14, 2019
@hanase
Copy link
Collaborator Author

hanase commented Jan 17, 2019

@jensenmj Mike, given Beccah's notes on the FLU columns, how do you think this should be changed?

@hanase hanase reopened this Jan 17, 2019
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

1 participant