Move Package out of hugr-cli
#1530
Assignees
Labels
bug
Something isn't working
Comments
github-merge-queue bot
pushed a commit
to CQCL/tket2
that referenced
this issue
Sep 30, 2024
This is a small bundle of changes aimed at fixing the example notebooks. - Use packages when building hugrs. This includes parsing guppy packages from rust, as well as returning packages from the circuit builder. - This required adding exports for the tket2 extensions from python. - We temporarily add a py dependency on `hugr-cli`, since packages are currently defined there. See CQCL/hugr#1530 The `guppy -> pytket` pipeline is still broken, as guppy now generates many more function calls and nested control flow primitives, which we do not currently support. drive-by: Make `crate::serialize::guppy::find_function` public. Closes #621.
github-merge-queue bot
pushed a commit
that referenced
this issue
Oct 17, 2024
Moves the definition of packages out of `hugr-cli`. (We leave a re-export there to avoid breaking the API, it should be removed on the next breaking release). Adds some helper functions to load and validate the package. Closes #1530 drive-by: Replace `hugr-core` dependency on `hugr-cli` by `hugr`. There's no need to access the internals there.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Packageis a core type that should be exported from the core (at least untilhugr-modelis created).The current state of things requires tket2 to depend on
hugr-clifor the sole reason of using that type.The text was updated successfully, but these errors were encountered: