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

handle T2 > 2*T1 and clarify terminology #1994

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

handle T2 > 2*T1 and clarify terminology #1994

wants to merge 5 commits into from

Commits on Nov 8, 2023

  1. handle T2 > 2*T1 and clarify terminology 

    Since T1 and T2 are typically measured at different times, it's not that uncommon for the most recently reported T2 to exceed the most recently reported T1. Rather than treating this as an error, it seems better to just cap T2 at the physical limit and emit a warning.

While at it, I replaced some terms to clarify that T2 is the overall coherence time, not the relaxation time (T1) or the dephasing time (T_phi).
    @dtmcclure
    dtmcclure authored Nov 8, 2023
    Configuration menu
    Copy the full SHA
    057156f View commit details
    Browse the repository at this point in the history

  2. apply black and use lazy log formatting

    @dtmcclure
    dtmcclure authored Nov 8, 2023
    Configuration menu
    Copy the full SHA
    21b6c52 View commit details
    Browse the repository at this point in the history

  3. one more blank line

    @dtmcclure
    dtmcclure authored Nov 8, 2023
    Configuration menu
    Copy the full SHA
    0200adc View commit details
    Browse the repository at this point in the history

  4. update test for new T2 > 2*T1 behavior

    @dtmcclure
    dtmcclure authored Nov 8, 2023
    Configuration menu
    Copy the full SHA
    0f16fb3 View commit details
    Browse the repository at this point in the history

  5. improve comments in test function

    @dtmcclure
    dtmcclure authored Nov 8, 2023
    Configuration menu
    Copy the full SHA
    c7c55d7 View commit details
    Browse the repository at this point in the history