-
Notifications
You must be signed in to change notification settings - Fork 6
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
18 implement detailed geometry for sand ecal endcaps #40
Merged
ast0815
merged 33 commits into
master
from
18-implement-detailed-geometry-for-sand-ecal-endcaps
Jan 20, 2025
Merged
Changes from 28 commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
fd93be1
Implementation of detailed ECAL geometry
afa0042
Solving conflicts
e263bbc
Wrong subbuilder name corrected
07291e0
Corrected wrong Builder name
b9e539b
tab replaced with space at line 37 to avoid TabError: inconsistent us…
mt82 ffd2519
Merge remote-tracking branch 'origin/18-implement-detailed-geometry-f…
mt82 4b0ce52
alignment in config SAND_ECAL.cgf
mt82 4aebb1a
just code alignment
mt82 a574657
aluminum palte put inside mother volume
mt82 14dd40a
ECAL ec module element construction moved to function
mt82 3dd2a6c
renaming ActiveMat to ScintMat, so that they have the same names in t…
mt82 bbce300
single values
mt82 ad5dc23
definition of a shape for ECAL endcap module
mt82 3a1198b
The endcap module volume now is filled with the ECAL active/passive l…
mt82 48e3dad
renaming of ec module volume name
mt82 df7fce3
number of barrel modules into config file
mt82 7af1196
the whole endcap is now constructed and placed on both side of the ba…
mt82 64411f0
Corrected Al_plate position.
AlessandroRuggeri 4dc062b
Added dev option for the drift configuration.
AlessandroRuggeri 89d2557
Slab index starts at 0 at the outer radius.
AlessandroRuggeri c40dbc3
Merge pull request #31 from DUNE/29-fix-potitioning-of-the-active-and…
AlessandroRuggeri dd1d222
Fixed horizontal section IDs and positioning
AlessandroRuggeri fd33e4e
Merge branch '18-implement-detailed-geometry-for-sand-ecal-endcaps' i…
AlessandroRuggeri 7859fb5
Merge pull request #32 from DUNE/30-fix-potitioning-of-the-active-and…
AlessandroRuggeri 40a5cec
Updated but buggy geometry
AlessandroRuggeri c8a0f19
Added develop configuration.
AlessandroRuggeri cbe44b1
no overlaps; volume for yoke endcaps are removed from ecal endcap vol…
mt82 4faf4fd
Merge remote-tracking branch 'origin/master' into 18-implement-detail…
mt82 b1d2963
indentaion fixed
mt82 d0e5f99
removed useless commented lines
mt82 49b84b3
tab removed, 4 spaces instead
mt82 dca26d9
removed useless comments
mt82 bac7415
update of changelog
mt82 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The indentation does not align.