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

fix: should not list as a dependency of itself. #282

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

ArcticLampyrid
Copy link

No description provided.

@yannick-couzinie
Copy link

Why not just remove torchquantum from requirements.txt?

@ArcticLampyrid
Copy link
Author

ArcticLampyrid commented Nov 14, 2024

Why not just remove torchquantum from requirements.txt?

I'm not sure why it's in the original file. If possible, I agree to remove it.

@01110011011101010110010001101111
Copy link
Collaborator

Yes, I think you can just remove it from the file. You can do that then I’ll merge it in!

I think the reason it’s here is some weird issue with the docs needing the library itself as a dependency for some strange reason. Since the docs requirements and library requirements are now broken into two files, just removing tq from the requirements would be better!

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.

3 participants