-
Notifications
You must be signed in to change notification settings - Fork 53
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 sympy dependency #586
Comments
Based on what I could find, we can use numpy for It might be easier to keep |
As you mentioned, @purva-thakre , I think either I would need to handroll the solution or use another third party package. For now, let's keep the dependency as you suggested. Closing for now. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As
sympy
is a required install but only used for one function, it might be ideal to use another non-sympy approach to reduce the need to requiresympy
as a dependency.toqito/toqito/states/mutually_unbiased_basis.py
Line 4 in 39bad5d
We could then remove the dependency from the
toml
file:toqito/pyproject.toml
Line 48 in 39bad5d
and from the requirements:
toqito/docs/requirements.txt
Line 3 in 39bad5d
The text was updated successfully, but these errors were encountered: