Add new test in TrigAnalysisTest for RDO->RDO_FTK (ATR-19672)

Add new test for RDO->RDO_FTK test_mc_rdo_to_rdoftk_grid Add ls of PHYSVAL_WEB dorectory in test_mc_ntup_physval_grid

added 1 deleted label

added alsoTargeting:21.0 label

added 21.3 Trigger review-pending-level-1 labels

This merge request affects 1 package:

• Trigger/TrigValidation/TrigAnalysisTest

Adding @rbielski as watcher

added 1 commit

• fdd710d1 - fix test description and art-include

Compare with previous version

Hi @rbielski , Thanks for the checks. I fixed the first 2. For the post-processing the standard script gives lots of errors due to missing files. I'm thinking that we should maybe split up the script to deal with different output file types. Something we can discuss offline....

Julie

Sounds reasonable, thanks

resolved all discussions

This merge request affects 1 package:

• Trigger/TrigValidation/TrigAnalysisTest

Adding @rbielski as watcher

CI Result SUCCESS

	Athena
externals
cmake
make
required tests
optional tests

Full details available at NICOS MR-22721-2019-04-16-20-58
Athena: number of compilation errors 0, warnings 141
For experts only: Jenkins output [CI-MERGE-REQUEST 37189]

CI Result SUCCESS

	Athena
externals
cmake
make
required tests
optional tests

Full details available at NICOS MR-22721-2019-04-16-22-18
Athena: number of compilation errors 0, warnings 8
For experts only: Jenkins output [CI-MERGE-REQUEST 37204]

Hi @hartj and @rbielski,

from your discussion above I get the impression that you consider changing the MR? Or do you plan to address it in a future MR? If you are fine with the MR as it is, I will approve it, since it looks good and the CI test succeeded.

Best, Jannik (L1 shifter)

Hi @jgeisen,
Julie has already fixed the two minor issues and we agreed to follow up on the third suggestion in another MR. This one is fine now from our side.

Cheers,
Rafal

added review-approved label and removed review-pending-level-1 label

merged

mentioned in commit 782983e9

added sweep:done label

mentioned in commit 73ff29d8

mentioned in merge request !22776 (closed)

Sweep summary
successful:

• 21.0
• 21.9
• master

(see https://aiatlas154.cern.ch/jenkins/job/auto_git_sweeps/750/console for details)

mentioned in commit 74a218fc

mentioned in merge request !22777 (merged)

mentioned in commit 17bad7c7

mentioned in merge request !22778 (merged)

mentioned in commit fada29bb

mentioned in commit 5ded0528

mentioned in merge request !22918 (merged)

mentioned in commit 8d7ab8fb