-
Notifications
You must be signed in to change notification settings - Fork 50
Femcal update #855
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
Open
akioogawa
wants to merge
41
commits into
main
Choose a base branch
from
fEcal_update
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Femcal update #855
Changes from 11 commits
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
463e91c
initial version of fEcal update with current design
0c3a786
include beam pipe protector instead of insert.h
0259f97
z location update, new yml files, and hit update
3ae5445
Adding a version with actual 760 ScFi
ca908f4
adding fake CartesianGridXY for ScFi hits for speed while looking for
akioogawa 2ab0ad8
hit and detector name back to original. Small bug fix in map
akioogawa f133db2
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] f908817
Merge branch 'main' into fEcal_update
akioogawa d2f94ee
spdx
akioogawa 8326c34
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 15db2d2
changing craterlake.yml for auto overlap check
akioogawa be6af38
Delete configurations/craterlake_newfEcal.yml
akioogawa 76a7834
just renaming
akioogawa 8026edc
name change
akioogawa cea4eb1
rename preparing for merging
akioogawa 6d07ce2
Merging 2 geo.cpp
akioogawa 71a470a
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 18dcd8f
switching to new forward ecal geometry in all confiugration
akioogawa 0b7b36f
Fix typo which caused a crash
akioogawa 5d53535
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 89b3f8e
Merge branch 'main' into fEcal_update
akioogawa 44985a4
fix overlap with mother volume for bpp
akioogawa 2ee4f7e
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 6d18af3
fix material for homogeneous and name change for some volumes
akioogawa e2210eb
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] c70ed92
Merge branch 'main' into fEcal_update
akioogawa 4469a12
Merge branch 'main' into fEcal_update
veprbl cb2d525
Merge branch 'main' into fEcal_update
veprbl 4001c76
Merge branch 'main' into fEcal_update
akioogawa 0d99556
Merge remote-tracking branch 'origin/main' into fEcal_update
veprbl 4cdef89
rename internal compact name: forward_ScFi and forward_averaged_homog…
veprbl 0e2910e
StainlessSteel is not a material anymore, use StainlessSteelSAE304 as…
veprbl 73e7fb0
remove old compact/ecal/forward_homogeneous.xml
veprbl d992ae6
remove src/EndcapCalorimeterWithInsertCutout_geo.cpp
veprbl 01f4899
update outdated configurations
veprbl 47bdc84
Merge remote-tracking branch 'origin/main' into fEcal_update
veprbl 2e791f6
fix build
veprbl fc3c36e
fix ci
veprbl c180c17
fix spelling
veprbl 153f0d7
configurations: lfhcal_with_space_for_insert was deprecated on main
veprbl b3433a0
Merge branch 'main' into fEcal_update
akioogawa 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 hidden or 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
This file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,86 @@ | ||
|
||
<!-- SPDX-License-Identifier: LGPL-3.0-or-later --> | ||
<!-- Copyright (C) 2025 Akio Ogawa, Whitney Armstrong, Chao Peng, Sylvester Joosten, Zhongling Ji, Ryan Milton --> | ||
|
||
<lccdd> | ||
|
||
<info name="forwardEcal.xml" | ||
title="forward ECal 2025 design" | ||
author="@akioogawa" | ||
url="https://github.com/akioogawa" | ||
status="development" | ||
version="1.0"> | ||
<comment/></info> | ||
|
||
<define> | ||
<documentation> | ||
- Forward Ecal Homogeneous or ScFi Model | ||
</documentation> | ||
<constant name="ForwardEcal_Homogeneous_Scfi" value="1"/> | ||
|
||
<documentation> | ||
- Forward Ecal Slice Thickness | ||
</documentation> | ||
<constant name="ForwardEcalCoverThickness" value="0.343*mm"/> | ||
<constant name="ForwardEcalAirThickness" value="61.457*mm"/> | ||
<constant name="ForwardEcalPCBThickness" value="4.5*mm"/> | ||
<constant name="ForwardEcalLightGuideThickness" value="21.0*mm"/> | ||
<constant name="ForwardEcalScintillatorThickness" value="170.0*mm"/> | ||
<constant name="ForwardEcalMountingPlateThickness" value="12.7*mm"/> | ||
|
||
<documentation> | ||
- Forward Ecal computed total Thickness | ||
</documentation> | ||
<constant name="ForwardEcalThickness" | ||
value = "ForwardEcalCoverThickness + | ||
ForwardEcalAirThickness + | ||
ForwardEcalPCBThickness + | ||
ForwardEcalLightGuideThickness + | ||
ForwardEcalScintillatorThickness + | ||
ForwardEcalMountingPlateThickness" | ||
/> | ||
</define> | ||
|
||
<limits> | ||
</limits> | ||
|
||
<regions> | ||
</regions> | ||
|
||
<display> | ||
</display> | ||
|
||
<detectors> | ||
|
||
<documentation> | ||
### Forward (Positive Z) Endcap ScFi Electromagnetic Calorimeter | ||
Implementation of 2025 design | ||
Insert shape is cut out from middle of detector | ||
</documentation> | ||
<detector id="EcalEndcapP_ID" | ||
name="EcalEndcapP" | ||
type="epic_ForwardEcal" | ||
readout="EcalEndcapPHits"> | ||
<position x="0" y="0" z="EcalEndcapP_zmin"/> | ||
<dimensions rmin="EcalEndcapP_rmin" rmax="EcalEndcapP_rmax" z="ForwardEcalThickness"/> | ||
<slice name="Cover" material="Aluminum" thickness="ForwardEcalCoverThickness" vis="AnlGray"/> | ||
<slice name="Air" material="Air" thickness="ForwardEcalAirThickness" vis="AnlLight_Gray"/> | ||
<slice name="PCB" material="Fr4" thickness="ForwardEcalPCBThickness" vis="AnlGreen"/> | ||
<slice name="LG" material="Plexiglass" thickness="ForwardEcalLightGuideThickness" vis="AnlBlue"/> | ||
<slice name="WScFi" material="AvgTungstenScFi" thickness="ForwardEcalScintillatorThickness" vis="AnlOrange" sensitive="true"/> | ||
<slice name="MountingPlate" material="StainlessSteel" thickness="ForwardEcalMountingPlateThickness" vis="AnlGray"/> | ||
</detector> | ||
</detectors> | ||
|
||
<!-- Definition of the readout segmentation/definition --> | ||
<readouts> | ||
<readout name="EcalEndcapPHits"> | ||
<segmentation type="CartesianGridXY" grid_size_x="25*mm" grid_size_y="25*mm" offset_x="-37.5*mm" offset_y="-37.5*mm"/> | ||
<id>system:8,northsouth:1,blockrow:6,blockcol:5,x:2,y:2</id> | ||
</readout> | ||
</readouts> | ||
|
||
<plugins> | ||
</plugins> | ||
|
||
</lccdd> |
This file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,86 @@ | ||
<!-- SPDX-License-Identifier: LGPL-3.0-or-later --> | ||
<!-- Copyright (C) 2025 Akio Ogawa, Whitney Armstrong, Chao Peng, Sylvester Joosten, Zhongling Ji, Ryan Milton --> | ||
|
||
<lccdd> | ||
|
||
<info name="forwardEcalScFi.xml" | ||
title="forward ECal 2025 design" | ||
author="@akioogawa" | ||
url="https://github.com/akioogawa" | ||
status="development" | ||
version="1.0"> | ||
<comment/></info> | ||
|
||
<define> | ||
<documentation> | ||
- Forward Ecal Homogeneous(1) or ScFi Model | ||
</documentation> | ||
<constant name="ForwardEcal_Homogeneous_Scfi" value="2"/> | ||
|
||
<documentation> | ||
- Forward Ecal Slice Thickness | ||
</documentation> | ||
<constant name="ForwardEcalCoverThickness" value="0.343*mm"/> | ||
<constant name="ForwardEcalAirThickness" value="61.457*mm"/> | ||
<constant name="ForwardEcalPCBThickness" value="4.5*mm"/> | ||
<constant name="ForwardEcalLightGuideThickness" value="21.0*mm"/> | ||
<constant name="ForwardEcalScintillatorThickness" value="170.0*mm"/> | ||
<constant name="ForwardEcalMountingPlateThickness" value="12.7*mm"/> | ||
|
||
<documentation> | ||
- Forward Ecal computed total Thickness | ||
</documentation> | ||
<constant name="ForwardEcalThickness" | ||
value = "ForwardEcalCoverThickness + | ||
ForwardEcalAirThickness + | ||
ForwardEcalPCBThickness + | ||
ForwardEcalLightGuideThickness + | ||
ForwardEcalScintillatorThickness + | ||
ForwardEcalMountingPlateThickness" | ||
/> | ||
</define> | ||
|
||
<limits> | ||
</limits> | ||
|
||
<regions> | ||
</regions> | ||
|
||
<display> | ||
</display> | ||
|
||
<detectors> | ||
|
||
<documentation> | ||
### Forward (Positive Z) Endcap ScFi Electromagnetic Calorimeter | ||
Implementation of 2025 design | ||
Insert shape is cut out from middle of detector | ||
</documentation> | ||
<detector id="EcalEndcapP_ID" | ||
name="EcalEndcapP" | ||
type="epic_ForwardEcalScFi" | ||
readout="EcalEndcapPHits"> | ||
<position x="0" y="0" z="EcalEndcapP_zmin"/> | ||
<dimensions rmin="EcalEndcapP_rmin" rmax="EcalEndcapP_rmax" z="ForwardEcalThickness"/> | ||
<slice name="Cover" material="Aluminum" thickness="ForwardEcalCoverThickness" vis="AnlGray"/> | ||
<slice name="Air" material="Air" thickness="ForwardEcalAirThickness" vis="AnlLight_Gray"/> | ||
<slice name="PCB" material="Fr4" thickness="ForwardEcalPCBThickness" vis="AnlGreen"/> | ||
<slice name="LG" material="Plexiglass" thickness="ForwardEcalLightGuideThickness" vis="AnlBlue"/> | ||
<slice name="WScFi" material="AvgTungstenScFi" thickness="ForwardEcalScintillatorThickness" vis="AnlOrange" sensitive="true"/> | ||
<slice name="MountingPlate" material="StainlessSteel" thickness="ForwardEcalMountingPlateThickness" vis="AnlGray"/> | ||
</detector> | ||
</detectors> | ||
|
||
<!-- Definition of the readout segmentation/definition --> | ||
<readouts> | ||
<readout name="EcalEndcapPHits"> | ||
<!-- <segmentation type="NoSegmentation"/> --> <!-- Adding this slow down startup to >30min --> | ||
<segmentation type="CartesianGridXY" grid_size_x="10*cm" grid_size_y="10*cm" offset_x="-0.0*cm" offset_y="-0.0*cm"/> | ||
<id>system:8,northsouth:1,blockrow:6,blockcol:5,towerx:2,towery:2,fiberx:5,fibery:5,x:1,y:1</id> | ||
</readout> | ||
</readouts> | ||
|
||
<plugins> | ||
</plugins> | ||
|
||
</lccdd> |
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,37 @@ | ||
ebeam: 5 | ||
pbeam: 41 | ||
features: | ||
beampipe: | ||
tracking: | ||
definitions_craterlake: | ||
vertex_barrel: | ||
silicon_barrel: | ||
mpgd_barrel: | ||
support_service_craterlake: | ||
mpgd_outerbarrel: | ||
mpgd_forward_endcap: | ||
mpgd_backward_endcap: | ||
silicon_disks: | ||
tof_barrel: | ||
tof_endcap: | ||
pid: | ||
dirc: | ||
pfrich: | ||
drich: | ||
ecal: | ||
forwardEcal: | ||
bic_default: | ||
backward_PbWO4: | ||
solenoid: | ||
hcal: | ||
lfhcal_with_space_for_insert: | ||
forward_insert: | ||
barrel_gdml: | ||
barrel_flux_return: | ||
forward_endcap_flux: | ||
backward: | ||
backward_endcap_flux: | ||
far_forward: | ||
default: | ||
far_backward: | ||
default: |
akioogawa marked this conversation as resolved.
Show resolved
Hide resolved
|
This file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
features: | ||
fields: | ||
marco: | ||
ecal: | ||
forwardEcal: | ||
hcal: | ||
lfhcal_with_space_for_insert: | ||
forward_insert: |
akioogawa marked this conversation as resolved.
Show resolved
Hide resolved
|
This file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
features: | ||
beampipe: | ||
fields: | ||
marco: | ||
ecal: | ||
forwardEcal: |
akioogawa marked this conversation as resolved.
Show resolved
Hide resolved
|
This file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
features: | ||
beampipe: | ||
fields: | ||
marco: | ||
ecal: | ||
forwardEcalScFi: |
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.
Uh oh!
There was an error while loading. Please reload this page.