TST: Add direct tests for _convolveNd_c Cython extension#19455
TST: Add direct tests for _convolveNd_c Cython extension#19455dhruv1955 wants to merge 3 commits intoastropy:mainfrom
Conversation
|
Thank you for your contribution to Astropy! 🌌 This checklist is meant to remind the package maintainers who will review this pull request of some common things to look for.
|
|
@neutrinoceros @astrofrog @nstarman - tagging as this is part of my GSoC 2026 proposal. Same approach as #19410 (time) and #19407 (cosmology) - direct pytest-based tests calling the Cython extension interface, no standalone C files. |
|
Note is that this is "competing" with #19470 |
|
@neutrinoceros Thanks for the context. That works for me - happy to wait and harmonize once #19470 lands if needed. Let me know if there's anything you'd like me to adjust in this PR in the meantime. |
Part of GSoC 2026 proposal: Hardening Astropy's Core Stability. Follows the same approach as #19410 (time) and #19407 (cosmology).
This PR adds direct tests for the
_convolveNd_cCython extension inastropy/convolution/_convolve.pyx, bypassing the publicconvolve()API to test the compiled interface directly.Tests cover:
embed_result_within_padded_region=Falsewith a pre-padded array