Hi,
This MR fixes a bug that shows up when both flipped and unflipped versions of DL1(r) are scheduled. Currently, the input variables (minimum|maximum|average)TrackRelativeEta are not stored separately in the BTagging container for the flipped and unflipped versions of the tagger (see here). Since the BTagJetAugmenter for the flipped tagger is scheduled after the one for the unflipped taggers (see here and here), the input variables for the latter are overwritten and DL1(r) is always run on (minimum|maximum|average)TrackRelativeEta belonging to the flipped version.
The fix implemented here follows suggestions by @dguest and seems to work fine at first glance. More testing is probably desirable.
Note that this problem affects derivations that schedule the flipped taggers, i.e. all FTAG-derivations. Physics derivations without the flipped taggers should not be compromised. This means that the FTAG derivations at the moment contain a strange variant of DL1(r), which is different from the one in the physics derivations and also different from the tagger that was actually trained. I guess this is not really want we want for the next round of calibrations.
Any feedback is appreciated!
Cheers, Philipp