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

Add CI job for oldest supported dependencies #461

Merged
merged 17 commits into from
Feb 25, 2019

Conversation

jrbourbeau
Copy link
Member

This PR adds a CircleCI job to run our tests against the oldest supported version of our dependencies. Closes #456.

@stsievert
Copy link
Member

Should the minimum required version of dask by 1.0.0 instead of 0.18.2? #455 (comment) and #458 (comment)

@jrbourbeau
Copy link
Member Author

To start, I just wanted to match the minimum versions specified in setup.py (i.e. dask 0.18.2). But bumping the minimum supported version of dask to 1.0.0 would certainly be okay with me.

@jrbourbeau
Copy link
Member Author

Several of the test failures here (ref https://circleci.com/gh/dask/dask-ml/3484?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link) are related to newer features in our dependencies (e.g. dd.DataFrame.to_dask_array was included in the dask 0.19.0 release, the dtype argument for pandas.get_dummies was included in the pandas 0.23.0 release, etc.)

What minimum version numbers for our dependencies do we want to support? As @stsievert pointed out, there has been mention of dask 1.0.0 as a lower bound for dask. @TomAugspurger any thoughts on a minimum version for pandas? setup.py currently has pandas 0.21.0 as the minimum (which was released in October 2017).

@TomAugspurger
Copy link
Member

TomAugspurger commented Feb 16, 2019 via email

.circleci/config.yml Outdated Show resolved Hide resolved
@TomAugspurger
Copy link
Member

Hmm apparently the flake8 pulled from conda-forge is checking things in type comments? I'd recommend just removing those for now.

@TomAugspurger
Copy link
Member

Unrelated failures should be fixed, if you could merge master and push again.

@jrbourbeau
Copy link
Member Author

I think this should be ready for review now (cc @TomAugspurger). Summary of changes made in this PR:

  • Add CircleCI build to test against earliest supported versions of our dependencies
  • Update minimum versions in setup.py
  • Update a few tests that were failing with older dependency versions
  • Unpin NumPy version in ci/environment-2.7.yml (I suspect this could have been done in TST: Try numba RC #363)
  • Remove a couple of Python 2 style type annotations that newer versions of flake8 didn't like

@TomAugspurger
Copy link
Member

Thanks.

@TomAugspurger TomAugspurger merged commit beca674 into dask:master Feb 25, 2019
@jrbourbeau jrbourbeau deleted the update_ci branch February 25, 2019 15:10
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.

3 participants