-
Notifications
You must be signed in to change notification settings - Fork 21
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
Shared cost factor only applied to parking cost #138
Comments
Visitor tour mode choice has shared cost factors set to 1, inconsistent with visitor trip mode choice where shared 2 parking cost is divided by 0.57 |
Looks like the value of cost_share_s2 is overwritten in visitor trip_mode_choice.yaml, so visitor trip mode choice is being divided by 2, not by 0.57. Would still be good to update the value in common constants.yaml for consistency, and is still inconsistent with visitor tour mode choice |
We should also be splitting up TNC and Taxi costs per-passenger. Should this be done using the average occupancy for these modes? |
-- we are explicitly modeling party size so we should not be dividing cost or toll. This bug seems to exist in abm2+ as well #partially addressing Shared cost factor only applied to parking cost #138
costFactorS2 and costFactorS3 are applied only to parking cost, should be applied to all auto costs for shared rides
ABM/src/asim/configs/resident/trip_mode_choice.csv
Line 14 in b8ed161
Also looks like cost_share_s2 and cost_share_s3 used for non-resident models has inconsistent value. cost_share_s2 is 2 in crossborder/constants.yaml but 0.57 in common/constants.yaml. As a result, shared 2 parking cost is currently divided by 0.57 in visitor model.(Value is overwritten with 2 for visitor but is inconsistent with tour mode choice, see replies)ABM/src/asim/configs/common/constants.yaml
Line 247 in b8ed161
ABM/src/asim/configs/visitor/trip_mode_choice.csv
Line 10 in b8ed161
Shared cost factor appears to be missing completely from airport model
ABM/src/asim/configs/common_airport/trip_mode_choice.csv
Line 27 in b8ed161
The text was updated successfully, but these errors were encountered: