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

Remove usage of numpy and use builtin math when possible #8408

Open
silvanocerza opened this issue Sep 25, 2024 · 2 comments · May be fixed by #8418
Open

Remove usage of numpy and use builtin math when possible #8408

silvanocerza opened this issue Sep 25, 2024 · 2 comments · May be fixed by #8418
Labels
P3 Low priority, leave it in the backlog type:enhancement

Comments

@silvanocerza
Copy link
Contributor

Numpy is an heavy library to import and we overuse it all around the codebase.

We should remove it and fallback to use builtin math whenever possible to speed up import speed.

@silvanocerza silvanocerza added type:enhancement P3 Low priority, leave it in the backlog labels Sep 25, 2024
@vaishjah3
Copy link

Sorry for the naive question, but I am really new to this. So, do you expect numpy functions to be converted to math functions in the code wherever possible?

@silvanocerza
Copy link
Contributor Author

@vaishjah3 that's the idea. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P3 Low priority, leave it in the backlog type:enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants