Skip to content
Snippets Groups Projects

new parameter in CaloDataType

Merged Viktoriia Chekalina requested to merge WIP_Sayankotor into 2018-patches
All threads resolved!

Add isPhotonXGB parameter, to be used by CaloHypoEstimator to store result of new photon/pi0 separation classifier.

Edited by Marco Cattaneo

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
  • added 1 commit

    Compare with previous version

    • Explanation: we try to add new photon/pi0 tool, so we need a new parameter CaloHypoEstimator to store an answer of the new classifier as well as the old one.
    • moved to end, added to Name. Where can I find the logic of TypeMask field creation? I searched true example of using TypeMask, but didn't find out a relation between parameter name and TypeMask
  • Hi Vika, the TypeMask logic is (briefly) explained in the ICaloHypoEstimator interface :

    static const int TypeMask[Last] ={ // 0x1 : neutral ; 0x2 : charged ; 0x3 : both

    this is just a flag to indicate if the data is relevant for neutral CaloHypo (0x1), charged CaloHypo (0x2) or bot (0x3). This is used in the TupleToolCaloHypo for instance to filter the relevant tuple branch for a given particle, i.e. only neutral&both are associated to gamma and charged&both for electrons. In your case you must use the same mask as for the isPhoton (neutral only 0x1)

    Edited by Olivier Deschamps
  • Marco Cattaneo changed the description

    changed the description

  • added 1 commit

    Compare with previous version

  • @deschamp, thank you for an explanation

    Edited by Viktoriia Chekalina
  • Marco Cattaneo resolved all discussions

    resolved all discussions

  • Marco Cattaneo unmarked as a Work In Progress

    unmarked as a Work In Progress

  • Marco Cattaneo mentioned in commit d8290ce7

    mentioned in commit d8290ce7

  • Marco Cattaneo mentioned in commit 5f16cdc1

    mentioned in commit 5f16cdc1

  • Marco Cattaneo mentioned in merge request !1357 (merged)

    mentioned in merge request !1357 (merged)

  • Marco Cattaneo mentioned in commit 2e99a244

    mentioned in commit 2e99a244

  • Marco Cattaneo mentioned in merge request !1358 (merged)

    mentioned in merge request !1358 (merged)

  • Marco Cattaneo mentioned in merge request Rec!1087 (merged)

    mentioned in merge request Rec!1087 (merged)

  • Please register or sign in to reply
    Loading