-
Notifications
You must be signed in to change notification settings - Fork 134
Explicit error terms production code #1064
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
Open
janzill
wants to merge
318
commits into
ActivitySim:main
Choose a base branch
from
outerl:explicit_error_terms
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
318 commits
Select commit
Hold shift + click to select a range
cb807d1
remove usage of choose_individual_max_utility
f0cc563
interaction_sample util based w/p probs
417e90f
wrong method in cdap
4f825ae
style fixes
7a6cc29
need probs in interaction_sample - try it the quick way
f0143e9
remove false comparison
f6d81d6
bug fix
b449593
bug fix
dbc809a
add utility based choice option for transit virtual pathbuilder
4c6a524
custom chooser
2ec41b1
mem tracing in interaction_sample
47601e3
bug fix
e0555d4
interaction_sample memory saving implementation
2568b72
memory work
78630be
spelling
c6d75d1
delete after last ref, not before
7a66a67
set index
7df4c8d
to numpy once
d6f4f97
remove comment
26a7090
interaction sample dev
313bcd0
remove scale and location params from ev1 quartile fct
6451abd
use numpy's gumbel to draw from ev1
3973bcd
clean up
c0d1ca1
gumbel by hand by inverse cdf
6b0b4e9
memory logging
9a17395
resolves merge conflicts with main v1.2
b86093f
Merge branch 'master' into janzill/utilit_based_choices_2025
792d183
merge bug
08d013a
more merge bugs
75bb84d
more merge fixes, example runs through
c6feaef
log eet setting on start up
114b747
some variation checks, dev notebooks, to delete
e1e9d08
seeding corr test
14441a6
corr
73b8c32
sign on logsum
f5f365f
removes notebook folder
0f54cec
compute_setting overrides for eet, currently only for interaction_sam…
11d86a1
Adds subcomponent-specific eet setting
30ddb3e
default compute settings set before, no need to check if none
5105b17
Merge remote-tracking branch 'origin/master' into janzill/utilit_base…
c6d60c6
comment clean up
6a0987d
identifies eet todos
5637fb0
test SOA MC for disagg access
6f8958b
lint
9d12b08
comments
9a0fb66
more comments
0fa183e
jtp custom chooser for EET
9900f48
lint
892c3fb
move interaction sample without sampling out of MC loop to enable for…
d6ecb0d
update comments
3793ec2
fix no sampling in interaction_sample
fc68697
fix _interaction_sample return doc
28142cd
fix bool conversion of None for eet compute settings in sub-components
5de1d23
compute settings in disaggreagte accessibility w/o hack
2a563ff
lint
ef0be21
fix nest_spec is None bug
1f815e8
remove comment
41d49ff
clean up comments
582c998
comment clean up
49fcaa5
Merge remote-tracking branch 'origin/master' into janzill/utilit_base…
c40b989
avoid choosing alternatives corresponding to padded utilities in inte…
5636cd9
lint
94e629a
patch other instance of padded_utility for eet
e102a9e
adds validate utility method to align with unavailable choices in MC …
66cdac9
consistently treat utilities that would lead to zero choices in MC si…
3b48a07
lint
6f31c76
fix allow_zero_probs for prob calculation in EET interaction_sample
7fe8746
Merge branch 'explicit_error_terms_2'
jpn-- 57ba5bc
fix unbound variable in estimation mode
jpn-- 45ab2ca
blacken
jpn-- 0754bb0
Merge branch 'main' into eet
jpn-- 82f1f7d
update larch pin to 6.0.46
jpn-- 5604b3e
Merge pull request #1042 from driftlesslabs/eet
jpn-- 5c506f1
np array instead of for loop
janzill 710b3b0
memory reduction
janzill 60a744a
no duplicate arrays
janzill f97185e
bug fix: order of chooser_idx in interaction_simulate
janzill 9d69dab
add tests, docstrings for logit
tomstephen a9db131
Add basic with/without EET test for interaction simulate
tomstephen a7f2e8f
Fix, complete tests for interaction sample, simulate
tomstephen df574ee
Linting
tomstephen cf92aad
Normalise number of choosers, alternatives and minimum tolerance for …
tomstephen 73f45fe
reshape, do not flatten for potential performance
janzill 6440d3f
undo stray comment
janzill 1df7d0d
unify mc and eet reporting during choice making
janzill 5e9847d
series not array
janzill 1213b56
reinstate test, up number of draws for comparison
janzill fa54204
numpy not loop
janzill 1b2b6dc
interaction_sample test to catch index order bug
janzill 4e0c7e8
make test clearer
janzill 1aa85ba
reset rng offset on iterate_location_choice (shadow pricing)
janzill bd4211f
Add tests for logit NL, ordering, and all models using EET
tomstephen 76cacb2
Add basic docs for EET
tomstephen f628505
Add tests checking that choices made using eet and from probabilities…
TylerP132 362eef0
Linting, minor changes to test_simulate.py
tomstephen d0a6429
roll back changes to core tests to minimize noise
janzill 65fd171
Implement Jan's suggestion of how to calculate household ids for test…
TylerP132 f55b2cd
Add test for compute_nested_utilities. Computes nested utilities on a…
TylerP132 4debaad
Linting
tomstephen 743c56f
Move nest_spec to a fixture in simulate.py tests
TylerP132 b62001e
Merge with Tom's linting changes
TylerP132 b021cee
Finish removing nest_spec definition in simulate.py tests
TylerP132 52f75fa
Merge pull request #5 from outerl/tom/nested-logit-tests-and-docs
janzill ae71433
lint
janzill 6ef25e1
clean up
janzill d491e45
Merge branch 'jzill/runtime_improvement' into jzill/reset_rng_on_shad…
janzill cc8fda0
reset rng only for eet for now. discuss in engineering meeting
janzill c1806fc
Close out remaining TODO-EETs
tomstephen 3270474
Shuffle check to logit.py
tomstephen dd54711
Linting
tomstephen a73f6ea
Fix test
tomstephen b455e2a
Roll back validate_utils
tomstephen a490999
eet doco start
janzill 661c6b3
doco part two
janzill a0bc5ba
Merge branch 'jzill/runtime_improvement' of github.com:outerl/activit…
janzill 40b0b06
add scale comment
janzill 21daa5c
Revert linting changes
tomstephen 0da80bf
.
tomstephen 1d139e0
Fix failing tests
tomstephen 605366f
doc fix
janzill 8c74d2a
docs
janzill f542e58
doco clean up
janzill 2d1dbcb
doco re memory
janzill 837ae41
doco clean up
janzill d674268
memory usage doco
janzill 7959c94
clean up doco
janzill a7e8529
undo changes in unrelated code to reduce noise
janzill c5bd23f
Merge pull request #3 from outerl/jzill/runtime_improvement
janzill be3a270
testing for semcog model, update to shadow pricing reset
janzill 7209985
lint
janzill 22a4f62
disable test unitl further investigation
janzill 220c5cf
mp rng reset, disabling one new test for no
janzill d50ad3c
semcog test with different seed for eet mp to work around non-determi…
janzill d5a8fd4
no need to restrict to specific choosers for reset, each process has …
janzill 033f08a
Add eet multiple zone test for zone=2
TylerP132 f5b6301
Change formatting from using ruff to black
TylerP132 5b24d44
add back accidentally removed import
janzill 9661d01
re-add 3-zone tests
janzill 7c96cbc
base seed same as non-eet, update regress eet data
janzill c786919
progressive 2 zone test
janzill 08b2b03
Merge pull request #6 from outerl/explicit_error_terms-multiple_zone_…
janzill 0d2356e
Copy changes from m-richards:matt/poisson_eet_2026_04
tomstephen 89dbe5c
Tidy, complete implementation of alts_context, add tests, rebuild reg…
tomstephen 47ed600
Commenting/fix alts context constructions, update semcog test
tomstephen 2d181b2
Deterministic randoms when adding/removing tours for trip scheduling …
tomstephen cfe6166
PR noise
tomstephen 7a22612
Roll back final_trips.csv for production_semcog
tomstephen bc56a87
PR noise, make sure alts_context built on un-modified alternatives in…
tomstephen 531b562
linting
tomstephen 19750a0
updates outdated comment
janzill 834fc90
re-add arc test, remove parking choice column from regress trips
janzill 6d29f18
re-instate arc tests
janzill 12a3d8a
Revert "Deterministic randoms when adding/removing tours for trip sch…
janzill 3bd3251
no alts_context for trip_scheduling
janzill 8d7ae41
updates arc regress files
janzill f2d7bfa
removes outdated comment
janzill d7c1345
trip scheduling comment for EET
janzill e9fb130
disable arc sharrow test until regress trips are updated
janzill 10db123
doco update
janzill fcef6b3
Merge pull request #7 from outerl/deterministic-alt-sample-randoms
janzill 1e9f26c
Matt's Poisson sampling changes from PR 1065
janzill 3bde470
bug fix
janzill 346f0e8
add second return arg for zero probs as well
janzill 32a2662
eet for nested logit
janzill e0dbad8
towards a unified choice interface
janzill e944d1e
test and doco updates
janzill 968da57
jtp fix
janzill 04b2a28
fused location sampling
janzill 8f0ae70
c order to map the first n_alt rands to all alternatives for the firs…
janzill 510c6ab
fused mnl ev1 argmax with alt_info
janzill 3ae81fa
stable sampling
janzill cb0c5d4
some doco updates
janzill bd762c8
doc clean up
janzill acaa22a
Merge remote-tracking branch 'origin/jzill/eet_poisson_sampling_from_…
janzill 248defd
poisson sample tests and runtime improvements
janzill cfdd7da
tests for random sampling as fallback, some doc strings and removal o…
janzill ee3c429
avoid unneccessary dataframe construction, better doco
janzill 68faa94
update interaction sample tests for poisson
janzill 73d83e6
integration of sampling methods with clear separation from eet as sim…
janzill e8fb1d7
poisson sampling optimization
janzill ccd5098
poisson sampling with stable rands across scenarios where attractions…
janzill fecb26d
do not throw on poisson fallback sampling with sample_size > alternat…
janzill 0cb9289
debug logging
janzill 3c4a3b1
clean up
janzill e4e415a
sampling doco
janzill b7b1d90
clean up
janzill bf24aa5
doco update
janzill 067448c
linting
janzill 1f731ca
razzmatazz. I mean maz for taz.
janzill 7731971
test fixes and stable sorting for tour/trip loc choice
janzill 480c8ad
no stable alts for MC
janzill c8470ec
lint
janzill a77cbec
stable two-zone maz_from_taz for Poisson
janzill e91e566
more doco
janzill a6ff459
no op if no alts_context
janzill 7aaeb1b
do not throw on interaction_sample_simulate w/o alts_context
janzill e9a2813
eet special casing
janzill 6d0a24e
tour_od with maz_taz stable, clean up
janzill 5b0c79e
arc eet golden trips
janzill 429f954
Merge branch 'jzill/eet_runtime_inc_nl_and_poisson' of github.com:out…
janzill 5c961b7
switch base seed to avoid trip_scheduling (probabilistic) to come up …
janzill a325f45
forgot the corresponding golden trips
janzill 4fcd239
test multiple_zone golden
janzill 8933d6c
removes requirement of interaction_sample_simulate to have alts_context
janzill 270d450
fix test by using eet as intended, remove stable_indexing in tour_des…
janzill 01b38c7
clean up
janzill 4b78e20
conditional stable sample indexes
janzill ae278cc
stable alts only for eet with poisson sampling for two-zone
janzill de74dfc
trip maz-for-taz stable alts for eet with poisson sampling
janzill 77b80b3
clean up
janzill bdfe757
stable two-zone alts for tour_od
janzill 67fd055
tour_od stable alt cond
janzill 249acfa
resolve_smapling_method
janzill 3ebe20c
decouple sample and simulation methods
janzill dc52907
separate RNG for shadow pricing to enable loc choiec rng reset for si…
janzill d58d0c5
lint
janzill 7f68160
logging and comments
janzill e6b75ea
debug logging for sample method, add warning for disagg acc and poisson
janzill 4803032
doco
janzill d11f2be
info, nto warning
janzill 7b7170b
compute settings, not model settings
janzill b844372
compute settings, not model settings
janzill 196704a
variable name
janzill 1567d5f
Merge pull request #8 from outerl/jzill/eet_runtime_inc_nl_and_poisson
janzill 99239c2
log log on subset of rands only
janzill 7768547
optionally bias poisson disagg acc
janzill aa13d2d
bias also for non-mand
janzill 95c1058
avoid copy in favour of log log for gumbel choice
janzill 82cd511
switch to bias all location choice logsums for poisson sampling
janzill 5bfd4b3
add setting
janzill 7424aec
adds missing import in trip dest
janzill 0f2ddbd
adds missing import in trip dest
janzill 3d6f1db
ling
janzill 57156aa
Merge pull request #9 from outerl/jzill/eet_cleanup
janzill 979337b
addresses claude code review
janzill a57e78a
consistent spelling
janzill ac2adf3
mal-formed html comment
janzill d3b64ba
doc ref
janzill 9a10edb
addresses copilot code review
janzill 05452ec
remove try block
janzill c226928
clean up
janzill a93da9c
logit clean up
janzill 28b4011
clean up
janzill 82b208f
lint
janzill a859214
estimation safe guard only for poisson
janzill 1cc0ce5
merge origin/main
janzill ecb3131
deletes arc test as discussed in engineering meeting in June
janzill d439c7f
adding skip_failed_choices into test, removing dead function parameter
janzill c93d307
fix eet trip_dest presample test
janzill fa5b1b7
Merge pull request #10 from outerl/jzill/eet_llm_review
janzill File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggest we parameterize this 3.0 value in logit.py (named like UTIL_LARGE_ENOUGH or something). Shows up in interaction_sample_simulate too in the same context.