Skip to content
Snippets Groups Projects

Upgrade support for geometry building from .sqlite files

Merged Nicholas Styles requested to merge nstyles/athena:master-sqliteForITk into master
All threads resolved!

Allow running of ITk/PLR/HGTD simulation using "pre-minted" .sqlite inputs, as is planned for the Run4 workflow.

This can be toggled by use of ConfigFlags.GeoModel.SQLiteDB = "../ITk.db" or similar.

This does not yet guarantee anything sensible for SiHitIDs, nor does it allow the ReadoutGeometry to be built (waiting on further developments to push necessary information to "auxiliary tables" in .sqlite files) but it allows simulation to run and will allow for e.g. validation of the material between the old and new geometry workflows.

Adding a few interested people.

For Upgrade SW: @sroe @hhayward @llongo @npetters @jmasik @dshope @sridouan @aleopold @dfellers

For Detector Description: @tsulaia @mbandier @boudreau @rbianchi

Merge request reports

Pipeline #4742270 passed

Pipeline passed for bda23720 on nstyles:master-sqliteForITk

Approval is optional
Ready to merge by members who can write to the target branch.

Merge details

  • 2 commits and 1 merge commit will be added to master.
  • Source branch will be deleted.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    Compare with previous version

  • Nicholas Styles resolved all threads

    resolved all threads

  • :pencil: There were multiple CI triggers for this MR and commit. The system ignored duplicates but the GitLab pipeline status may incorrectly show the job as failed. Once the remaining job finished running, the CI results will be posted as usual.

  • This merge request affects 7 packages:

    • HighGranularityTimingDetector/HGTD_DetDescr/HGTD_GeoModelXml
    • InnerDetector/InDetDetDescr/BCMPrimeGeoModelXml
    • InnerDetector/InDetDetDescr/InDetGeoModelUtils
    • InnerDetector/InDetDetDescr/PLRGeoModelXml
    • InnerDetector/InDetDetDescr/PixelGeoModelXml
    • InnerDetector/InDetDetDescr/ServiceExtensionGeoModelXml
    • InnerDetector/InDetDetDescr/StripGeoModelXml

    Affected files list will not be printed in this case

    Adding @goetz ,@dshope ,@stsuno ,@cohm ,@lshan ,@battagl ,@sroe as watchers

  • :white_check_mark: CI Result SUCCESS (hash bda23720)

    Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon
    externals :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :white_check_mark: AthSimulation: number of compilation errors 0, warnings 0
    :white_check_mark: AthGeneration: number of compilation errors 0, warnings 0
    :white_check_mark: AnalysisBase: number of compilation errors 0, warnings 0
    :white_check_mark: AthAnalysis: number of compilation errors 0, warnings 0
    :white_check_mark: DetCommon: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 61694]

  • merged

  • Tadej Novak mentioned in commit 45d261af

    mentioned in commit 45d261af

  • David Richard Shope mentioned in merge request !64053 (merged)

    mentioned in merge request !64053 (merged)

  • Please register or sign in to reply
    Loading