Skip to content
Snippets Groups Projects

Define GeometryBackend for DD4Hep and DetDesc and use DataType=2024/Run3 when DetDesc is picked up from Run2Support

Merged Menglin Xu requested to merge mexu_backend_for_geometry into master
  • Issue: As https://lhcb-nightlies.web.cern.ch/nightly/lhcb-sim11-run5/10/Gauss/x86_64_v2-el9-gcc12-opt/tests showned, all dd4hep tests are failred as they are trying to access git:///Conditions/Online/valid_runs.txt. It is unreasonable because DD4hep does not use conditions.

  • Reason: In LHCb, UseDD4Hep is a compile time constant, while in Gauss it is decided at runtime.

  • Solution: DDDB should be removed form Run2Support, keep it only at LHCb, but it will take a long time to implement as some Gauss/Gaussion tasks are still rely on that. And this issue is blocking the progress of DD4hep related work, especially blocking the merge request for Upgrade2 new sub-detector's merge requst in Detector (we can not test them). A kind of temporary solution is introduce a new properties, Geometrybackend, for the Geometry.

  • Another change: Use DataType=2024/Run3 when DetDesc is picked up from Run2Support

Work with Run2Support!58 (merged) and Gauss!1056 (merged)

Includes !4388 (closed)

@gcorti @clemenci

Edited by Gloria Corti

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Menglin Xu mentioned in merge request Gauss!1056 (merged)

    mentioned in merge request Gauss!1056 (merged)

  • Menglin Xu added 1 commit

    added 1 commit

    • 9b72cd87 - making GeometryBackend either DD4Hep or DetDesc clearer

    Compare with previous version

  • Menglin Xu added 1 commit

    added 1 commit

    Compare with previous version

  • Menglin Xu added 1 commit

    added 1 commit

    • 7bf1dddb - consistent sanity check on the value

    Compare with previous version

  • mentioned in merge request Run2Support!58 (merged)

  • Menglin Xu changed the description

    changed the description

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading