Scenario Heavy
According to this codimd, The changes for Scenario Heavy included this MR are:
- add
compact/components/TV/trunk_Sx_H/
- More material: Module substrate 2 times 500 um of silicon – simple solution is to double the density
- RF-shield: thickness 150um
- add
compact/run5/trunk_Sx_H/
for visualization
Layout: keep as 32 station, missing modules are going to be delivered by masking in reco.
Validated by
-
Core Software -
RTA -
Simulation
Merge request reports
Activity
added Simulation TV labels
requested review from @tlatham
- Resolved by Thomas Latham
Thanks Ganrong! @tlatham Hi Tom, with this being in a new folder of
components
andrun5
, if it is merged now/soon, will any further changes to the xml's need to be propagated to both subfolders, a rebase wouldn't understand this as a copy from trunk with edits right?Just thinking at what moment is best to merge this...
Edited by Dan Thompson
- Resolved by Menglin Xu
The changes to the
compact/components/TV
area look good to me. As I see, the diff betweentrunk
andtrunk_Sx_H
is:--- trunk/parameters.xml 2024-03-21 08:54:49.648582813 +0000 +++ trunk_Sx_H/parameters.xml 2024-03-21 08:55:00.125690189 +0000 @@ -173,7 +173,7 @@ <!-- CYLINDRICAL FOIL SETTINGS --> <!-- For the Cylindical Foil, placed here for convinience --> - <constant name="TV:CylindRFFoilThickness" value="0.075*mm"/> + <constant name="TV:CylindRFFoilThickness" value="0.150*mm"/> @@ -885,7 +885,7 @@ </material> <material name = "TV:SubstrateSilicon" state="solid" > - <D type="density" unit="g/cm3" value="2.3300"/> + <D type="density" unit="g/cm3" value="4.6600"/> <fraction n="1" ref = "Silicon"/> </material>
So doubling the material of the foil and substrate.
However, I'm not sure we're meant to create new folders under
compact/run5
, that should be done by the project maintainers. What is the intention behind that new folder? If it's just to allow to easily run a debug xml then we could probably just create new files in the existing debug folder.
requested review from @tevans
added 1 commit
- aa73fbce - Make new LHCb version branch for run5 low scenario
mentioned in merge request !517 (closed)
- Resolved by Thomas Latham
@gawang, why have you closed this MR? I think this is the one that we need to put the heavy scenario into
master
, hence why I rebased it recently. Or am I wrong?
added 20 commits
-
2012883a...2863e268 - 19 commits from branch
master
- 2f767077 - Sx_heavy
-
2012883a...2863e268 - 19 commits from branch