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

Fix min inbound departure times #323

Closed
wants to merge 3 commits into from

Conversation

mxndrwgrdnr
Copy link
Contributor

@mxndrwgrdnr mxndrwgrdnr commented Jun 29, 2020

Updates to two modules to fix the issue in which the first inbound trip of a given tour is scheduled to depart prior to the last outbound trip of the tour:

  • models/trip_scheduling.py -- new method to update earliest column in trips table after outbound trips are scheduled
  • models/util/trip.py -- updates def of failed_trip_cohorts. cohort must include all trips associated with a tour that has a failed trip because of the new interdependence between inbound and outbound departure times.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.6%) to 86.98% when pulling 155c53b on ual:fix_min_inbound_depart into a3dca1f on ActivitySim:master.

@bstabler bstabler changed the base branch from master to develop July 9, 2020 23:30
@bstabler bstabler changed the base branch from develop to master July 9, 2020 23:31
@bstabler
Copy link
Contributor

Hi @mxndrwgrdnr, thanks for the PR. The master branch is our current release branch and is only pushed to by Travis once develop passes the tests. In addition, since this PR has some revisions already in the RSGInc fork develop branch, I think it is best to submit this PR to the RSGInc fork develop branch. I'm going to close this PR and then we can follow-up on the new PR. Thanks.

@bstabler bstabler closed this Jul 15, 2020
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.

3 participants