Skip to content
Snippets Groups Projects

Acts IDPVM implementation for ART Tests

Merged Carlo Varni requested to merge cvarni/athena:ActsIDPVM into master
All threads resolved!

Implementing a tool for running Acts Validation with runIDPVM.py. Right now it only produces distributions for Acts Clusters, which should be stable enough now and ready for being monitored. This tool will prepare the root file needed by dcube. As a choice I've remove all non-acts plots from the xml file so not to produce plots we are not interested it. Thus the summary plot webpage will only contain Acts plots.

Also adding an ART test (tt-bar PU200) that will compute these distributions and compare them with latest as well as athena distributions (i.e. the very same distributions obtained with Athena Clusters).

Produced plots look like this: https://cvarni.web.cern.ch/ActsArtTest/dcube_last/

Action needed:

  • Update xml for plot definitions (see below)
  • Upload reference file containing same distributions produced with Athena clusters instead

The xml file looks like this

<TDirectory name="/">
   <TDirectory name="SquirrelPlots">
    <TDirectory name="Acts">
      <TDirectory name="ITkPixelClusters">
        <hist2D name="global_xy_barrel" plotopts="box" tests="KS" type="TH2F"/>
        <hist2D name="global_xy_endcap" plotopts="box" tests="KS" type="TH2F"/>
        <hist2D name="global_zr_barrel" plotopts="box" tests="KS" type="TH2F"/>
        <hist2D name="global_zr_endcap" plotopts="box" tests="KS" type="TH2F"/>
        <hist1D name="barrelEndcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="layerDisk_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="layerDisk_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="phiModule_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="phiModule_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="etaModule_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="etaModule_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="isInnermost_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="isInnermost_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="isNextToInnermost_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="isNextToInnermost_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="eta_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="eta_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="perp_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="perp_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="global_x_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="global_x_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="global_y_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="global_y_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="global_z_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="global_z_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="local_x_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="local_x_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="local_y_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="local_y_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="localCovXX_barrel" plotopts="logy" tests="KS" type="TH1F"/>
        <hist1D name="localCovXX_endcap" plotopts="logy" tests="KS" type="TH1F"/>
        <hist1D name="localCovYY_barrel" plotopts="logy" tests="KS" type="TH1F"/>
        <hist1D name="localCovYY_endcap" plotopts="logy" tests="KS" type="TH1F"/>
        <hist1D name="sizeX_barrel" plotopts="logy" tests="KS" type="TH1F"/>
        <hist1D name="sizeX_endcap" plotopts="logy" tests="KS" type="TH1F"/>
        <hist1D name="sizeY_barrel" plotopts="logy" tests="KS" type="TH1F"/>
        <hist1D name="sizeY_endcap" plotopts="logy" tests="KS" type="TH1F"/>
      </TDirectory>
      <TDirectory name="ITkStripClusters">
        <hist2D name="global_xy_barrel" plotopts="box" tests="KS" type="TH2F"/>
        <hist2D name="global_xy_endcap" plotopts="box" tests="KS" type="TH2F"/>
        <hist2D name="global_zr_barrel" plotopts="box" tests="KS" type="TH2F"/>
        <hist2D name="global_zr_endcap" plotopts="box" tests="KS" type="TH2F"/>
        <hist1D name="barrelEndcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="layerDisk_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="layerDisk_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="phiModule_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="phiModule_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="etaModule_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="etaModule_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="sideModule_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="sideModule_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="eta_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="eta_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="perp_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="perp_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="global_x_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="global_x_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="global_y_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="global_y_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="global_z_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="global_z_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="local_x_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="local_x_endcap" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="localCovXX_barrel" plotopts="logy" tests="KS" type="TH1F"/>
        <hist1D name="localCovXX_endcap" plotopts="logy" tests="KS" type="TH1F"/>
        <hist1D name="sizeX_barrel" plotopts="logy" tests="KS" type="TH1F"/>
        <hist1D name="sizeX_endcap" plotopts="logy" tests="KS" type="TH1F"/>
        <hist1D name="hitsInThirdTimeBin_barrel" plotopts="" tests="KS" type="TH1F"/>
        <hist1D name="hitsInThirdTimeBin_endcap" plotopts="" tests="KS" type="TH1F"/>
      </TDirectory>
    </TDirectory>
   </TDirectory>
  </TDirectory>

/cc @pagessin @tbold @ncalace @lgagnon @tadej @jmasik

Edited by Carlo Varni

Merge request reports

Pipeline #4893773 passed

Pipeline passed for e58c852c on cvarni:ActsIDPVM

Approval is optional

Merged by Walter LamplWalter Lampl 2 years ago (Dec 16, 2022 8:10am UTC)

Merge details

  • Changes merged into master with 8b99bb9f (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
  • :white_check_mark: CI Result SUCCESS (hash b2c77c4d)

    Athena
    externals :white_check_mark:
    cmake :white_check_mark:
    make :white_check_mark:
    tests :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
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 63579]

  • Carlo Varni resolved all threads

    resolved all threads

  • Carlo Varni added 1 commit

    added 1 commit

    • e58c852c - Cetralize persistification of EDM

    Compare with previous version

  • This merge request affects 4 packages:

    • InnerDetector/InDetValidation/InDetPhysValMonitoring
    • PhysicsAnalysis/PhysicsValidation/PhysValMonitoring
    • Tracking/Acts/ActsInterop
    • Tracking/Acts/ActsTrkTools/ActsTrkAnalysisTools

    Affected files list will not be printed in this case

    Adding @goetz ,@amorley ,@lgagnon ,@battagl ,@sroe ,@pagessin ,@tbold ,@stsuno ,@toyamaza ,@lshan ,@cvarni as watchers

  • :white_check_mark: CI Result SUCCESS (hash e58c852c)

    Athena
    externals :white_check_mark:
    cmake :white_check_mark:
    make :white_check_mark:
    tests :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
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 63588]

  • Looks good for me. There are a lot of changes in this MR and there are changes in the .sh file. I'll redirect to L2 so that someone else can see it.

    Sergei L1

  • Edson Carquin Lopez
  • Edson Carquin Lopez
  • Hello @cvarni, the MR looks good to me, I have only a coupld of minor comments that you could decide to address now or in a follow up MR, approving then --L2

  • Edson Carquin Lopez resolved all threads

    resolved all threads

  • Carlo Varni resolved all threads

    resolved all threads

  • added urgent label

  • Author Developer

    setting urgent hoping this makes it for tonight ART tests

  • merged

  • Walter Lampl mentioned in commit 8b99bb9f

    mentioned in commit 8b99bb9f

  • Please register or sign in to reply
    Loading