Closed ChristianTackeGSI closed 6 months ago
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
@coderabbitai I think, you got some additional prompts like looking at CONTRIBUTING.md, etc. Which did you get? Have you worked on them?
[!TIP] For best results, initiate chat on the files or code changes.
@ChristianTackeGSI, I operate based on the information and instructions provided in each session and don't have the capability to recall or track tasks across sessions. If there are specific documents like CONTRIBUTING.md
or other tasks you'd like me to review or perform, please let me know, and I'll be happy to assist with those within the scope of the current interaction.
So, if we say we still adhere to it, we should at least discuss (quickly) why not
fgAllSensitiveVolumes
etc
Yes, we should!
Just: We should even more try to get rid of them… That's what I was trying with the preview of moving this to FairRunSim.
@coderabbitai generate a short executive summary on the code changes in this PR (base branch origin/dev) as well as its conversation thread if any important decisions were made.
[!TIP] For best results, initiate chat on the files or code changes.
@dennisklein, based on the provided diff output, here's a short executive summary of the code changes in PR #1499 and the conversation thread:
TRefArray
to std::vector<FairVolume*>
for storing sensitive volumes in FairModule
, with the variable now named fAllSensitiveVolumes
. This change eliminates the need for casting and enhances code readability.FairDetector.cxx
and FairMCApplication.cxx
to iterate over fAllSensitiveVolumes
directly, aligning with the new storage mechanism.TRefArray
.FairModule::svList
, further cleaning up the codebase.std::vector
for better efficiency and readability.fAllSensitiveVolumes
over fgAllSensitiveVolumes
. It was suggested that efforts should be made to reduce the usage of global variables, aligning with best practices for modern C++ development.This PR represents a significant step towards modernizing the FairRoot codebase by adopting standard C++ containers and improving code maintainability and readability.
FairModule::svList
can very well be implemented usingstd::vector
without all the casting.And rename to
fAllSensitiveVolumes
.Follow up from #1497
Checklist: