Skip to content

Update np.accumulate workaround comment#12129

Merged
jrbourbeau merged 1 commit intodask:mainfrom
jacobtomlinson:numpy-comment-accumulate-sig
Nov 3, 2025
Merged

Update np.accumulate workaround comment#12129
jrbourbeau merged 1 commit intodask:mainfrom
jacobtomlinson:numpy-comment-accumulate-sig

Conversation

@jacobtomlinson
Copy link
Copy Markdown
Member

Even though numpy/numpy#30104 is merged we still need to maintain our workaround for numpy/numpy#30095 until our minimum numpy version is higher than 2.3.4.

This PR updates the comment in the workaround to make it clear when the workaround can be removed by some future maintainer.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Nov 3, 2025

Unit Test Results

See test report for an extended history of previous test failures. This is useful for diagnosing flaky tests.

      9 files  ±0        9 suites  ±0   3h 11m 9s ⏱️ + 3m 16s
 18 139 tests ±0   16 924 ✅ ±0   1 215 💤 ±0  0 ❌ ±0 
162 479 runs  ±0  150 384 ✅ ±0  12 095 💤 ±0  0 ❌ ±0 

Results for commit ba74884. ± Comparison against base commit 17a245a.

@jrbourbeau jrbourbeau changed the title Update np.accumulate workaround comment Update np.accumulate workaround comment Nov 3, 2025
@jrbourbeau jrbourbeau merged commit 12112b6 into dask:main Nov 3, 2025
24 of 25 checks passed
@jacobtomlinson jacobtomlinson deleted the numpy-comment-accumulate-sig branch November 4, 2025 10:51
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.

3 participants