-
Notifications
You must be signed in to change notification settings - Fork 19
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.
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
dual readout Detectors added to FCCDetectors #34
base: main
Are you sure you want to change the base?
Conversation
align cellid 'system' field with other detectors
…rflow of field with small # of rotations)
Add .gitignore for standard .gitignore for a gaudi based project.
CMake Cleanup
This was introduced in 10.7 and is required as of 11.0, see https://geant4-userdoc.web.cern.ch/UsersGuides/ForApplicationDeveloper/html/TrackingAndPhysics/physicsProcess.html#enhanced-time-constants-new-in-10-7
…DRsegmentation DRsensitive subdirectories
As discussed, please add a test and resolve the conflicts. The |
I added a test to scan the X0 parameter using the materialScan command from DD4hep. The bash script takes 3 arguments, the compact file and two arguments that are used for defining the test: |
Hi,
This branch merged the dual readout Detectors sub-directory into the FCCDetectors repo.
I followed these steps:
dual-readout
repo using git filter, as explained hereI checked that the proyect can be built using this flags,
cmake ../FCCDetectors/ -DCMAKE_INSTALL_PREFIX=$PWD/../install -DEXECUTABLE_OUTPUT_PATH=$PWD/../install
If the last flag
EXECUTABLE_OUTPUT_PATH
is not provided, there is an issue with the DD4hep macro called in line 31 of this file, and the project can not be configured.line 31: dd4hep_configure_scripts( ddDRcalo DEFAULT_SETUP WITH_TESTS )
Can this be an issue?
Thanks you for your time.