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

Adding LLNL optics #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Adding LLNL optics #9

wants to merge 2 commits into from

Conversation

cmargalejo
Copy link
Member

This PR is the starting point to implement the LLNL telescope into REST. It includes the file with radii and angles of the mirrors and a simplified mirror definition.

It is simplified because it is assuming it is a bilayer, whereas in reality it is multilayer, but REST still does not support multilayer (see TRestAxionOpticsMirror). In this PR, the thicknesses of the outermost bilayer is used. However, not all the mirrors have the same thicknesses of each bilayer, although here the same is applied to all.

The correct reflectivities are calculated in PR #8

@cmargalejo cmargalejo requested review from jgalan and jovoy June 19, 2023 11:11
<constant name="pi" value="3.1415927" />
</globals>

<TRestAxionTrueWolterOptics name="xmm" verboseLevel="warning" >
Copy link
Contributor

Choose a reason for hiding this comment

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

Here no "true" (because llnl is cone shaped) and "llnl" for the name

<parameter name="initialRadius" value="0cm"/>
</TRestSpiderMask>

</TRestAxionTrueWolterOptics>
Copy link
Contributor

Choose a reason for hiding this comment

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

No "true"

<TRestAxionTrueWolterOptics name="xmm" verboseLevel="warning" >
<parameter name="opticsFile" value="LLNL.Wolter" />

<parameter name="mirrorLength" value="150" />
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be the length of one mirror module (225mm or something?)

<parameter name="substrate" value="SiO2" />
</TRestAxionOpticsMirror>

<TRestSpiderMask name="spider" verboseLevel="warning">
Copy link
Contributor

Choose a reason for hiding this comment

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

Need to figure out how to make that into the 2 mm wide graphite spacers. they were in the center which is 90° from the y-axis (as it is on the x-axis) so I think putting the armsSeperationAngle at 90° (or 360°/4) would make sense. The width is here in degree but can probably also be put in mm somehow.

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.

2 participants