Skip to content
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

Adapt to ROOT 6.22 python library ROOTTPython #632

Merged
merged 1 commit into from
Mar 20, 2020

Conversation

andresailer
Copy link
Member

Fixes another part of #625

BEGINRELEASENOTES

  • CMake: Adapt to ROOT 6.22 python library ROOTTPython

ENDRELEASENOTES

This PR is at the moment for testing against dev3.
While this PR here include s #631, #631 should just be merged by itself.

@andresailer
Copy link
Member Author

The installation is now working, but ddsim fails:

---running test :  ' ddsim --compactFile=/DD4hep/DDDetectors/compact/SiD.xml --runType=batch -G -N=2 --outputFile=testSid.root --gun.position "0.0 0.0 1.0*cm" --gun.direction "1.0 0.0 1.0" --gun.energy 100*GeV --part.userParticleHandler= '
Traceback (most recent call last):
  File "/DD4hep/bin/ddsim", line 24, in <module>
    RUNNER.run()
  File "/DD4hep/lib/python2.7/site-packages/DDSim/DD4hepSimulation.py", line 294, in run
    import DDG4
  File "/cvmfs/sft-nightlies.cern.ch/lcg/nightlies/dev3/Wed/ROOT/HEAD/x86_64-centos7-gcc9-opt/lib/python2.7/site-packages/ROOT/_facade.py", line 102, in _importhook
    return _orig_ihook(name, *args, **kwds)
  File "/DD4hep/lib/python2.7/site-packages/DDG4.py", line 13, in <module>
    from dd4hep_base import *  # noqa: F403
  File "/cvmfs/sft-nightlies.cern.ch/lcg/nightlies/dev3/Wed/ROOT/HEAD/x86_64-centos7-gcc9-opt/lib/python2.7/site-packages/ROOT/_facade.py", line 102, in _importhook
    return _orig_ihook(name, *args, **kwds)
  File "/DD4hep/lib/python2.7/site-packages/dd4hep_base.py", line 124, in <module>
    import_namespace_item('core', 'run_interpreter')
  File "/DD4hep/lib/python2.7/site-packages/dd4hep_base.py", line 67, in import_namespace_item
    attr = getattr(scope, nam)
AttributeError: <namespace cppyy.gbl.dd4hep at 0x5d57250> has no attribute 'run_interpreter'. Full details:
  type object 'dd4hep' has no attribute 'run_interpreter'
  'dd4hep::run_interpreter' is not a known C++ class
  'run_interpreter' is not a known C++ template
  'run_interpreter' is not a known C++ enum

*** Error in `python': double free or corruption (!prev): 0x0000000005a96410 ***

@andresailer andresailer marked this pull request as ready for review March 20, 2020 09:30
@andresailer andresailer changed the title [WIP] Adapt to ROOT 6.22 python library ROOTTPython Adapt to ROOT 6.22 python library ROOTTPython Mar 20, 2020
@petricm petricm merged commit 057a3be into AIDASoft:master Mar 20, 2020
@andresailer andresailer deleted the fixDDPython2 branch July 13, 2023 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants