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

Parking Location Choice Memory Reduction #849

Closed
wants to merge 2 commits into from

Conversation

dhensle
Copy link
Contributor

@dhensle dhensle commented Apr 16, 2024

This PR removes unused columns in the parking location choice model when creating the interaction dataframe combining alternatives and choosers.

Built on top of the function drop_unused_chooser_columns that is part of #833.

@jpn-- jpn-- requested review from jpn-- and i-am-sijia April 18, 2024 17:05
i-am-sijia added a commit to wsp-sag/activitysim that referenced this pull request Apr 19, 2024
jpn-- added a commit that referenced this pull request Apr 21, 2024
* auto drop vars in choosers table

* formatting

* vars hardcoded in the custom chooser functions

* accidentally committed outputs

* skip dropping when tracing

* skip dropping if estimation mode

* test dropping before interaction df

* black

* revert drop inplace

* cleaning dup codes to function

* drop unused columns for logsum calculation

* blacken

* add drop_unused_columns to compute_settings

* check setting before dropping columns

* rename method; cherry-pick parking location changes from #849

* protect additional columns not in spec

---------

Co-authored-by: Jeff Newman <[email protected]>
@dhensle
Copy link
Contributor Author

dhensle commented Apr 22, 2024

Changes included in #833

@dhensle dhensle closed this Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant