-
Notifications
You must be signed in to change notification settings - Fork 0
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
Apply QC to leak-corrected traces. Fix subtraction plots. #24
Conversation
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.
Some unnecessary lines/syntax added
@@ -914,12 +910,6 @@ def run_qc_for_protocol(readname, savename, time_strs, args): | |||
before_currents[sweep, :] = before_raw | |||
after_currents[sweep, :] = after_raw | |||
|
|||
R_leftover = np.full(before_currents.shape[0], np.nan) |
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.
This looks important, was it duplicated somewhere else?
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.
Yes, it was originally (and still is) in `extract_protocol', meaning we get an R_leftover value for every single protocol in the dataset and not just the staircase protocol.
Description
Related to #11
Apply the hERG QC to leak-corrected traces instead of raw traces (as in 10.1016/j.bpj.2019.07.029)
Additionally, fix subtraction plots which had incorrect leak-subtraction. Modify the notation in the subtraction plots so that it matches our in-progress publications.
Types of changes
Testing
Documentation checklist