CQCL / qnexus

Quantinuum Nexus python client.
https://docs.quantinuum.com/nexus/
Apache License 2.0
6 stars 1 forks source link

fix: update websockets dependency to be compatible with pytket-quantinuum #109

Closed quantinuum-richard-morrison closed 3 weeks ago

quantinuum-richard-morrison commented 3 weeks ago

pytket-quantinuum updated in version 0.38.1 (October 2024) See https://github.com/CQCL/pytket-quantinuum/commit/6828486190c3b335756412774468649a8cd02e5d

Our only dependency is qnexus itself, and we only use the imported code in a very "vanilla" fashion, so I've kept >11 for maximum compatibility with other libraries that might be installed alongside qnexus.

What I'm not sure about is whether to just remove the upper bound on the constraint and go with an open-ended >11.0 (similar to pytket-quantinuum which now has >=13.1). For correctness I've left it in, because websockets might release a breaking change in their version 14.x. But we could discuss.

vanyae-cqc commented 3 weeks ago

the integration test failure isn't this PR, I fix the issue here: https://github.com/CQCL/qnexus/pull/92