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

Cleanups having to do with fidelity of sets of instructions #916

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Cleanups having to do with fidelity of sets of instructions #916

wants to merge 1 commit into from

Commits on Feb 13, 2024

  1. Cleanups having to do with fidelity of sets of instructions 

    Move defs near to use sites; refactor instrs fidelity

Fidelity calculations used by the compressor and by the fidelity
addresser were needlessly constructing logical schedules.

Peeking at the logical schedule fidelity calculations, they too have
been cleaned up and refatored for clarity and performance.

Generally opting for treating fidelity of a circuit as the minimum
fidelity of its constituent gates. Prior to this commit, when a composite
fidelity needed to fall in the range (0, 1] the calculation

(exp (- (sqrt (reduce #'+ fidelities
                 :key (lambda (f) (* (log f) (log f)))))))

was being used. For fidelities f in (0, 1] this produced a
non-negative value f0 that was at most the minimal f.
    @macrologist
    macrologist committed Feb 13, 2024
    Configuration menu
    Copy the full SHA
    3f281cf View commit details
    Browse the repository at this point in the history