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

Move pytket requirement to extra #128

Closed
wants to merge 5 commits into from
Closed

Move pytket requirement to extra #128

wants to merge 5 commits into from

Conversation

toumix
Copy link
Contributor

@toumix toumix commented Dec 11, 2023

Pytket is not needed when using lambeq as a parser.

@dimkart
Copy link
Contributor

dimkart commented Apr 11, 2024

For this one to make more sense, we need first to decouple Tket from our diagram manipulation and make Tket only relevant to TketModel, as it should be. This is already work in progress internally, and it will be probably included in Release 0.5.0.

@dimkart
Copy link
Contributor

dimkart commented May 8, 2024

This would be now closed (see above comment).

@dimkart dimkart closed this May 8, 2024
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.

2 participants