data-apis/array-api-extra

ENH: add `sinc`

lucascolley opened this issue · 4 comments

Does everyone use the normalized definition of sinc?

The array libraries we test with in SciPy all follow NumPy in using the normalized definition.

We could add a normalized=True kwarg if the unnormalized use-case ever arose?

Not unless that is already common. It doesn't look like NumPy has it. You can always divide by pi if you want an unnormalized sinc.

My main concern is whether we would be deciding on a convention here, or whether that convention has effectively already been decided by the community.

My main concern is whether we would be deciding on a convention here, or whether that convention has effectively already been decided by the community.

My two cents is that this library is not trying to weigh in on conventions at all, it is just taking what is used out in the wild and making that accessible in the array-agnostic context. If multiple conventions are used for some function name we should support all common use cases. If/when an API is considered for the standard is when there is a decision on convention.

It sounds like @rgommers would roughly agree from #10 (comment).