Re: Deprecate `scipy.integrate.romberg`/`scipy.integrate.quadrature`?
Ralf Gommers <[email protected]> Wed, 29 Nov 2023 10:14:37 +0100
| Newsgroups | gmane.comp.python.scientific.devel |
|---|---|
| Message-ID | <CABL7CQh3svnQu3G=t=jWboOFoVaUEcRdXVSnXEAR9b4r9mS1XA@mail.gmail.com> |
On Mon, Nov 27, 2023 at 11:42 PM Matt Haberland <[email protected]> wrote: > Hi team, > > Thanks for the feedback on deprecating these functions. There was support > for deprecating them, but also some concerns, so the plan has changed to > the following: > > - Deprecate `scipy.integrate.quadrature` (so we can reclaim the name > for a more general-purpose function in the future). > > This is something we try to never do. There are very few examples of us deprecating and then removing a name, and then reintroducing it with different behavior. There has to be an exceptionally good reason to do so, and I don't see one in the PR discussion (it's quite long, so I may have missed it). I can't even think of a potential reason, because the `quadrature` name isn't special enough that you can't pick a different name for whatever you want/need here. Cheers, Ralf > - Introduce `scipy.integrate.gauss_quad` as an alias for > `scipy.integrate.quadrature`, and point users to `gauss_quad` in the > `quadrature` deprecation messages. > - Mark `scipy.integrate.gauss_quad` and `scipy.integrate.romberg` as > legacy functions. > > We welcome additional feedback in gh-19510 ( > https://github.com/scipy/scipy/pull/19510). > Thanks, > Matt > > On Sat, Nov 11, 2023 at 4:10 PM Matt Haberland <[email protected]> > wrote: > >> Hi Team, >> >> `scipy.integrate.romberg` and `scipy.integrate.quadrature` do not >> reliably hit their target accuracies, and when they do, they typically >> require more function evaluations and take more wall clock time than >> `scipy.integrate.quad` (all with default settings). This is documented in >> https://github.com/scipy/scipy/issues/18574#issuecomment-1703320598, >> which compares the accuracies, function call counts, and wall clock times >> on a benchmark set of ~800 integrals with known analytical solutions. >> >> gh-19510 (https://github.com/scipy/scipy/pull/19510) proposes to >> deprecate these functions, suggesting `scipy.integrate.quad` as a drop-in >> replacement. Please let us know your thoughts either here or in gh-19510! >> >> Thanks, >> Matt >> > _______________________________________________ > SciPy-Dev mailing list -- [email protected] > To unsubscribe send an email to [email protected] > https://mail.python.org/mailman3/lists/scipy-dev.python.org/ > Member address: [email protected] > _______________________________________________ SciPy-Dev mailing list -- [email protected] To unsubscribe send an email to [email protected] https://mail.python.org/mailman3/lists/scipy-dev.python.org/ Member address: [email protected]