Closed qgallouedec closed 1 day ago
The issue here is that np._core
does not exist in older numpy versions (probably < 2.0 but I haven't checked), so this would fail there (as happens in the failing doc build). We would need to do a version check and use either np.core
or np._core
.
good point!
The very first version that includes np._core
is ~1.26.1~ 2.0.0.
What about something like
np_core = np._core if version.parse(np.__version__) >= version.parse("2.0.0") else np.core
alternatively we can do something like
np_core = getattr(np, "_core", np.core)
but I find it less intuitive to know when the condition is met or not, and we'll probably forget to remove this if/else once we've increased the min version of numpy
EDIT: min version is actually 2.0.0
wait it seems like np._core
for np<2.0 is not the same as for np>=2.0. Checking right now
EDIT: version threshold is actually 2.0.0 (not 1.26.1). Editing my above comment
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.
What does this PR do?
We've been getting this warning:
Before submitting
Who can review?
@muellerzr @BenjaminBossan @SunMarc