Skip to content

Si disk with tiles #1073

Draft
ShujieL wants to merge 29 commits into
mainfrom
si_disk_hole
Draft

Si disk with tiles #1073
ShujieL wants to merge 29 commits into
mainfrom
si_disk_hole

Conversation

@ShujieL

@ShujieL ShujieL commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

Briefly, what does this PR introduce?

Assemble SVT disk with sensor tiles.

What kind of change does this PR introduce?

  • Bug fix (issue #__)
  • [ x] New feature (issue #__)
  • Documentation update
  • Other: __

Please check if this PR fulfills the following:

  • Tests for the changes have been added
  • [x ] Documentation has been added / updated
  • [x ] Changes have been communicated to collaborators

Does this PR introduce breaking changes? What changes might users need to make to their code?

Does this PR change default behavior?

@github-actions github-actions Bot added topic: infrastructure Regarding build system, CI, CD topic: tracking topic: forward Positive-rapidity detectors (hadron-going side) topic: materials labels Apr 7, 2026
@ShujieL

ShujieL commented Apr 7, 2026

Copy link
Copy Markdown
Contributor Author

The current code will read tile layout from SVT_endcap_tiles.csv, and place them in each disk envelope volume. The disk envelope is a SubtractionSolid = Disk - two beampipe tubes.

It won't run with EICrecon:
[] [acts] [error] D2A_L:InnerS Disc layer has wrong shape - needs to be TGeoTubeSeg!
[] [acts] [error] Error during DD4Hep -> ACTS geometry conversion: Disc layer has wrong shape - needs to be TGeoTubeSeg!

I submitted an ACT issue at acts-project/acts#5320

@github-actions

github-actions Bot commented May 7, 2026

Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot added the topic: documentation Improvements or additions to documentation label Jun 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic: documentation Improvements or additions to documentation topic: forward Positive-rapidity detectors (hadron-going side) topic: infrastructure Regarding build system, CI, CD topic: materials topic: tracking

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant