Skip to content
Snippets Groups Projects

Add an extra check for []CC in the decay descriptor of FunTuple

Merged Jiahui Zhuo requested to merge jzhuo_cccheck into master

Addressing the issue Analysis#47 (comment 7219259)

This MR adds an extra function to check for []CC in the decay descriptor of FunTuple. The final target is to issue a warning when a self-conjugated particle is marked with []CC.

To be tested with: Analysis!1018 (merged) DaVinci!986 (merged)

Solve the issue: Analysis#47 (comment 7219259)

FYI: @pkoppenb @erodrigu @amathad

Edited by Jiahui Zhuo

Merge request reports

Merge request pipeline #6612462 passed

Merge request pipeline passed for b865535b

Approved by

Merged by Abhijit MathadAbhijit Mathad 1 year ago (Dec 15, 2023 12:47pm UTC)

Merge details

Pipeline #6622616 passed

Pipeline passed for 08bb7a53 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Jiahui Zhuo added 1 commit

    added 1 commit

    Compare with previous version

  • Jiahui Zhuo resolved all threads

    resolved all threads

  • Abhijit Mathad approved this merge request

    approved this merge request

  • Jiahui Zhuo added 1 commit

    added 1 commit

    Compare with previous version

  • Eduardo Rodrigues approved this merge request

    approved this merge request

  • assigned to @mzdybal

  • Edited by Software for LHCb
  • Patrick Koppenburg approved this merge request

    approved this merge request

  • mentioned in issue Moore#665 (closed)

  • Assigning to new shifter @jagoodin .

  • Milosz Zdybal assigned to @jagoodin and unassigned @mzdybal

    assigned to @jagoodin and unassigned @mzdybal

  • mentioned in issue Moore#669 (closed)

  • Jamie Gooding assigned to @mramospe and unassigned @jagoodin

    assigned to @mramospe and unassigned @jagoodin

  • Jiahui Zhuo added 1 commit

    added 1 commit

    Compare with previous version

  • Jiahui Zhuo added 1 commit

    added 1 commit

    Compare with previous version

  • Miguel Ramos Pernas assigned to @bjashal and unassigned @mramospe

    assigned to @bjashal and unassigned @mramospe

  • added 1 commit

    • 01f23958 - SimplifiedDecayParser.py - replace List, Dict with list, dict for type checking

    Compare with previous version

  • added 1 commit

    • d12e1aea - SubstitutePID.py - replace List with list for type checking

    Compare with previous version

  • Eduardo Rodrigues resolved all threads

    resolved all threads

  • Brij Kishor Jashal approved this merge request

    approved this merge request

  • Rosen Matev approved this merge request

    approved this merge request

  • Abhijit Mathad resolved all threads

    resolved all threads

    • Resolved by Abhijit Mathad

      @erodrigu mentioned here the failing test

          raise RuntimeError(
      RuntimeError: The decay 'B0 -> ( [ J/psi(1S) -> tau+ mu- ]CC ) (K*(892)0 -> K+ pi-)' has more than one arrow. Please check!
      ERROR: Failed to call DaVinciExamples.tupling.option_davinci_tupling_relation_isovariables:main(options), see above for more information.

      Not sure if this has been fixed @jzhuo . Let me trigger a new test since it has been a week.

      Edited by Abhijit Mathad
  • Jiahui Zhuo added 1 commit

    added 1 commit

    Compare with previous version

  • Jiahui Zhuo added 1 commit

    added 1 commit

    Compare with previous version

  • Abhijit Mathad resolved all threads

    resolved all threads

  • Abhijit Mathad resolved all threads

    resolved all threads

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