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

Transmission window implementation #35

Merged
merged 10 commits into from
Sep 21, 2022
Merged

Transmission window implementation #35

merged 10 commits into from
Sep 21, 2022

Conversation

jgalan
Copy link
Member

@jgalan jgalan commented Sep 12, 2022

jgalan Ok: 94

This process will allow to define a combination of windows that will be used to calculate the transmission.

Inside the process one defines the name of the TRestAxionXrayWindow that wants to be implemented. The process will combine them to create a unique window definition with different layers.

For example:

        <addProcess type="TRestAxionTransmissionProcess" name="window" verboseLevel="debug" value="ON">
            <parameter name="center" value="(0,0,7500)mm" />
            <observable name="transmission"/>
            <window name="cathode" />
            <window name="strongBack" />
        </addProcess>

The TRestAxionXrayWindow should be defined inside TRestRun.

<TRestRun ...>
...
        <TRestAxionXrayWindow name="strongBack" verboseLevel="warning">
            <parameter name="center" value="(0,0,0)mm" />
            <parameter name="thickness" value="200um" />
            <parameter name="material" value="Si" />

            <TRestGridMask name="mask" verboseLevel="warning" maskRadius="8mm" offset="(2,2)mm" 
                rotationAngle="0.5" gridGap="4mm" gridThickness="0.5mm" />
        </TRestAxionXrayWindow>

        <TRestAxionXrayWindow name="cathode" verboseLevel="info" center="(0,0,0)mm" thickness="0.02um" material="Al" />
....
</TRestRun>

@jgalan jgalan requested a review from jovoy September 13, 2022 11:14
@jgalan jgalan marked this pull request as ready for review September 13, 2022 11:14
@jovoy
Copy link
Contributor

jovoy commented Sep 13, 2022

A value that combines all probabilities like window_transmission and optics_efficiency is still missing but that does not necessarily belong in this PR.

@jgalan
Copy link
Member Author

jgalan commented Sep 21, 2022

A value that combines all probabilities like window_transmission and optics_efficiency is still missing but that does not necessarily belong in this PR.

This was discussed somewhere else. Efficiencies can be combined later on using the generated ROOT analysis tree

@jgalan jgalan merged commit fcecd9c into master Sep 21, 2022
@jgalan jgalan deleted the jgalan_transmission branch September 21, 2022 09:54
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