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

Update beampipe #87

Merged
merged 2 commits into from
Apr 12, 2017
Merged

Update beampipe #87

merged 2 commits into from
Apr 12, 2017

Conversation

danieljeans
Copy link
Contributor

BEGINRELEASENOTES
various updates to beampipe description:

  • inner radius of central tube reduced from 14 to 13.5mm to match DBD
  • tube thickness in conical regions corrected for the cone angle
  • second cone: now made of Be-Cu mix, and given uniform thickness of 2.7mm. this is to simulate a 2mm-thick Be beampipe with 0.7mm of Cu cables around it.
    ENDRELEASENOTES

Daniel Jeans added 2 commits April 12, 2017 10:53
beampipe: reduce size of inner tube to match DBD model; correct treat…
8bce136 
…ment of cone's wall thickness; conservative design of second cone: uniform 2mm thick Be
beampipe second cone: change material to Be+Cu mix and increase thick…
58329d6 
…ness, to model cables from inner tracking detectors
@gaede gaede merged commit c528262 into key4hep:master Apr 12, 2017
@danieljeans danieljeans deleted the updateBeampipe branch March 14, 2018 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gaede gaede gaede approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@danieljeans @gaede