Repository navigation
12-Sector CyMBaL - #1146
12-Sector CyMBaL#1146ybedfer wants to merge 16 commits into
Conversation
- One and only one model of stave, thanks to the fish scale layout. - New, more realistic, stack-up (and more colors for its display). - No <service> any more (will have to be re-introduced).
- Both phi and Z pitches were bugged. - Resolutions are added, as <constant>'s. - "PCB" tag for module parameters, instead of "Active", to underline the presence of DeadZone.
for more information, see https://pre-commit.ci
|
This is the second one of the series of PRs meant to commit a 12-sector version of CyMBaL. The 12-sector layout has a fish scale structure that allows to have a unique model of module in all four sections along Here is a picture of the layout, w/ its 12 sectors.: The new version has a more detailed and realistic specification of the stack-up. It's based on the following list parameters (provided by Damien Neyret): The new stack-up contains more material than the previous version. See the following plot of %X0 as a function of θ, where new is red and old is blue: Below plots of the distributions of Another feature of the new version is the DeadZone lining the PCB on all four sides. Here is a picture of the resulting |
|
Hit-to-Track association (barrel MPGDs are subdivided in 'p', 'n' and 'p&n' sub-bins, 'p' being 'φ' or 'U' and 'n' being 'Z' or 'V'):
=> The association efficiency of 12SCyMBaL looks reasonable: it's as good as that of the Outer MPGD, slightly worse still than that of the rest of the barrel trackers.
=> CyMBaL, which is the next to most outward detector (on this picture there is no TOF), displays a discontinuity in radius when one moves from the outer Z-sections to the inner ones: this translates the outward shift applied to the outer sections mentioned supra (the actual radius of curvature is the same for all sections). (Note that I have had to edit the And I removed TOF from my |
|
@veprbl |
|
@ybedfer Thanks for the pointer. I'll look into that. |
|
(I did see it originally, but need to get to it) |
|
@veprbl
|
|
@veprbl Note that the error concerning is always there, even w/ the |
|
This last clue makes sense. If you change the geometry significantly, it won't load together with the current material map. But that's what |
|
I have to admit that I am completely lost.
I am not sure I understand. Do you mean that
I don't get how I can do this. How can take out "current"? And to do what?
I have spent an entire week, if not more, trying to understand precisely this. I listed supra all the changes I made to the 12-Sector geometry. In addition, I also removed the "tilt" (that "tilt" that gives rise to the fish-scale structure), so that I got a visual aspect very similar to the 8-Sector version: to no avail. The only way I could get rid of the error was to fully go back to the 8-Sector It could also be that I forget some requisite action (setting detector's flag or the like) in the detector builder. But I checked my source code several times and could not find... Something that I wanted to try, but could not, is to produce a material map w/out CyMBaL, removing And then: And then the error about |
|
epic/scripts/material_map/run_material_map_validation.sh Lines 209 to 210 in 25fb5c1 One operates on |
I did that. I think I did it correctly. |
|
@veprbl => No Then, as I remembered having read that => Same outcome: no |
|
I was told CBOR was fixed. I will try running it now. |
Yes. It definitely improves the situation. |
…12-Sector-CyMBaL. - This alternative file is meant to exist only temporary, until 12-Sector-CyMBaL is committed. - It will allow github checks to succeed for the PR of epic (this, once EICrecon PR is accepted).
|
Now that I have committed the link to the material map dedicated to |
This comment was marked as resolved.
This comment was marked as resolved.
|
Commit #5c6769693 (5c6769693") does have an impact (for the worse) on the % of X0. See picture of %X0 as a f(θ): |
|
And that makes sense - the overall Cu mass is now not scaled twice. |
Capybara summary for PR 1146
Material map validation
|
### Briefly, what does this PR introduce? Please link to any relevant presentations or discussions. New version of MPGD plugin and digitization compatible w/ upcoming 12-Sector-CyMBaL (see eic/epic#1146). ### What is the urgency of this PR? - [ *] Medium ### What kind of change does this PR introduce? - [* ] New feature (no related issue ever created) ### Please check if any of the following apply None. --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: epic-capybara <139920704+epic-capybara@users.noreply.github.com> Co-authored-by: Dmitry Kalinkin <dmitry.kalinkin@gmail.com>









Briefly, what does this PR introduce? Please link to any relevant presentations or discussions.
12-Sector CyMBaL, aiming at simplifying maintenance, see e.g. "https://indico.bnl.gov/event/28403"
What is the urgency of this PR?
What kind of change does this PR introduce?
Please check if any of the following apply
None