Small change to hlt2_reco_baseline_with_mcchecking_no_gec.py

I'm changing a few lines in the hlt2_reco_baseline_with_mcchecking_no_gec.py options file so we can more easily change the number of events and the name of the output file and include tests without GECs on the dashboard, see https://gitlab.cern.ch/lhcb-rta/data-challenges/-/issues/17.

The changes make the options will more similar to hlt2_reco_baseline_with_mcchecking.py. It will change the outcome of hlt2_reco_baseline_with_mcchecking_no_gec.qmt, which will now run over 100 events, as specified in default_input_and_conds_hlt2.py.

assigned to @chasse

Note that this clashes with: !969 (diffs), where I set it manually to 100, but if we trust that it will always be 100 from other included files, I can also remove the manual setting.

Ok, it's good to know that you already wanted to change it to 100. I prefer not defining the number of events in these options files; the corresponding qm tests now sets it twice so you get different results depending on the order in which you run the options files.

That's fine. So if mine is merged first, and yours afterwards, then it should be correct by itself.

assigned to @sklaver and unassigned @chasse

Hi @sklaver just now getting to this. The other MR was merged first as that had already been tested in lhcb-head. But on the plus side this MR should now be quicker because it now probably doesn't need a ref udpate anymore

Would you mind rebasing this MR on top of the current master? Then feel free to assign this to the current shifter (@rangel) for a quick test and then we can merge this.

added 9 commits

• a4abce85...3982c8d7 - 6 commits from branch master
• 8b54815c - remove the number of events from this options file
• 803f2dbb - streamline the naming to hlt2_reco_baseline_with_mcchecking.py
• f88100ec - Merge branch 'sklaver_options_no_gec' of ssh://gitlab.cern.ch:7999/lhcb/Moore...

Compare with previous version

assigned to @rangel

unassigned @sklaver

/ci-test --merge

added ci-test-triggered label

• [2021-08-23 14:49] Validation started with lhcb-master-mr#2757

resolved all threads

approved this merge request

merged

mentioned in commit 1f143cd7

mentioned in issue #313 (closed)

added testing label