What happened with DataScaleFactor property for the PRW tool?

[jbossios]

I remember that it was hardcoded at some point, but why remove it instead of making it configurable?

[ntadej]

It was just renamed and changed into a map<string, float> (but after the discussion with @kratsg and @fscutti, the type of the output will change). In the previos form there was no way to specify different triggers to get DataPrescale for (just OR of all of them). If you can tell me your use-case I can adapt it efficiently for you or even return the default OR of all triggers.

[jbossios]

Hi @ntadej

I'm not taking about prescaling factors but instead the "DataScaleFactor" property of the PileupReweighting tool:
https://twiki.cern.ch/twiki/bin/view/AtlasProtected/ExtendedPileupReweighting

Anyway, in my case even when I'm using prescaled triggers I don't care about prescale factors when I run with xAH because I calculate the corresponding luminosity of the OR of used triggers afterwards with a dedicated code.

Best,
Jona

[ntadej]

Ah, yes, sorry 😊

Sure, we can make it configurable again.