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

Use cibuildwheel in Pyodide CI job for NumPy #1

Closed
wants to merge 58 commits into from

Conversation

agriyakhetarpal
Copy link
Owner

jorenham and others added 23 commits July 22, 2024 05:35
This uses `PyObject_GetOptionalAttr` which is backported in capi
compat.  This function significantly speeds up many attribute
lookups and thus dispatching protocols.
We had logic in place to drop (most) metadata, but the change
had a small bug: During saving, we were still using the one with
metadata...

Maybe doesn't quite close it, but big enough of an improvement
for now, I think, so

Closes numpygh-14142
ENH: Use ``PyObject_GetOptionalAttr``
BUG: Do not accidentally store dtype metadata in ``np.save``
When many bins are requested in a small value region,
it may not be possible to create enough distinct bin
edges due to limited numeric precision. Up to now,
`histogram` then returned identical subsequent bin
edges, which would mean a bin width of 0. These bins
could also have counts associated with them.

Instead of returning such unlogical bin distributions,
this PR raises a value error if the calculated bins
do not all have a finite size.

Closes numpy#27142.
These were skipped due to failing, but they are passing again now.

Closes numpygh-26824
CI: Re-enable nightly OpenBLAS test runs
jorenham and others added 26 commits August 11, 2024 07:19
MAINT: use npy_argparse for einsum
This is based on what Matti wrote in numpygh-27105 but also adding it
to the migration guide.

Closes numpygh-27083

Co-authored-by: Matti Picus <[email protected]>
REV: Revert undef I and document it
MAINT: update default NPY_FEATURE_VERSION after dropping py39
BUILD: improve download script
The value was simply hardcoded to the wrong thing in the dynamic
path...
BUG: Fix NPY_RAVEL_AXIS on backwards compatible NumPy 2 builds
…x_imports

TYP: Deprecate calling ``numpy.save`` with ``fix_imports`` (PEP 702)
…e-overrides

TYP: Fix incompatible overrides in the ``numpy._typing._ufunc`` stubs
…ntime

TYP: Simpllified ufunc imports in ``numpy._typing``
Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.26.0 to 3.26.2.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@eb055d7...429e197)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
This will help for those that don't use the mypy plugin.
…-bins-float

TYP: Fixed & improved type hints for ``numpy.histogram2d``
TYP: Sane defaults for the  platform-specific ``NBitBase`` types.
…ithub/codeql-action-3.26.2

MAINT: Bump github/codeql-action from 3.26.0 to 3.26.2
…tions

TYP: Annotate ``numpy._core._type_aliases``.
This commit performs the following changes:

1. cibuildwheel version 2.20.0 is added for building
and testing the Pyodide wheels for NumPy in its workflow
for out-of-tree builds.
2. The workflow sets the CIBW_PLATFORM environment
variable in order to point to Pyodide as the build target.
3. The version of Pyodide used is bumped to the recent
version 0.26.1.
4. The TOML tables are updated to use cibuildwheel's
provided overrides for the test command, the wheels'
repair command, and the setup arguments needed at the
time of building the wheel.

[skip azp] [skip circle] [skip cirrus]
@agriyakhetarpal
Copy link
Owner Author

Closing since this is merged upstream, this PR was opened just for testing purposes.

@agriyakhetarpal agriyakhetarpal deleted the cibuildwheel-pyodide branch August 15, 2024 20:44
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.