B2OC: Add Hlt2 LbToLambdaD lines
B2OC: Add 3 LbToLambdaD lines B2OC_LbToLambdaLLD0_D0ToHH B2OC_LbToLambdaLLD0_D0ToHHHH B2OC_LbToLambdaLLD0_D0ToHHHHWS
LAZY_AND: Hlt2B2OC_LbToLambdaLLD0_D0ToHHDecisionWithOutput #=10000 Sum=0 Eff=|( 0.000000 +- 0.00000 )%|
LAZY_AND: Hlt2B2OC_LbToLambdaLLD0_D0ToHHHHDecisionWithOutput #=10000 Sum=0 Eff=|( 0.000000 +- 0.00000 )%|
LAZY_AND: Hlt2B2OC_LbToLambdaLLD0_D0ToHHHHWSDecisionWithOutput #=10000 Sum=0 Eff=|( 0.000000 +- 0.00000 )%|
Merge request reports
Activity
added B2OC label
assigned to @hlei
hello @hlei cc @shunan, can you please run:
curl https://gitlab.cern.ch/lhcb/Moore/-/jobs/35590235/artifacts/raw/apply-formatting.patch | git am
to fix formatting and right after push again to your branch, this command is given if you click on the red bullet of the pipeline
as long as the pipeline is red (except on doc) we can not proceed further with others LHCb automatic checks
0 out of 10000 is kind of less than 1/10000 so less than 0.01 % which would be rather large so you may have to tune the rate further
please fix formatting first !
Edited by Alessandro Bertolin- Resolved by Shunan Zhang
- Resolved by Shunan Zhang
@hlei cc @shunan if we submit the code as it is now it is certain, 100 % probability, that we will get the question:
why are you leaving commented code in, for example, https://gitlab.cern.ch/lhcb/Moore/-/blob/b2oc_hlei_LbToLambdaD/Hlt/Hlt2Conf/python/Hlt2Conf/lines/b_to_open_charm/bbaryon_to_lightbaryon_d.py#L40-L51 ?
I see several other commented blocks of code
can you please remove them all and push a new version of bbaryon_to_lightbaryon_d.py ? I do not expect formatting issues when removing commented code but please check also formatting
ahhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhh but if you are commenting out downstream downstream lines please do not do that ! leave them uncommented in the code ! the DD lines just have to be removed from the list of lines being run and then we can put them back at the right moment
so in this last case remove the comment and leave the definition of the line
Edited by Alessandro Bertolin
- Resolved by Shunan Zhang
@hlei cc @shunan in this new line you are booking:
def make_LbToLambdaLLD0_D0ToHH(process): if process == 'spruce': lm = basic_builder.make_lambda_LL() dz = d_builder.make_dzero_to_hh() elif process == 'hlt2': lm = basic_builder.make_lambda_LL() dz = d_builder.make_dzero_to_hh() line_alg = b_builder.make_lb( particles=[lm, dz], descriptors=['Lambda_b0 -> Lambda0 D0', 'Lambda_b~0 -> Lambda~0 D0']) return line_alg
I do not see any difference between the 'spruce' and the 'hlt2' case right ? so I would remove the if elif block
same for the other new lines you have added if needed
Edited by Alessandro Bertolin
added PR/Moore_hlt2_bandwidth label
@shunan do you have anything else here ? if not, on Monday can you ask a CI round at least with the bandwidth platform ?
- Resolved by Alessandro Bertolin
@jagoodin as far as I know the PR bandwidth tests are back. We would like to study the rate and bandwidth of new lines added here so I temporarily changed the target branch to
master
. Could you launch a ci-test with platformx86_64_v3-el9-gcc13+detdesc-opt+g
? Many thanks.
added RTA label
added ci-test-triggered label