-
Notifications
You must be signed in to change notification settings - Fork 102
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
Import ddsim #420
Import ddsim #420
Conversation
…erywhere Move DD4hepSimulation.py and RunProg.py to DDSim folder Rename RunProg.py to ddsim Add cmake install for DD4hepSimulation to lib/python, ddsim to bin which is already in PATH
* Move DD4hepSimulation et al into python module DDSim, needed for proper importing * Can now control magneticfield setup, gun, particle handler, output via command line. ** added ConfigHelper and daughter classes implementing these options DDSim/DDSim/Helper, easily extensible to additional features ** also easier changing of these parameters via steering file See example/steeringFile.py *** No more functions, no more "global GeV" ** --help string can be given by using @Property, see e.g. Helper/Output.py * added optional argcomplete, allows command line "tab"complete for all command line options (and values!) Needs argcomplete (sudo pip install argcomplete), bash completion and bash (or zsh). and 'eval "$(register-python-argcomplete ddsim)"' * move import statements to reduce startup time until "--help", mandatory to reduce latency for argcomplete
to avoid conflicts on case-insensitive OSs
which is the default location on more OSs except SL ...
…print the configuration parameters
in command line for example "--action.mapActions 'tcp TPCSDAction'" or see example/steeringFile.py
…omething goes wrong
use absolute path for steering, compact and macro file remove magneticFieldDict because no longer used
…r actions see the steering file or the ConfigHelper/Action docstring for an example for CaloPreShowerSDAction
…sitive detectors See example/steeringFile.py and DDSim/Helper/Filter.py for examples
changed member variables to private: _errorMessages, _dumpParameter, _dumpSteeringFile removed abspath from macroFile and compactFile because otherwise the default values break error checking
Move module doc to class doc so we can use it in the automatically created steering file Change docstrings to be more sensible
Move applyFilter to Helper/Filter Adapt steeringFile
if not specific filter is set the default filter for tracker or calorimeter is applied (if a default filter is set) To turn off all filters just assign an empty string, empty list or None to the mapDetFilter update doc
…of inputfile is used multiple times
None is default for setupFuction, so we can allways pass the action and don't need the else setupSD now accepts tuples
…2054) with the additional range cut parameter created Physics Helper object the DD4hepSimulation physicsList attribute is now deprecated, use physics.list instead The default value for the rangecut is the one from geant4 10.1: 700 micron
Adapt to the new way the rangeCut is added
…of parent to set the flag needs dd4hep revision 2071
…ventNumberOffset to steer lcio output parameters
Could you please add a test that uses this, or modify one test in externals to use this? |
Added the test which simulates the SiD detector https://travis-ci.org/AIDASoft/DD4hep/jobs/407032010#L1552 |
New test_ddsim expects libraries |
You have to be a lot more precise about the problem and its circumstances. |
|
Doesn't happen on our mac tests, https://gitlab.cern.ch/CLICdp/DetectorSoftware/DD4hep/-/jobs/1852148 |
yes, I do
…On 01/08/2018 09:36, Andre Sailer wrote:
Doesn't happen on our mac tests,
https://gitlab.cern.ch/CLICdp/DetectorSoftware/DD4hep/-/jobs/1852148
Do you source thisdd4hep.sh before running the tests?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#420 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AI-LINOmVo-Eu-xiHhF87olHzAnWgRQzks5uMVqVgaJpZM4VX5Sx>.
|
> cmake --version CMAKE - command: cmake DD4HEP_LIBRARY_PATH=/opt/DD4hep/install/lib:/opt/xerces/install/lib:/opt/geant4/geant4-install/lib64:/opt/root/root-install/lib:/opt/QT/5.9.2/lib:/opt/boost/lib: > otool -L libDDCore.1.8.dylib libDDCore.1.8.dylib: > otool -l libDDCore.1.8.dylib libDDCore.1.8.dylib: |
Please continue here #425 |
BEGINRELEASENOTES
For example:
ddsim --help
ddsim --dumpSteeringFile > mySteer.py
ddsim --steeringFile=mySteer.py --compactFile myDetector.xml
ENDRELEASENOTES