Adding DV tests for S34r0p1

Nightlies tests for S34r0p1.

Do not merge until the database tag is replaced with the real tag instead of with the branch.

Will not run the tests yet because the functor cache is screwed up. I will open a JIRA task later.

added 1 commit

• dfc49ff7 - Add LokiAddParticleHeader as a post-config option to include additional needed...

Compare with previous version

Needs Phys!493 (merged) before merging.

added 1 commit

• ad771923 - Add temporarty tag to DV-Stripping34r0p1-Stripping.py (to be replaced when...

Compare with previous version

added 1 commit

• 63e5a55f - Correct typo in input file and add reference file

Compare with previous version

unmarked as a Work In Progress

changed the description

resolved all discussions

• [2019-02-02 00:11] Validation started with lhcb-2018-patches#477
• [2019-02-03 00:08] Validation started with lhcb-2018-patches#478
• [2019-02-04 00:12] Validation started with lhcb-2018-patches#479
• [2019-02-05 00:14] Validation started with lhcb-2018-patches#480
• [2019-02-06 00:09] Validation started with lhcb-2018-patches#481

Tests are not working as usual (the validators really needs to be changed) and there are a lot of warnings from StrippingCache. I think I know how to fix it, I'll give it a try.

@cvazquez, you've been very active these last 3 days. I will try and catch up with all your messages and commits asap.

@erodrigu I asked Liang and Michal to please try to have the db tags ready for tomorrow morning. After this is done, this MR will be ready for merge and a new DV release will be requested (so this will likely happen tomorrow).

OK!. BTW, that will hopefully solve the fact that the test is failing?

No, the problem with the test is that the validators are not properly chosen. This needs a bit more of investigation. I'm fully dedicating to the Stripping these two weeks, so once we go for validation, this is my next priority, as well as to fix the StrippingCache warnings.

@erodrigu Could you please merge this? I will change the db tag later. I have a proposed solution for the warnings in compilation and I don't want to mix up MRs (it involves the CMakeLists).

Also Phys!493 (merged) needs to be merged.

I can merge this now that I merged the dependent MR in Phys.

I won't propagate this to run2-patches and master, though, as this is not the final word on these tests. OK?

One sec, let me put this in WIP, I have another addition.

marked as a Work In Progress