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

allow passing python scalars as atol / rtol to isclose #1044

Merged
merged 3 commits into from
Mar 8, 2020

Conversation

keewis
Copy link
Contributor

@keewis keewis commented Mar 8, 2020

This makes isclose match the behaviour of allclose.

  • Executed black -t py36 . && isort -rc . && flake8 with no errors
  • The change is fully covered by automated unit tests

Edit: the diff is slightly bigger because black touched a few files.

@hgrecco
Copy link
Owner

hgrecco commented Mar 8, 2020

bors r+

@bors
Copy link
Contributor

bors bot commented Mar 8, 2020

Build succeeded

@bors bors bot merged commit 7206b5c into hgrecco:master Mar 8, 2020
@keewis keewis deleted the dimensionless-tolerances branch March 8, 2020 23:42
@keewis keewis mentioned this pull request Apr 2, 2020
1 task
bors bot added a commit that referenced this pull request Apr 2, 2020
1068: Run notebooks on docs build r=hgrecco a=keewis

As noted in #1044, the notebooks (`pint-pandas.ipynb` and `numpy.ipynb`) were static, so they would have needed manual reruns. This strips the cell output and updates `requirements_docs.txt` so `nbsphinx` can run the notebook when the documentation is built. I checked it with my own RTD instance to make sure this update does not break building the docs.

Note that since `pint-pandas` has never been released, it is installed from the github repository.

- [x] Executed ``black -t py36 . && isort -rc . && flake8`` with no errors


Co-authored-by: Keewis <keewis@posteo.de>
bors bot added a commit that referenced this pull request Apr 2, 2020
1068: Run notebooks on docs build r=hgrecco a=keewis

As noted in #1044, the notebooks (`pint-pandas.ipynb` and `numpy.ipynb`) were static, so they would have needed manual reruns. This strips the cell output and updates `requirements_docs.txt` so `nbsphinx` can run the notebook when the documentation is built. I checked it with my own RTD instance to make sure this update does not break building the docs.

Note that since `pint-pandas` has never been released, it is installed from the github repository.

- [x] Executed ``black -t py36 . && isort -rc . && flake8`` with no errors


Co-authored-by: Keewis <keewis@posteo.de>
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.

2 participants