SEMCOG / semcog_urbansim

7 stars 6 forks source link

`residential_developer` step `target_units` #34

Closed tianxie1995 closed 2 years ago

tianxie1995 commented 2 years ago

In the residential_developer step, current target_units is calculated by https://github.com/SEMCOG/semcog_urbansim/blob/53212be07bc18f2a80a72766fe813704e1703630/models.py#L1324 and not taking the under-developing mcd_total_forecast table into account. So after the residential model step, it may still be possible for some MCDs to have insufficient HU supply for the HH target. Are we expecting this to happen?

sirishau commented 2 years ago

When that happens, can we decrement the target_vacancy by increments of 0.01, until there is enough HU supply?

From: tianxie @.> Sent: Thursday, June 16, 2022 3:02 PM To: SEMCOG/semcog_urbansim @.> Cc: Subscribed @.***> Subject: [SEMCOG/semcog_urbansim] residential_developer step target_units (Issue #34)

In the residential_developer step, current target_units is calculated by https://github.com/SEMCOG/semcog_urbansim/blob/53212be07bc18f2a80a72766fe813704e1703630/models.py#L1324 and not taking the under-developing mcd_total_forecast table into account. So after the residential model step, it may still be possible for some MCDs to have insufficient HU supply for the HH target. Are we expecting this to happen?

— Reply to this email directly, view it on GitHubhttps://github.com/SEMCOG/semcog_urbansim/issues/34, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADB4VUDAQY3Z7DANKXD4ZF3VPN2ZFANCNFSM5Y7YIHSA. You are receiving this because you are subscribed to this thread.Message ID: @.**@.>> CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.

tianxie1995 commented 2 years ago

That's a good idea. But I think instead of decreasing, for MCDs which need more HU supply, the target vacancy should be increased so num_agents / (1 - target_vacancy) is larger.

sirishau commented 2 years ago

Isn’t num_agents households? So you are releasing HU based on vacancy as opposed to placing HH.

From: tianxie @.> Sent: Thursday, June 16, 2022 4:03 PM To: SEMCOG/semcog_urbansim @.> Cc: Uppalapati, Sirisha @.>; Comment @.> Subject: Re: [SEMCOG/semcog_urbansim] residential_developer step target_units (Issue #34)

That's a good idea. But I think instead of decreasing, for MCDs which need more HU supply, the target vacancy should be increased so num_agents / (1 - target_vacancy) is larger.

— Reply to this email directly, view it on GitHubhttps://github.com/SEMCOG/semcog_urbansim/issues/34#issuecomment-1158080989, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADB4VUCDF562PQJGRXUMTHLVPOCAPANCNFSM5Y7YIHSA. You are receiving this because you commented.Message ID: @.**@.>> CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.

semcogli commented 2 years ago

How about we use target HHs from MCD forecast to replace num_agents?


From: tianxie @.***> Sent: Thursday, June 16, 2022 4:03:19 PM To: SEMCOG/semcog_urbansim Cc: Subscribed Subject: Re: [SEMCOG/semcog_urbansim] residential_developer step target_units (Issue #34)

That's a good idea. But I think instead of decreasing, for MCDs which need more HU supply, the target vacancy should be increased so num_agents / (1 - target_vacancy) is larger.

— Reply to this email directly, view it on GitHubhttps://github.com/SEMCOG/semcog_urbansim/issues/34#issuecomment-1158080989, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABVYO7KAJN6WTHYKWIR4KO3VPOCAPANCNFSM5Y7YIHSA. You are receiving this because you are subscribed to this thread.Message ID: @.***>

CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.