Replace x.ptp() with np.ptp(x)

joezuntz commented 2 months ago

Numpy 2.0 removes the array method .ptp() in favour of the function np.ptp. This replaces the handful of calls in the code.

segasai commented 2 months ago

Thanks @joezuntz ! I wasn't aware of the .ptp deprecation. I'll merge this after tests run

coveralls commented 2 months ago

Pull Request Test Coverage Report for Build 9611131905

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

2 of 2 (100.0%) changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 91.657%

Totals
Change from base Build 7180723466:	-0.02%
Covered Lines:	4142
Relevant Lines:	4519

💛 - Coveralls

segasai commented 2 months ago

The new version of dynesty on pypi contains this fix

joshspeagle / dynesty