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

abort handler #2417

Draft
wants to merge 24 commits into
base: master
Choose a base branch
from
Draft

abort handler #2417

wants to merge 24 commits into from

Conversation

kainagel
Copy link
Contributor

No description provided.

@kainagel kainagel marked this pull request as draft February 16, 2023 14:27
kainagel and others added 15 commits March 13, 2023 17:11
Dear Prof Nagel @kainagel, I am merging my script to the "abortHandler" branch. 

Currently, the model works as expected. I have also added a spacial plan file and vehicle file to prove the new model functions as desired. A manual check by going through the output events file return a positive results. Perhaps an integrated test can be added later on.  

In addition, there are some comments (marked with "//TODO Chengqi:" or simply "Chengqi"). Perhaps we can make some improvements.

Best regards,
Chengqi Lu
In addition to the functionality test (simpleTest), we also have iterations test, where mode choice is enabled. The necessary vehicle file for this purpose is also added.

@Inject MyAbortHandler(Network network, Map<String, TravelTime> travelTimeMap, Config config ){
travelTime = travelTimeMap.get( TransportMode.car );
router = new SpeedyALTFactory().createPathCalculator( network, new OnlyTimeDependentTravelDisutility( travelTime ), travelTime );
Copy link
Member

Choose a reason for hiding this comment

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

This router is created with for the default network and car travel times. This may not work for all drt modes.

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.

None yet

4 participants