-
Notifications
You must be signed in to change notification settings - Fork 1
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
Run7/step08 #13
Run7/step08 #13
Conversation
Will need new path for |
Updated results from from
|
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.
As noted, please update FILTERCONFIG
entries and remove Run7/calib.cfg
, assuming it's redundant.
96f1b46
to
88efd9b
Compare
Adding step08 in line with BOT EO reverification plan Removing step02 from git commits Reordering flat acquisitions to be linear in flux level Removing dark interrupt Cleaning up files Removing duplicate acquisitions in B and D protocol Removing calib.cfg, removing DARKINTERRUPT call Adding revised FILTERPATH path
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.
The [FLAT]
acquisition sequences are all ordered by signal level, but their orders should be randomized.
From Aaron: Before I forgot, following up our discussion of this for B sequences - one solution there is to make these monotonically increasing instead of randomized. The randomization is to help separate out other effects (ie temp change) but we mostly understand that now From me: Ok, I can make that change with flat acquisitions, reordering the flat acquisitions in ascending flux levels From Aaron: Yes. Suggest that we keep the same config for the inital B sequence run as Run 6, to be able to compare the results as similarly as possible. But then you could make this change for subsequence B protocols. |
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.
The discussion with Aaron makes sense, so lgtm.
File structure determined from confluence page
Output from cfg_interp_Run7.py, columns correspond to time in minutes, number of acquisitions, total image volume in TB