Clarify the FLUKA comparison notes and point at the shipped NIEL weights - #62
Merged
Merged
Conversation
This follows up the Geant4 scoring section with the file location and one trap. - The damage weights now ship as share/Detectors/gconfig/data/rd50_niel.csv. - Without G4.g4fluenceweight the weighted scorer silently returns plain fluence. - The shipped weights have no electron table. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
…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>
This fixes the description of optical photons in the FLUKA/Geant4 comparison. - The pages said FLUKA does not produce Cherenkov photons in the FT0, whereas both engines do. - FLUKA does not receive the FT0 optical properties, so about six times more photons leave the detector than in Geant4. - This doubles the unfiltered total fluence near the FT0 planes; dose, 1 MeV n eq and hadron fluence are unaffected. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR corrects and completes the scoring pages added in #61, following a review of the FLUKA part by the FLUKA expert.
The MFT black-hole pitfall is now described by its scope. The name clash exists in O2 from February 2020 (AliceO2Group/AliceO2#2877) until September 2026, so it concerns only TFluka runs with an O2 version from that period and the MFT in the geometry. The official ALICE FLUKA calculations predate it, and the FoCal calculations on the A side are not affected.
The other changes:
FlukaParam.lowNeutron=true, which is off by default ino2-sim;FlukaParam.scoringFilealso switches it on.$O2_ROOT/share/Detectors/gconfig/data/rd50_niel.csv, now shipped with O2, and the page warns that the weightedcellFluxflag alone does nothing withoutG4.g4fluenceweight=true.Assisted by Claude Code.