TRestAxionTransmissionProcess. Fixing the location of the window #38
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Any process should translate the particle to the Z=0 plane in order to evaluate the helioscope component (process).
It is
TRestAxionEventProcessthe responsible to counter-move the axion particle. So that the module is described at the origin and perfectly aligned.Added a fix so that physical placement of the window works properly when using
<parameter name="center" value="(x,y,z)" />.Added the possibility to add a vacuum window. Now
TRestPatternMaskis not pure virtual, and it allows to use a foil material without a pattern.TRestAxionXrayWindowRML definition now requires a pattern explicitily defined, as it is for example defined atwindows.rmlat axionlib-data. If we useTRestPatternMaskthen we get a window without pattern.Added a validation pipeline to check the
TRestAxionTransmissionProcessin different conditions.Requires PR rest-for-physics/framework#304 to be merged