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

MAINT: SciPy 1.10.1 backports round 2 #18001

Merged

Conversation

tylerjereddy
Copy link
Contributor

@tylerjereddy tylerjereddy commented Feb 18, 2023

After manually scanning through bug fixes that were merged after 1.10.0, and adding a few more to the list, the backports listed below have been included here.

I updated .mailmap as well as I could from a quick glance at folks' GitHub profiles.

Although I think we dealt with the 32-bit Debian testing issues with SciPy, I wasn't able to provide the level of supported I wanted to Fedora in this regard just yet (see: #17912 (comment)). I suggest we move forward with 1.10.1 nonetheless, at around 30 backports merged now, and perhaps I'll get better at using the Fedora tooling to better support/reproduce in the future.

Includes these backports:

  1. BUG: Correct intermediate overflow in KS one asymptotic in SciPy.stats #17783
  2. BUG: signal: fix check_malloc extern declaration type #17790
  3. [sparse.csgraph] Fix a bug in dijkstra and johnson algorithm #17800
  4. BUG: Fix handling of powm1 overflow errors #17855
  5. BUG: fix Johnson's algorithm #17863
  6. BUG: fix powm1 overflow handling #17872
  7. BUG: Use raw strings for paths in __config__.py.in #17936
  8. DOC: update link of the logo in the readme #17984
  9. BUG: at least one entry from trial should be used in exponential crossover #17997

TODO:

  • make sure the full CI matrix is passing
  • make sure the wheels build properly after that (a bit more convenient to catch issues before I start release process proper)

aherbert and others added 10 commits February 18, 2023 15:17
The definition of check_malloc takes a size_t argument.
Fixes a lto-type-mistatch warning.
Related to scipy#16098
…ipy#17800)

* MAINT: Fix wrong 'directed' being passed in the tests for shortest path algorithms
* BUG: scipy.sparse.csgraph: Fix a bug in FibonacciHeap used for the dijkstra function
* TST: PR 17800 revisions to pin down exact test cases

---------

Co-authored-by: Tyler Reddy <tyler.je.reddy@gmail.com>
Co-authored-by: CJ Carey <perimosocordiae@gmail.com>
Co-authored-by: Kai Striega <kaistriega@gmail.com>
Remove unnecessary reset of dist_array during Bellman-Ford
weight updates.
Add unit test for negative weighted shortest paths.
See scipy#14980
* `\U` leads to strings being treated as unicode, hence we escape it
  with raw strings
* Changed bool parameters real bools instead of string `"True"` and
  `"False"`

[wheel build]
@tylerjereddy tylerjereddy added the maintenance Items related to regular maintenance tasks label Feb 18, 2023
@tylerjereddy tylerjereddy added this to the 1.10.1 milestone Feb 18, 2023
@tylerjereddy
Copy link
Contributor Author

Main CI is clean -- the linter failure about a single line being too long is irrelevant, especially on a maintenance branch.

Proceeding with wheel build test next.

* test wheel builds in scipygh-18001

[wheel build]
@tylerjereddy
Copy link
Contributor Author

tylerjereddy commented Feb 19, 2023

CI is good with wheel builds as well, apart from the usual linting complaint, so in it goes and probably onward with the 1.10.1 release process.

Perhaps for 1.10.2 the Fedora/RHEL testing issues can be addressed.

@tylerjereddy tylerjereddy merged commit 5f022bd into scipy:maintenance/1.10.x Feb 19, 2023
@tylerjereddy tylerjereddy deleted the treddy_scipy_1_10_1_rd2 branch February 19, 2023 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Items related to regular maintenance tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants