-
Notifications
You must be signed in to change notification settings - Fork 3
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
doc: Update readme #121
doc: Update readme #121
Conversation
2c2433a
to
f12c88b
Compare
README.md
Outdated
It allows you to write high-level hybrid quantum programs with classical control flow and mid-circuit measurements using Pythonic syntax: | ||
|
||
```python | ||
from guppylang import guppy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Strictly speaking, we also need to import Qubit
or do from __future__ import annotations
.
Do we want to add that here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add import Qubit
README.md
Outdated
It allows you to write high-level hybrid quantum programs with classical control flow and mid-circuit measurements using Pythonic syntax: | ||
|
||
```python | ||
from guppylang import guppy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add import Qubit
README.md
Outdated
``` | ||
|
||
which will create a directory `./guppy-exports` populated with hugr modules serialised in msgpack. | ||
|
||
## Packaging | ||
### Packaging |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think get rid of packaging, if we're on pypi there should only be one place to call this (on a release actions workflow ideally)
Closes #120