-
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/step11 #14
Run7/step11 #14
Conversation
Will need new path for |
Updated parser results
These do not make a lot of sense, it appears to only be parsing the first entry of each PTC flat config, instead of the full list. I'm guessing this has something to do with the re-ordered flat flux lists, but wonder if it is only for the parser time estimating parser, or would extend to an actual acquisition |
Interestingly, other reordered PTC flat field lists are being parsed correctly |
495e50d
to
b9b07b6
Compare
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.
All of the flat acquisition sequences should be randomized.
One comment from @aaronroodman about the ordering of flat acquisitions:
I can add the dark overhead, but does this sound reasonable @jchiang87 ? |
sure, sounds fine. |
Add step11 in line with BOT EO reverification plan Modifying PTC files for only flat acquisition, and no bias acquisition at the start of .cfg sequence Add ordering to flat acquisition Turn off dark interleaving Cleaning up files Removing duplicates in PTC flats Removing calib.cfg, removing DARKINTERRUPT call Adding revised FILTERCONFIG path Removing a calib file that came along for the ride
Changes made |
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.
It looks like the dark
acquisitions are being taken after the flat
acquisitions for a number of these .cfg files.
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.
One more file to fix, otherwise looks fine.
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