forked from ActivitySim/activitysim
-
Notifications
You must be signed in to change notification settings - Fork 2
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
XBorder PR #9
Closed
Closed
XBorder PR #9
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
pull latest updates
pull some prettier expression file formatting updates
* move other resources into folder since examples now part of package as well * Add new skim lookups for the TNC bridge tolls return charges below. - odr - origin to destination reverse - dor - destination to origin reverse TNC revisions: - Update TNC bridge tolls so return charges are included - Added ride-hail single and taxi constants for trip mode choice within ride-hail tour mode choice - Reduced TNC wait time mean and sd slightly to in the highest density category - Turned on origin density index constant (already used for walk, bike and walk-transit modes) for TNC single and TNC shared * allow for a small number of stop locations within walking distance * updated tnc ascs from tm 1.5 run * add notebooks * add readme for notebooks folder * Interactive tutorial (#3) * Add verification jupyter notebook * add shapefile * code review * cleanup * update shapefile note Co-authored-by: Ben Stabler <[email protected]> Co-authored-by: bstabler <[email protected]> * clean up folder setup * finish summaries notebook * updated getting started tutorial * continue work on the recipe book * estimation notebooks for larch (#19) * notebooks for auto ownership and workplace location estimation * minimally functional larch estimation mode notebooks includes auto ownership, workplace location, tour mode requires larch >= 5.3.7 * revised notebooks uses new EDB for tour_mode_choice * update example EDBs and estimation notebooks * Fix tour mode choice notebook (#20) Fill in missing values in values DataFrame * update notebook * update getting started notebook * more notebook updates * notebook for estimating school location (#21) * create two and three zone test data, see also https://github.com/ActivitySim/activitysim/wiki/Multiple-Zone-Systems-Design * update notebook markdown * after merge from develop, resolve conflicts, pycodestyle, and add header to school location notebook * small doc fix * update tests EXPECT_TOUR_COUNT * resolve conflict Co-authored-by: Blake <[email protected]> Co-authored-by: Jeffrey Newman <[email protected]> Co-authored-by: Jeff Doyle <[email protected]>
… has yet to be implemented and adding a draft estimation functionality flow chart
…tr formatting in final trips table test targets
Merge in changes to Develop
remove deprecated method from trip scheduling
Docstring progress:
|
merge Develop
dhensle
pushed a commit
that referenced
this pull request
Feb 2, 2023
Work From Home & MWCOG extensions
dhensle
pushed a commit
that referenced
this pull request
Mar 28, 2024
BayDAG Contribution #9: Mode Choice Logsum Extraction
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I. Background
This branch was cloned from
develop
so that the codebase can be edited/updated without interfering with the working version of the XBorder code that lives indevelop
.To Discuss:
SANDAG/ABM/src/asim_cross_border/extensions/
and which changes should/must remain here incore/
/abm/
?To Do:
Goals:
master
, submit second PR to merge changes to the main (upstream) ActivitySim fork.II. Detail
Brand new model steps:
abm/models/util/
module called probabilistic_scheduling.py. This approach has the added benefit of streamlining the "trip_scheduling" code.example_sandag_xborder/extensions
Brand new modules:
abm/models/util/trip.py
origin
anddestination
columns that are inherited from the parent tour. The trip.py module provides a more intuitive home in the code base for this code, and consolidates it under a single module:initialize_from_tours()
. By isolating the trips initialization process in this way, it also makes it possible to initialize trips from within any number of arbitrary model steps. This in turn allows the user to compute trip-level metrics (e.g. trip mode choice logsums) for use in a model step that precedes "stop_frequency".abm/models/util/tour_od.py
abm/models/util/probabilistic_scheduling.py
Existing models with substantial changes:
trip.initialize_from_tours()
method to create a table of half-tour trips (one outbound, one inbound) based on the tour origin and destination of each tour for each potential tour mode choice alternative. Then use this table of pseudo-trips to call the "trip_mode_choice" model from within "tour_mode_choice". Save the trip_mode_choice logsums and append them to the tours table.abm/models/util/trip.py
abm/models/util/trip.py
module here.tour_earliest
constraint after the last outbound trip is scheduled as seen herescheduling_mode
parameter is completely optional, with the default set to the existing departure-based scheduling mode to ensure complete backwards compatibility; 2) This change is really a bug fix. So although it is fully backwards compatible with existing ActivitySim implementations, new inbound trip departure times are to be expected.Existing modules with substantial changes: