Skip to content
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.

Sign up for GitHub

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

Jump to bottom

WIP: development of Drift Chamber for CEPC #13

Merged
merged 5 commits into from
Sep 9, 2020
Merged

WIP: development of Drift Chamber for CEPC #13

merged 5 commits into from
Sep 9, 2020

Conversation

mirguest
Copy link
Member

@mirguest mirguest commented Aug 31, 2020 •
edited
Loading

  • detector description (DD4hep)
    • The dimensions are defined as the constants (See commit 2d3a664)
    • Envelope for the Drift Chamber
      • radii: 235 mm, 1716 mm
      • z: -2225 mm, 2225 mm
    • inner chamber
      • radii: 235 mm, 909 mm
      • z: -2225 mm, 2225 mm
    • outer chamber
      • radii: 1085 mm, 1716 mm
      • z: -2225 mm, 2225 mm
    • silicon between inner and outer chamber
    • B-Field: 3 Tesla, uniform: See commit a663edc
  • detector response (Geant4 Sensitive Detector)
    • register the Tracker SD for Drift Chamber as the first step.
    • need a specific SD for Drift Chamber to simulate:
      • the drift distance and time
      • dE/dx
    • Need a DD4hep Segmentation to calculate the CellID
  • digitization
  • Definition of the cellID
    • system: 8bit
    • chamber: 1bit (0: inner, 1: outer)
    • layer: 7bit (for inner: 67 layers, for outer: 63 layers)

@mirguest
Copy link
Member Author

A very simple setup of geometry: a tube with inner radius of 235mm and outer radius of 1716mm.
image

@mirguest
Copy link
Member Author

An example of the MCTruth hits (primary particle: 1 GeV electron):
image

@mirguest mirguest added this to the v0.1 milestone Aug 31, 2020
ihepzhangyao
ihepzhangyao reviewed Sep 1, 2020
View reviewed changes
Copy link
Contributor

@ihepzhangyao ihepzhangyao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you have the example to run the simulation? And it would be better to have a basic analysis of validate the output of the simulation.

Detector/DetDriftChamber/src/driftchamber/DriftChamber.cpp Outdated

dd4hep::Material det_mat(theDetector.material("Air"));

dd4hep::Tube det_solid(23.5, 171.6, 222.5);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shall we make the numbers based on a unified unit system in the CEPCSW?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The dimensions are moved to the xml file. See commit 2d3a664

@mirguest
Copy link
Member Author

mirguest commented Sep 7, 2020

Add the two chambers (inner and outer). The gap between two chambers is (1085mm - 909mm) = 176mm.
image

@mirguest
Copy link
Member Author

mirguest commented Sep 9, 2020

Hi @xshi ,
Even though some tasks listed here are not implemented yet, I would ask you to merge this PR first. Thank you.

Tao

@mirguest mirguest mentioned this pull request Sep 9, 2020
Open
18 tasks
@mirguest
Copy link
Member Author

mirguest commented Sep 9, 2020

Hi @xshi , I have create an issue #18 to track the development of drift chamber.

@mirguest mirguest changed the title WIP: start the development of Drift Chamber for CEPC. WIP: development of Drift Chamber for CEPC Sep 9, 2020
@xshi xshi merged commit cfb793f into cepc:master Sep 9, 2020
@mirguest mirguest deleted the sdt branch December 22, 2020 14:52
@ihepzhangyao ihepzhangyao mentioned this pull request Jan 29, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ihepzhangyao ihepzhangyao ihepzhangyao left review comments

@xshi xshi xshi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.1
Development

Successfully merging this pull request may close these issues.
3 participants
@mirguest @xshi @ihepzhangyao