Skip to content

Conversation

@bicarlsen
Copy link
Contributor

The link at https://docs.scipy.org/doc/scipy/reference/tutorial/interpolate.html pointing to the SciPy Interpolation tutorial is broken. An alternative tutorial was not found in the new documentation.

  • closes #xxxx (Replace xxxx with the Github issue number)
  • Tests added and passed if fixing a bug or adding a new feature
  • All code checks passed.
  • Added an entry in the latest doc/source/whatsnew/vX.X.X.rst file if fixing a bug or adding a new feature.

The link at https://docs.scipy.org/doc/scipy/reference/tutorial/interpolate.html pointing to the SciPy Interpolation tutorial is broken. An alternative tutorial was not found in the new documentation.
@pep8speaks
Copy link

pep8speaks commented Feb 14, 2022

Hello @bicarlsen! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2022-02-15 07:47:49 UTC

@jreback jreback added the Docs label Feb 15, 2022
@jreback
Copy link
Contributor

jreback commented Feb 15, 2022

/precommit

@jreback
Copy link
Contributor

jreback commented Feb 15, 2022

https://github.com/pandas-dev/pandas/runs/5191233428?check_suite_focus=true

is failing i think you need to remove a line

@mroeschke mroeschke added this to the 1.5 milestone Feb 15, 2022
@mroeschke mroeschke merged commit c4baf5d into pandas-dev:main Feb 15, 2022
@mroeschke
Copy link
Member

Thanks @bicarlsen

yehoshuadimarsky pushed a commit to yehoshuadimarsky/pandas that referenced this pull request Jul 13, 2022
* Removed broken link from NDFrame#interpolate docs.

The link at https://docs.scipy.org/doc/scipy/reference/tutorial/interpolate.html pointing to the SciPy Interpolation tutorial is broken. An alternative tutorial was not found in the new documentation.

* Fixed whitespace formatting.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants