Fix an error in the coating brownian noise calculation

A discrepancy was found by Gabriele Vajente between the gwinc CTN calculation and several other codes, especially in the case of large Poisson ratio in the coating.

See: https://git.ligo.org/gabriele-vajente/thermal-noise-models

This commit resolves the discrepancy by fixing an apparent error in the Yam et al calculation that was implemented in gwinc.

added noise label

Attaching a jupyter notebook that reproduces the calculation.

This change moves the aLIGO CTN curve a little. But it's possible that some material parameters may need updating, such that the curve stays where it was (for instance if any parameters were obtained from fits using the previous version of the calculation).

The equation being changed is from equation (1) in doi:10.1103/PhysRevD.91.042002. Is the conclusion now that this equation is wrong?

resolved all threads

added 1 commit

• c9112f1d - coatingthermal.py: formatting fixes

Compare with previous version

resolved all threads

added 1 commit

• 84791178 - Fix an error in the coating brownian noise calculation

Compare with previous version

approved this merge request

@christopher.wipf you want to followup with a separate MR to update the test cache?

I want to figure out a better way to handle MR that change noise curves. We want to generate the report that shows the change, but then not have to go through a separate step to update. Open to suggestions...

mentioned in merge request !78 (merged)

resolved all threads

approved this merge request

changed the description

unapproved this merge request

added 36 commits

• 84791178...a5f2d45e - 35 commits from branch master
• 8d3962e4 - Fix an error in the coating brownian noise calculation

Compare with previous version

approved this merge request

added 7 commits

• 8d3962e4...131d6887 - 6 commits from branch master
• a813f541 - Fix an error in the coating brownian noise calculation

Compare with previous version

approved this merge request

enabled an automatic merge when the pipeline for a813f541 succeeds

merged

mentioned in commit fc21470a