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.

Sign up for GitHub

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

Jump to bottom

Remove numpy backend #1149

Open
dpanici opened this issue Jul 24, 2024 · 0 comments · May be fixed by #1150
Open

Remove numpy backend #1149

dpanici opened this issue Jul 24, 2024 · 0 comments · May be fixed by #1150
Assignees
@YigitElma
Labels
P1 Lowest Priority, will get to eventually

Comments

@dpanici
Copy link
Collaborator

dpanici commented Jul 24, 2024

DESC requires JAX now to run (due to interpax dependency), so the numpy backend is not really used or relevant anymore. We should remove it then to reduce amount of code to maintain (and it is not used at all)
@dpanici dpanici added the low priority Nice to have, but not needed right away label Jul 24, 2024
@YigitElma YigitElma linked a pull request Jul 26, 2024 that will close this issue
Open
@YigitElma YigitElma self-assigned this Jul 26, 2024
@dpanici dpanici added P1 Lowest Priority, will get to eventually and removed low priority Nice to have, but not needed right away labels Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@YigitElma YigitElma

Labels
P1 Lowest Priority, will get to eventually
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove numpy backend PlasmaControl/DESC
2 participants
@dpanici @YigitElma