Skip to content

MFT: Refactoring Supports & PCBs according to latest design - #2877

Merged
sawenzel merged 19 commits into
AliceO2Group:devfrom
rpezzi:MFTSup_Sept2019
Feb 14, 2020
Merged

sawenzel merged 19 commits into
AliceO2Group:devfrom
rpezzi:MFTSup_Sept2019

Conversation

@rpezzi

@rpezzi rpezzi commented Feb 2, 2020

Copy link
Copy Markdown
Collaborator

MFT supports and PCB geometry description rewritten taking advantage of pointers and function template to simplify repetitive shape boolean operations.

@rpezzi
rpezzi requested a review from bovulpes February 2, 2020 11:20
bovulpes
bovulpes previously approved these changes Feb 11, 2020

@bovulpes bovulpes left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

It was planned to replace the preliminary with the newest version from the mechanical blueprints.

@sawenzel

Copy link
Copy Markdown
Collaborator

There seem to be a few problems in o2checkcode such as nullptr and wrong file headers.

@rpezzi rpezzi changed the title Refactoring MFT Disks PCB & Supports MFT: Refactoring Supports & PCBs according to latest design Feb 13, 2020
@rpezzi

rpezzi commented Feb 14, 2020

Copy link
Copy Markdown
Collaborator Author

Hi @sawenzel , problems with o2checkcode have been fixed.

@sawenzel

Copy link
Copy Markdown
Collaborator

Thanks

@sawenzel
sawenzel merged commit ff810b7 into AliceO2Group:dev Feb 14, 2020
sawenzel added a commit to AliceO2Group/simulation that referenced this pull request Sep 29, 2026
…n weights

This updates the scoring pages after review by the FLUKA expert.

- The MFT black-hole paragraph now states which O2 versions are affected, and that the official ALICE FLUKA and FoCal A-side calculations are not.
- FLUKA low-energy neutron transport needs FlukaParam.lowNeutron=true, which is off by default in o2-sim.
- The NIEL weight file now carries the RD50 electron table; its source is cited.

https://rd50.web.cern.ch/NIEL/
AliceO2Group/AliceO2#2877

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants