Skip to content

Abm3 recalib#384

Open
aletzdy wants to merge 6 commits intomainfrom
ABM3_recalib
Open

Abm3 recalib#384
aletzdy wants to merge 6 commits intomainfrom
ABM3_recalib

Conversation

@aletzdy
Copy link
Collaborator

@aletzdy aletzdy commented Jan 28, 2026

Proposed changes

Updated mode choice configs as a result of calibration using 2023 OBTS data

Impact

Model boardings are now in line with 2023 OBTS data.

Types of changes

What types of changes does your code introduce to ABM?
Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update (if none of the other choices apply)

How has this been tested?

Please describe the tests that you ran to verify your changes.

  • Model was run from start to finish.
  • Test B

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

Further comments

This new code will be dependent on the new bike model branch, since the updated configs use the latest logsums out of the new bike model.

@aletzdy aletzdy requested a review from JiaXu1024 January 28, 2026 21:53
@JiaXu1024 JiaXu1024 requested a review from Copilot January 28, 2026 22:02
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR implements a recalibration of the ABM3 mode choice model using 2023 OBTS (On-Board Transit Survey) data. The calibration updates mode choice coefficients across resident, visitor, crossborder, and airport models to align model boardings with observed 2023 transit ridership patterns.

Changes:

  • Updated transit mode choice calibration coefficients across all model segments
  • Refined bike logsums and transfer penalty coefficients
  • Disaggregated transit modes from aggregated (WALK_TRANSIT, PNR_TRANSIT, etc.) to service-specific modes (WALK_LOC, WALK_PRM, WALK_MIX, etc.)
  • Updated calibration utilities to support disaggregated transit modes

Reviewed changes

Copilot reviewed 24 out of 28 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
abm3_settings.yaml Updated IVT multipliers and ASC values for transit modes
visitor trip/tour mode choice files Added transit calibration coefficients by purpose
resident trip/tour mode choice files Transitioned from aggregated to disaggregated transit mode coefficients
crossborder trip mode choice files Added purpose-specific transit calibration coefficients
airport trip mode choice files Added access-mode-specific transit calibration coefficients
calibration utility scripts Updated mode mappings and scaling logic to handle disaggregated transit modes
calibration settings files Adjusted sample sizes and process counts for calibration runs

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

ready: 0.322
no_pass: 0.459
num_tours: 120700
no_pass: 0.45899999999999996
Copy link

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The no_pass share value contains floating point precision artifacts. Round this to a cleaner value like 0.459 to match the precision of other share values in the file.

Suggested change
no_pass: 0.45899999999999996
no_pass: 0.459

Copilot uses AI. Check for mistakes.
else: # only enters if above for loop breaks
tour_mc_calib_target_tables.append(df_ct)

print(tour_mc_calib_target_tables)
Copy link

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This debug print statement should be removed or converted to proper logging for production code.

Copilot uses AI. Check for mistakes.
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.

1 participant