Skip to content
Snippets Groups Projects

LoKiCore : add new class Polarization

Merged Vanya Belyaev requested to merge vanya-polarization-v0 into 2018-patches

LoKiCore : add new class Polarization with functions for calculation of various polarization quantities:

  • frames
  • direction cosines
  • polarization variables

This MR for 2018-patches and master branches. If there are no conflicts and compilation errors, it could be propagates to 2017-patchesbranch, but it is not necessary.

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
  • Edited by Software for LHCb
  • Vanya Belyaev added 1 commit

    added 1 commit

    • 98d6960a - Polariztaion: use the trick with tagged_bool, thanks to Gerhard!

    Compare with previous version

  • Vanya Belyaev added 2 commits

    added 2 commits

    • c1752605 - update LoKi:Tensors and Polarization
    • 452f9969 - update LoKi::Tensors and Polarization

    Compare with previous version

  • Vanya Belyaev added 1 commit

    added 1 commit

    • e7c038d4 - Polarization: add polarization vectors

    Compare with previous version

  • Author Developer

    @cattanem hi marco. Am I right that I am too late with this MR for LHCb v50r0 ?

  • Yes. But there will be other releases... In any case the master stack is not really intended for mainstream Run2 analysis

  • Marco Cattaneo resolved all discussions

    resolved all discussions

  • @ibelyaev Something is causing many of the lhcb-2018-patches tests crash in the dictionary system, I can't really tell whether it is due to this MR but I am making it WIP to test. Could you please have a look at the stderr of any of the failing tests in, e.g., https://lhcb-nightlies.cern.ch/logs/tests/nightly/lhcb-2018-patches/155/x86_64-slc6-gcc62-opt/LHCb/

  • Marco Cattaneo marked as a Work In Progress

    marked as a Work In Progress

  • Author Developer

    @cattanem I can't reproduce the error locally with LoKiGen, LoKiMC,LoKiNumbers, ...

  • What can I say? I restarted the nightly without this MR (https://lhcb-nightlies.cern.ch/nightly/lhcb-2018-patches/build/156/) and it's a lot happier than with it (https://lhcb-nightlies.cern.ch/nightly/lhcb-2018-patches/build/155/)

    All I can suggest is that I put this back and try again, just in case it was a glitch... Will be build 157 tomorrow morning.

  • Marco Cattaneo unmarked as a Work In Progress

    unmarked as a Work In Progress

  • Author Developer

    I've managed to reproduce the failures (one needs to make a fresh build afte r make purge, otherwise the error is not reproducible. I think I've found a solution, will commit it soon

  • Vanya Belyaev added 1 commit
  • Author Developer

    @cattanem I think I've "fixed" it. At least LoKi-tests are ok. The problem is a kind of unknown. I've replaced Gaudi::tagged_bool with plain bool, and the problem have been disappeared. Note just the presence of the header #include "GaudiKernel/TaggedBool.h" is enough to produce the problem, even without usage of this class @graven

    Edited by Vanya Belyaev
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading