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

feat: material maps #557

Merged
merged 1 commit into from
Nov 16, 2023
Merged

Conversation

niermann999
Copy link
Contributor

@niermann999 niermann999 commented Sep 20, 2023

Add material grids as material maps and add the relevant types to the
detector metadata. The material maps of different shapes are tested
in a dedicated unittest.

Material maps are added to the toy detector and wire chamber via
the detector helper. The material is a silicon and aluminium mixture
that is mapped in configurable binning onto the disc and cylinder
portals.

@niermann999
Copy link
Contributor Author

niermann999 commented Sep 20, 2023

Based on #528

@niermann999 niermann999 force-pushed the feat-material-maps branch 3 times, most recently from 946b8e8 to fe970a5 Compare September 21, 2023 14:30
@niermann999 niermann999 force-pushed the feat-material-maps branch 3 times, most recently from 7f498fb to d6eb700 Compare October 2, 2023 11:48
@niermann999 niermann999 marked this pull request as ready for review October 2, 2023 11:48
@niermann999 niermann999 added enhancement New feature or request priority: high high priority labels Oct 2, 2023
@niermann999 niermann999 force-pushed the feat-material-maps branch 4 times, most recently from f53a2bb to 901e8f9 Compare October 8, 2023 16:16
@niermann999 niermann999 marked this pull request as draft October 9, 2023 07:45
@niermann999 niermann999 force-pushed the feat-material-maps branch 2 times, most recently from ed1e418 to 7fe1c2e Compare October 11, 2023 10:22
@niermann999 niermann999 force-pushed the feat-material-maps branch 7 times, most recently from 947db56 to ed76a26 Compare November 6, 2023 16:25
@niermann999 niermann999 marked this pull request as ready for review November 6, 2023 16:26
@niermann999
Copy link
Contributor Author

This still needs to be followed up by adding optional maps to sensitive surfaces and IO, so that the material scan works nicely. Until then, the maps that are generated are largely not validated

detector metadata. The material maps of different shapes are tested
in a dedicated unittest.

Material maps are added to the toy detector and wire chamber via
the detector helper. The material is a silicon and aluminium mixture
that is mapped in configurable binning onto the disc and cylinder
portals.
@niermann999 niermann999 merged commit a156321 into acts-project:main Nov 16, 2023
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority: high high priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants