Skip to content
Snippets Groups Projects
Commit 008a1b10 authored by Dan Thompson's avatar Dan Thompson Committed by Yunxuan Song
Browse files

Adding mue versions for the Lb2pkllLoose lines in Bu2LLK

parent 9876da2a
No related branches found
No related tags found
3 merge requests!1919Fixes for Bd2K1MuMuLine,!1751Adding mue versions for the Lb2pkllLoose lines in Bu2LLK,!1742Rd 2018 patches
Loading
  • Maintainer

    @dathomps, Is the plan to re-run StrippingBu2LLK in its entirety for the incremental re-stripping campaign or only to run the Lb2pkll lines? If it is the former, the reproducibility issues pointed out in #41 need to be fixed.

    On another point of order, BU2LLK is already huge, and adding more lines to this might not be very modular. Perhaps we could consider making a separate module? @ngrieser @fredi

  • Maintainer

    As an outsider, it makes no sense that a module called Bu2LLK.py houses so many things that are not Bu decays. I strongly encourage analysers if they want to keep using this to consider segmenting to something more reasonable. At the very least, this needs to be renamed to actually reflect what it is housing (and from my brief view, I cannot even make out why all these things are put together because it is so big..)

    Edited by Nate Grieser
  • Author Developer

    Hi @avenkate @ngrieser , Unless there was any request from the wider wg the plan was only to run the new pkmeLoose and pkmeSSLoose lines, as I have made no changes affecting other lines and can use the last restripping version in addition as a normalisation if we actually end up using the pkllLoose lines.

    I agree the line is getting busy, but I see this as completing the pkll set added in the last restripping.

  • Maintainer

    Okay that seems reasonable in this case, and I think we will just freeze anything else ever being added to this module.

    Please make sure your liaisons have a bookkeeping of which lines to run such that only the necessary ones make it into the config dictionary (maybe only report the timing/rates for the ones you want, like you've done is enough?)

  • Yingao Tang @yitang

    mentioned in merge request !1777 (closed)

    ·

    mentioned in merge request !1777 (closed)

    Toggle commit list
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment