Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: update numpy recipe in Python examples page #1015

Merged
merged 1 commit into from
Aug 12, 2024

Conversation

rgommers
Copy link
Contributor

@rgommers rgommers commented Aug 11, 2024

A couple of reasons for these changes:

  • the current example for numpy 1.26.4 uses a couple of submodules that no longer exist
  • the NumPy C API export changed a lot, so advertise that (and no longer use pin_subpackage)
  • there's a second entrypoint in numpy 2.0 (numpy-config) that's also nice to add
  • the build/host split may be confusing to the reader, since it contradicts what is explained in the cross compiling docs ("tools that run on the build machine go under build:"); the current setup only makes sense when using crossenv, like conda-forge does.

A couple of reasons for these changes:
- the current example for numpy 1.26.4 uses a couple of submodules that no longer exist
- the NumPy C API export changed a lot, so advertise that (and no longer use `pin_subpackage`)
- there's a second entrypoint in numpy 2.0 (`numpy-config`) that's also nice to add
- the build/host split may be confusing to the reader, since it contradicts what is
  explained in the cross compilinng docs ("tools that run on the build machine
  go under `build:`); the current setup only makes sense when using `crossenv`, like
  conda-forge does.
Copy link
Contributor

@Hofer-Julian Hofer-Julian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot @rgommers!

I tested the recipe locally and it works as expected :)

@Hofer-Julian Hofer-Julian enabled auto-merge (squash) August 12, 2024 08:18
@Hofer-Julian Hofer-Julian merged commit bdcd89d into prefix-dev:main Aug 12, 2024
16 checks passed
@rgommers rgommers deleted the update-numpy-example branch August 12, 2024 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants