-
Notifications
You must be signed in to change notification settings - Fork 1
Update scipy to the current latest version to suppress FutureWarning #153
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
Conversation
@nicolaskruchten how can we run the CI file even when the branch is not on |
It's a setting we have to add in CircleCI, I can do it. |
OK I've checked off this setting, so we need a new commit on this PR to kick off the CI. |
@nicolaskruchten Thanks! Pushed an empty commit to trigger the CI. |
The CI fails to build the following pages. These two pages use
|
Updating |
The future warning still remains. https://github.com/plotly/plotly.py-docs/blob/built/2019-07-03-images.html#L423-L426 |
Looks like I misunderstood something. Where can I see the built pages? |
They get uploaded as artifacts in CI I believe. Thanks for looking into this for us :) |
OK so this is working now? the |
Yes this is working |
oh, I found another one. This one seems to be from pandas. https://plot.ly/python/choropleth-maps/#full-county-choropleths
|
One workaround that I came up with is adding the css below to hide div.output_stderr {
display: none;
} |
Thanks for the PR! We should look into that other warning for sure. Really what we should do is have CI grep the HTML output for |
This PR updates
scipy
to the current latest version1.3.1
(release history) to resolve #148.