Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

Raahim-Lone
Copy link

@Raahim-Lone Raahim-Lone commented Jun 14, 2025

Adds 'dask.array.simpson(aliassimps`; simpson is used since simps is legacy).

  • Supports both uniform (dx) and irregular (x) grids
  • Handles even-length uniform grids ("avg" — 1/3 rule + 3/8 tail; "first"/"last" — 1/3 + trapezoid)
  • Public names exported in dask/array/__init__.py
  • Unit tests added under dask/array/tests/test_simps.py

Checklist

Please review API/edge-cases; happy to iterate.

@Raahim-Lone Raahim-Lone force-pushed the simpson-impl-2629 branch 2 times, most recently from 8cf3646 to 883d273 Compare June 14, 2025 14:55
Copy link
Contributor

github-actions bot commented Jun 14, 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 19m 16s ⏱️ + 3m 28s
 18 038 tests +1   16 821 ✅ ±0   1 217 💤 + 1  0 ❌ ±0 
161 416 runs  +9  149 300 ✅  - 1  12 116 💤 +10  0 ❌ ±0 

Results for commit c0f9581. ± Comparison against base commit 20d7d4d.

♻️ This comment has been updated with latest results.

@Raahim-Lone Raahim-Lone marked this pull request as ready for review June 14, 2025 18:55
@Raahim-Lone Raahim-Lone changed the title DRAFT: add dask.array.simpson (SciPy integrate.simps) [#2629] Add dask.array.simpson (SciPy integrate.simps) [#2629] Jun 14, 2025
@Raahim-Lone
Copy link
Author

Just checking in on this PR. Please let me know if there’s anything I can do to move it forward/revise it. Appreciate your time, especially during the summer!

@github-actions github-actions bot added the needs attention It's been a while since this was pushed on. Needs attention from the owner or a maintainer. label Oct 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs attention It's been a while since this was pushed on. Needs attention from the owner or a maintainer.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.