Skip to content

CI: Numpy compat #34321

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

Merged
merged 1 commit into from
May 22, 2020
Merged

CI: Numpy compat #34321

merged 1 commit into from
May 22, 2020

Conversation

alimcmaster1
Copy link
Member

@alimcmaster1 alimcmaster1 commented May 22, 2020

Likely related to: #32510.

Seems doc builds have started using:
numpy 1.17.5 py38h95a1406_0 conda-forge
https://github.com/pandas-dev/pandas/pull/34317/checks?check_run_id=700337761

(Previous pipelines where using 1.18)

In numpy 1.17.5 BitGenerator path is:
np.random.bit_generator.BitGenerator

In numpy 1.18.0 path is
np.random.BitGenerator

@alimcmaster1 alimcmaster1 added the CI Continuous Integration label May 22, 2020
@alimcmaster1 alimcmaster1 added this to the 1.1 milestone May 22, 2020
@TomAugspurger
Copy link
Contributor

I don't really follow the issue / fix here: Did the changes go like this?

  • numpy 1.17.4: np.random.BitGenerator
  • numpy 1.17.5: np.random.bit_generator.BitGenerator
  • numpy 1.18: np.random.BitGenerator

Or am I misunderstanding?

@alimcmaster1
Copy link
Member Author

alimcmaster1 commented May 22, 2020

  • numpy 1.17.4: np.random.BitGenerator

For 1.17.4

I see np.random.bit_generator.BitGenerator ?

-> hence I thought the easiest fix is just to require numpy 1.18 ?

Or am i missing something?

Ill also will need to update the test changes made in https://github.com/pandas-dev/pandas/pull/32510/files

@TomAugspurger - mind doing the azp run command -> I don't have permissions.

@jreback
Copy link
Contributor

jreback commented May 22, 2020

@alimcmaster1 restarted the failing job on travis (was a network error i think). but this looks good to go.

@jreback jreback merged commit 29d8ce5 into pandas-dev:master May 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants