-
Notifications
You must be signed in to change notification settings - Fork 97
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
los.py expecting number of TAZs in land use to match length of skim index #837
Comments
The source code already provides a solution to this issue. See code snippet below. When the TAZs in the land use data mismatch the skims (which is reasonable as you said re: externals), Users can specify a activitysim/activitysim/abm/tables/landuse.py Lines 50 to 61 in 739c2e4
|
The current beta release of ActivitySim expects the length of TAZs in the land use data to match the length of skim indices. This can cause a crash if the skims include external stations, but the land use data only contains internal zones. This is the case with SEMCOG implementation and might be with other models too.
I am brining this up here so others be aware of it, and we can discuss it if needed.
@JilanChen
The text was updated successfully, but these errors were encountered: