Skip to content
Snippets Groups Projects

Update coating parameters for aLIGO and Aplus

Merged Nicholas Demos requested to merge nicholas.demos/pygwinc:ctn into master
1 unresolved thread

Updated coating material properties from MIT coating thermal noise experiment results (https://dcc.ligo.org/LIGO-G2000880) and LMA 2020 paper (https://iopscience.iop.org/article/10.1088/1361-6382/ab77e9)

Merge request reports

Merge request pipeline #134321 passed

Merge request pipeline passed for f8ae2e73

Approved by

Merged by Jameson RollinsJameson Rollins 4 years ago (Jun 16, 2020 10:45pm UTC)

Merge details

  • Changes merged into master with c51f8f48 (commits were squashed).
  • Deleted the source branch.

Pipeline #134327 passed

Pipeline passed for c51f8f48 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Jameson Rollins
  • Jameson Rollins
  • Jameson Rollins
  • Jameson Rollins
  • Jameson Rollins
  • Hey @nicholas.demos thanks so much for the patch. Before we merge can you remove all the comments from the changed values? They're not needed and just end up cluttering the code. Describe the change, including the reference in the git commit log and that should be sufficient. Thanks!

  • Nicholas Demos resolved all threads

    resolved all threads

  • Nicholas Demos added 1 commit

    added 1 commit

    • 26d58ec1 - Remove previous value comments

    Compare with previous version

  • Kevin Kuns approved this merge request

    approved this merge request

  • Nicholas Demos added 1 commit

    added 1 commit

    • f8ae2e73 - Remove previous value comments

    Compare with previous version

  • Kevin Kuns approved this merge request

    approved this merge request

  • Jameson Rollins mentioned in commit c51f8f48

    mentioned in commit c51f8f48

  • Nicholas Demos mentioned in merge request !78 (merged)

    mentioned in merge request !78 (merged)

  • Please register or sign in to reply
    Loading