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.

Sign up for GitHub

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

Jump to bottom

Generalize concept of removing OpTypes #624

Merged
merged 10 commits into from
Jun 5, 2024
Merged

Generalize concept of removing OpTypes #624

merged 10 commits into from
Jun 5, 2024

Commits on Jun 4, 2024

  1. ✨ Generalized the concept of removing OpTypes in a circuit.

    @lsschmid
    lsschmid committed Jun 4, 2024
    Configuration menu
    Copy the full SHA
    48fc440 View commit details
    Browse the repository at this point in the history

  2. ✅ Added tests for removing Operations.

    @lsschmid
    lsschmid committed Jun 4, 2024
    Configuration menu
    Copy the full SHA
    e527dee View commit details
    Browse the repository at this point in the history

  3. Merge branch 'refs/heads/main' into na-zx-hybrid-synthesis

    @lsschmid
    lsschmid committed Jun 4, 2024
    Configuration menu
    Copy the full SHA
    b5a58ff View commit details
    Browse the repository at this point in the history

  4. 🚨 Fixed linter warnings

    @lsschmid
    lsschmid committed Jun 4, 2024
    Configuration menu
    Copy the full SHA
    2f9be2a View commit details
    Browse the repository at this point in the history

  5. 🐛 Fixed removal of multi-cotrolled Identities

    @lsschmid
    lsschmid committed Jun 4, 2024
    Configuration menu
    Copy the full SHA
    d64c630 View commit details
    Browse the repository at this point in the history

Commits on Jun 5, 2024

  1. ⚡ use unordered_set for faster membership test 

    Signed-off-by: burgholzer <[email protected]>
    @burgholzer
    burgholzer committed Jun 5, 2024
    Configuration menu
    Copy the full SHA
    f52e8c7 View commit details
    Browse the repository at this point in the history

  2. ✏️ remove comment 

    It doesn't work for nested compound operations, but neither did the original implementation.

Signed-off-by: burgholzer <[email protected]>
    @burgholzer
    burgholzer committed Jun 5, 2024
    Configuration menu
    Copy the full SHA
    90b494b View commit details
    Browse the repository at this point in the history

  3. ♻️ use the new Operation::getNqubits() method 

    Signed-off-by: burgholzer <[email protected]>
    @burgholzer
    burgholzer committed Jun 5, 2024
    Configuration menu
    Copy the full SHA
    728457d View commit details
    Browse the repository at this point in the history

  4. ✅ add an explicit compound operation test 

    Signed-off-by: burgholzer <[email protected]>
    @burgholzer
    burgholzer committed Jun 5, 2024
    Configuration menu
    Copy the full SHA
    23d51c0 View commit details
    Browse the repository at this point in the history

  5. ⚗️ adjust test 

    Signed-off-by: burgholzer <[email protected]>
    @burgholzer
    burgholzer committed Jun 5, 2024
    Configuration menu
    Copy the full SHA
    992da1a View commit details
    Browse the repository at this point in the history