-
Notifications
You must be signed in to change notification settings - Fork 47
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
add rv3 to changelog #263
add rv3 to changelog #263
Conversation
the docs check is removed because of duplication in the build_and_test workflow |
…ket into docs/add-rv3-to-changelog
pytket/docs/changelog.rst
Outdated
The new method will use a list of user given method, each of them suitable only | ||
for a specific set of sub circuits. Users can add there own methods if they want to. | ||
Take a look on the `mapping_example <https://github.com/CQCL/pytket/blob/main/examples/mapping_example.ipynb>`_ | ||
notebook to find out more. All compiler passes in pytket are updated to use the new method. |
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.
Let's leave out the reference to the notebook. (It isn't live yet,)
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 have removed it
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.
see 7114822
pytket/docs/changelog.rst
Outdated
notebook to find out more. All compiler passes in pytket are updated to use the new method. | ||
The methods already given by pytket are ``LexiRouteRoutingMethod``, | ||
``LexiLabellingMethod``, ``MultiGateReorderRoutingMethod``, | ||
``BoxDecompositionRoutingMethod``, ``AASRouteRoutingMethod`` and ``AASLabellingMethod``. |
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.
Leave out BoxDecompositionMethod
(not merged yet).
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 have removed it
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.
see 7114822
Co-authored-by: Alec Edgington <[email protected]>
Co-authored-by: Alec Edgington <[email protected]>
Co-authored-by: Alec Edgington <[email protected]>
No description provided.