Closed JamesKunstle closed 4 months ago
I've unpinned all the versions to catch everything up, but I'm going to repin them after this PR. I'm going to try to use pipenv but if that still sucks I'll check out Poetry
@cdolfi 1) re-pinned dependencies using modern Python app dependency management best-practice 2) removed comments you noted, ty for finding those 3) I think the commits I made are discrete changes and shouldn't be squashed.
@JamesKunstle Sweet, can you bring the branch back up to date and ill merge! Not too sure how it got out of date but since it is package stuff i want to make sure we are thorough
@cdolfi should be up to date
@JamesKunstle Sorry should have done the merge in the opposite order. Could you bring it up to date and squash any unnecessary commits?
@cdolfi squashed commits, rebased onto current dev, should be good to go
had to make small pandas API changes since we're updating like 6 versions.