Skip to content

Conversation

@lflake
Copy link

@lflake lflake commented Jun 7, 2021

No description provided.

@lflake
Copy link
Author

lflake commented Aug 16, 2021

@bstabler -- it sounds like this might need to be used soon for HTS weighting; can we merge this one?

@bstabler bstabler self-requested a review August 18, 2021 04:03
@bstabler
Copy link
Contributor

@lflake - please PR to develop instead of master. Thanks.

@lflake lflake changed the base branch from master to develop August 18, 2021 12:34
@lflake
Copy link
Author

lflake commented Aug 18, 2021

Done -- looks like the CI build is failing with this error in a few places, which I don't think is related to the code in the commits for this PR?

E ModuleNotFoundError: No module named 'cytoolz'

@lflake
Copy link
Author

lflake commented Aug 18, 2021

I see this is an issue on the other PR as well/is already known

@bstabler bstabler merged commit 41d3ea3 into ActivitySim:develop Aug 20, 2021
@bstabler
Copy link
Contributor

this looks good, thanks

@bstabler
Copy link
Contributor

#131

bstabler added a commit that referenced this pull request Aug 26, 2021
* update package version number as well

* Allow non-binary incidence (#123)

* Allow non-binary incidence

* style

* update tests to pass

* add some progress indication

* tidy up validation script, use histogram for a histogram

* fix render and some typos

* increment version

* deprecate py2.7

* Multiprocess (#130)

* [Bugfix] Allow seed and meta geography to be the same (#139)

* Fixes bug where if the seed geography is the same as the meta_geography, pandas has a small panic attack and the run will fail.

* add cytoolz to the "requirements"

* fix another activitysim change

* Absolute bounds (#136)


* adding upper/lower bounds to weighting use case

* #137, #134, #133, #131

Co-authored-by: Jamie Cook <[email protected]>
Co-authored-by: Blake Rosenthal <[email protected]>
Co-authored-by: Ben Stabler <[email protected]>
Co-authored-by: Leah Flake <[email protected]>
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.

2 participants