Skip to content
Snippets Groups Projects

Test for reading from local AFP DB

Merged Petr Balek requested to merge pbalek/athena:master-afp-db-align-consts-test-v3 into master

Added a script that reads alignments constants from local AFP COOL DB and prints them. The also prints some other (randomly selected) condition data for comparison.

Edited by Petr Balek

Merge request reports

Pipeline #2426784 passed

Pipeline passed for cac88605 on pbalek:master-afp-db-align-consts-test-v3

Approval is optional

Merged by Adam Edward BartonAdam Edward Barton 4 years ago (Mar 24, 2021 9:50am UTC)

Merge details

  • Changes merged into master with 15b2521d.
  • Did not delete the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • @pbalek I flagged instances of commented-out code that could be removed. Otherwise looks good! -- L1

  • Petr Balek added 2 commits

    added 2 commits

    • 4b1560ab - removing comments
    • 7a5fa72e - Merge branch 'master-afp-db-align-consts-test-v3' of...

    Compare with previous version

  • Petr Balek resolved all threads

    resolved all threads

  • Author Developer

    Hi @lgagnon, I've removed the commented lines. Would it be OK now?

    Cheers Petr

  • This merge request affects 2 packages:

    • ForwardDetectors/AFP/AFP_DBTools
    • ForwardDetectors/AFP/AFP_Reconstruction/AFP_LocReco

    Affected files list will not be printed in this case

    Adding @goetz ,@ggach as watchers

  • :white_check_mark: CI Result SUCCESS (hash 7a5fa72e)

    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:
    required tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    optional 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
    :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 30443]

  • Petr Balek added 1 commit

    added 1 commit

    • cac88605 - fix return, remove unused variable

    Compare with previous version

  • This merge request affects 2 packages:

    • ForwardDetectors/AFP/AFP_DBTools
    • ForwardDetectors/AFP/AFP_Reconstruction/AFP_LocReco

    Affected files list will not be printed in this case

    Adding @goetz ,@ggach as watchers

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