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

Add support for NumPy #16

Closed
wants to merge 1 commit into from

Conversation

andreittr
Copy link
Contributor

@andreittr andreittr commented Jul 25, 2023

This change set adds support for the numpy extension in the main Python build.

Part of work on Python 3.10

and thus contains all its commits as well.
Only 5ad2760 is the actual numpy support. (Will rebase once Python PR is merged)

The NumPy microlibrary is a separate PR:

Config.uk Show resolved Hide resolved
@andreittr andreittr force-pushed the ttr/py310-numpy branch 2 times, most recently from e36e2ed to a4a1ad1 Compare August 7, 2023 20:08
@razvand razvand requested a review from StefanJum August 9, 2023 10:00
Copy link
Member

@StefanJum StefanJum left a comment

Choose a reason for hiding this comment

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

All good @andreittr, will add the tag after #15 is merged.

Copy link
Contributor

@mariasfiraiala mariasfiraiala left a comment

Choose a reason for hiding this comment

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

Works well, thanks!

Reviewed-by: Maria Sfiraiala maria.sfiraiala@gmail.com

@StefanJum
Copy link
Member

@andreittr you can rebase this now, #15 is merged 🎉

This change only adds the necessary module registrations and Kconfig
options. Numpy itself is provided by its own unikraft port.

Signed-off-by: Andrei Tatar <andrei@unikraft.io>
Copy link
Member

@StefanJum StefanJum left a comment

Choose a reason for hiding this comment

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

All good, thanks.

Reviewed-by: Stefan Jumarea stefanjumarea02@gmail.com

Copy link
Contributor

@razvand razvand left a comment

Choose a reason for hiding this comment

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

Approved-by: Razvan Deaconescu razvand@unikraft.io

@andreittr andreittr deleted the ttr/py310-numpy branch August 11, 2023 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/merged enhancement New feature or request
Projects
Status: Done!
Development

Successfully merging this pull request may close these issues.

None yet

5 participants