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 pattern mask classes #198

Merged
merged 42 commits into from
May 27, 2022
Merged

Adding pattern mask classes #198

merged 42 commits into from
May 27, 2022

Conversation

jgalan
Copy link
Member

@jgalan jgalan commented May 6, 2022

jgalan Large: 1594

This PR adds an abstract metadata class TRestPatternMask that encapsulates the implementation of masks (or filters) that define a pattern. The abstract class implements a virtual method GetRegion that must be implemented at the inherited class with the logic for each type of mask. This method must return an integer identifying the region where the particle was hit.

TRestPatternMask implements a common method to test the mask DrawMonteCarlo, generating a canvas with the result.

TRestPatternMask implements rotation, pattern shift, and pattern limits (circular mask), those can be applied at each GetRegion method by using the method ApplyCommonMaskTransformation.

Four types of mask have been implemented at this PR. Stripped, Grid, Rings, Spider.

masks

@jgalan jgalan added this to the Release v2.3.13 milestone May 13, 2022
@jgalan jgalan marked this pull request as ready for review May 27, 2022 09:10
@jgalan jgalan merged commit f814c0b into master May 27, 2022
@jgalan jgalan deleted the jgalan_masks branch May 27, 2022 16:07
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.

None yet

3 participants