Skip to content
Snippets Groups Projects

configuration to use correct (only) NN for Run I

Merged Gabriel Facini requested to merge gfacini/athena:RunINN-master into master
All threads resolved!

When running a Run I configuration, the ML techniques used for splitting merged pixel clusters and assigning positions to all pixel clusters needs to be the NN version (not the more recent LWTNN package with the MDN).

As noted in ATLASRECTS-5836 and similar to the issues seen in ATR-22510 but I do not know what to change in the Trigger configs to avoid the crash that comes if this is not set

@jmasik

Edited by Gabriel Facini

Merge request reports

Pipeline #2190505 passed

Pipeline passed for e5604a5f on gfacini:RunINN-master

Approval is optional

Merged by Walter LamplWalter Lampl 4 years ago (Dec 21, 2020 3:46pm UTC)

Merge details

  • Changes merged into master with 44cd6daa (commits were squashed).
  • Deleted 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
  • Gabriel Facini added 1 commit

    added 1 commit

    Compare with previous version

  • Gabriel Facini resolved all threads

    resolved all threads

  • This merge request affects 1 package:

    • InnerDetector/InDetExample/InDetRecExample

    This merge request affects 1 file:

    • InnerDetector/InDetExample/InDetRecExample/python/TrackingCommon.py

    Adding @goetz ,@amorley ,@sroe as watchers

  • :white_check_mark: CI Result SUCCESS (hash e5604a5f)

    Athena AthSimulation AthGeneration AnalysisBase
    externals :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:
    make :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:
    optional tests :cloud: :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
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 25507]

  • I guess the review-pending-level-1 was removed by accident. Every not-WIP Mr should have one of the 'review' labels set. (Aside: More expensive versions of gitlab allow for mutually-exclusive labels)

    • Walter
  • merged

  • Walter Lampl mentioned in commit 44cd6daa

    mentioned in commit 44cd6daa

  • Please register or sign in to reply
    Loading