-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
Closed
Labels
CythonIssues with the internal Cython code baseIssues with the internal Cython code basemaintenanceItems related to regular maintenance tasksItems related to regular maintenance taskstaskA straightforward change, verification or fix.A straightforward change, verification or fix.
Milestone
Description
Starting from Cython >=3.0.8, readonly fused typed memoryviews are finally allowed. Meaning we can add const
qualifiers to memoryviews so that readonly arrays do not get rejected. We should audit our usage of memoryviews with fused dtypes and sprinkle const
pretty much everywhere.
#20195 gives a representative example.
EDIT: a good way to go about this is one PR per submodule, using a grep like in #20196 (comment) to identify files to audit.
-
_lib
(no usage) -
cluster
(ENH: array types: add JAX support #20085) -
fftpack
(no usage) -
linalg
(MAINT: linalg: addconst
to Cython function signatures #20837) -
integrate
(no usage) -
interpolate
(MAINT: interpolate: const qualify cython arrays #20250, MAINT: interpolate: addconst
to Cython function signatures #20872) -
io
(no usage) -
ndimage
(no usage) -
optimize
(ENH: optimize: const qualify Cython array arguments #20265) -
signal
(MAINT: signal: addconst
to Cython function signatures #20887) -
sparse
(MAINT: sparse: addconst
to Cython function signatures #20873) -
spatial
(MAINT: spatial: addconst
to Cython function signatures #20874) -
special
(no usage) -
stats
(MAINT: stats: addconst
to Cython function signatures #20891)
rgommers, Kai-Striega and ilayn
Metadata
Metadata
Assignees
Labels
CythonIssues with the internal Cython code baseIssues with the internal Cython code basemaintenanceItems related to regular maintenance tasksItems related to regular maintenance taskstaskA straightforward change, verification or fix.A straightforward change, verification or fix.