-
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/step14 #24
Run7/step14 #24
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
# 20 15sec DARK images taken with room lights off and Shutter CLOSED | ||
# Before executing this config, make sure shutter is closed. The SHUTTER=None keyword | ||
[ACQUIRE] | ||
dark | ||
[DARK] | ||
BCOUNT=0 | ||
dark=15.0 20 # integration time and image count for dark set | ||
SHUTTER=None | ||
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
# 20 15sec DARK images taken with room lights off and Shutter OPEN | ||
[ACQUIRE] | ||
dark | ||
[DARK] | ||
dark=15.0 20 # integration time and image count for dark set | ||
SHUTTER=OPEN | ||
Comment on lines
+5
to
+6
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Based on the parsing of There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm just asking a question, if that's ok. If the order doesn't matter, that's fine. I didn't write this code, but there are cases where the order does matter, e.g., in the flat pair acquisitions, so I wanted to ask to be sure. Please ask someone else to review your changes in the future. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thank you for asking a question, and I apologize for not clarifying. I am not trying to be defensive or combative, and I understand that it reads that way. I apologize for my poor choice of words. I was asking if I was misunderstanding because we have been fairly arbitrary with the ordering of keywords, and I was curious if the order did impact things. Again, I apologize for being curt. That is not my intention. |
||
BCOUNT=0 |
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.
What does
SHUTTER=None
mean? I don't see it used anywhere else in this repo. I do seein
Run6/step13/opsimdark.cfg
. Based that config, I would have expected the entries here to bewith the
SHUTTER = CLOSE
entry before thedark=15.0 20
entry.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.
Explanation on the shutter entries is here.
There is no
SHUTTER = CLOSE
option supported. That is why I specify the comment at the head of the config fileThere 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.
Here is your comment:
I don't know what "The SHUTTER=None keyword" is supposed to mean. That's why I asked.