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

Feasibility Staleness hotfix #382

Open
wants to merge 26 commits into
base: main
Choose a base branch
from
Open

Feasibility Staleness hotfix #382

wants to merge 26 commits into from

Conversation

akselx
Copy link

@akselx akselx commented Sep 25, 2024

This branch is a hotfix for a stale feasiblity.

The issue is that a parcel may be feasible for multiple buildings - and may be assigned a residential building in residential_developer() only to be found feasible for a large retail project later, in retail_developer(). In practice, this joint allocation is unlikely, and should be handled only through explicit mixed use development forms.

Since the feasibility step is run before the first developer model step (residential) in a year's simulation cycle, it predates the placement of a building in the residential developer.

This fix re-runs the feasibility step before each developer model type.

The fix is implemented as:

  • additional feasiblity steps identified in baus.py
  • a separate filter for the developer model to only consider parcels not already developed by the developer model

Since this branch is built on top of #379 that PR should be merged first.


akselx and others added 26 commits August 28, 2024 20:09
Added a few inline comments on how to treat perhaps useful `task_name` argument
Added function to get git branch and commit hash to run inventory
…r from the baus runtime.

Data from BAUS run_setup.yaml
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

Successfully merging this pull request may close these issues.

2 participants