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.

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

M2 coating run in July 2019 #12

Merged
merged 1 commit into from
Dec 4, 2019
Merged

M2 coating run in July 2019 #12

merged 1 commit into from
Dec 4, 2019

Conversation

bxin
Copy link
Contributor

@bxin bxin commented Oct 3, 2019

This update includes the M2 reflectivity curve based on witness sample measurements from M2 coating run in July 2019
The M2 curve is getting a bit better than the June 2019 update.
See the latest m5 values in https://github.com/lsst-pst/syseng_throughputs/blob/update/M2Coating1907/documentation/Confirm%20Updates.ipynb

Copy link
Contributor

@rhiannonlynne rhiannonlynne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am curious though - why is there a difference between the June and July sample? Was the coating method changed?

@bxin
Copy link
Contributor Author

bxin commented Oct 4, 2019

Here is an answer from Tomi - "the reflectivity improvement is basically in the coating process, a better process gas tuning, power, mask shape, rotation speed, etc. All the coating related parameters."

@rhiannonlynne
Copy link
Contributor

@ivezic or @cclaver or @sandrinethomas -- can we get the additional final reviewer approval?

Copy link
Contributor

@cclaver cclaver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated M2 protected Ag reflectivity matches measured values post M2 coating in Chile.

@cclaver cclaver merged commit f01b58f into master Dec 4, 2019
@rhiannonlynne
Copy link
Contributor

Deleting branch update/M2Coating1907

@rhiannonlynne rhiannonlynne deleted the update/M2Coating1907 branch December 4, 2019 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants