Skip to content

Poisson EET changes#7

Open
m-richards wants to merge 159 commits intoVictoriaTransport:vitmfrom
m-richards:matt/eet_poisson_on_v24_1_5_2
Open

Poisson EET changes#7
m-richards wants to merge 159 commits intoVictoriaTransport:vitmfrom
m-richards:matt/eet_poisson_on_v24_1_5_2

Conversation

@m-richards
Copy link
Copy Markdown

I've opened this pull request for visibility mainly.
This compliments #6, in that it is the extension of that to add poisson sampling and the various fixes. That said I think it's arguably easier to look at https://github.com/m-richards/activitysim/pull/1/changes
which is the diff of #7 against #6 - this is how we were thinking about these changes during development. There's a few strategies to try and clean this up / re-synchronise the VITM2 fork cleanly.

  1. Merge EET on VITM 2 activitysim #6 and then merge Poisson EET changes #7 afterwards
  2. Throw away EET on VITM 2 activitysim #6 and just merge this (Poisson EET changes #7)
  3. Wait until the Outer Loop work is done, merged into the activitysim explicit_error_term branch, and then take that as the starting point, and apply the patches in Poisson EET changes #7 on top.

Happy to discuss / give a recommendation depending on what you want to achive and how the fork should be synced going forward.

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