Add UL case for JMS&JMR SD corr in fatJetUncertainties

[bartokm]

When using fatJetUncertaintiesProducer with UL era (e.g. UL2018) the code crashes in this line, since era string doesn't cover UL cases in the previous if statement. With this simple fix the code works. Please note, that the corrections in JetWtagging twiki doesn't have separate factors for UL.

This issue was reported here: #285

@lathomas, @kirschen, @gouskos, @nowaythatsok

[alefisico]

Just to comment about the last statement "that the corrections in JetWtagging twiki doesn't have separate factors for UL".
We dont have yet values for UL campaigns. We were waiting for the nanoAOD version with the latest PUPPI tune. We plan to release these numbers asap.
In that sense, maybe you can create new categories for UL and temporarily copy the EOY values into the UL.

[bartokm]

Thanks for your comment, I've created separate categories for UL in the latest commit.

[tomalin]

This PR is still needed, as the code crashes without it. But there has been no progress merging it in over a year. Is anyone reviewing NanoAOD PRs?

[clelange]

@swertz and @vlimant are the new XPOG conveners who are historically responsible for this repository. I think the previous ones decided to deprecate nanoAOD-tools though and if so, this repository could be archived... The new Common Analysis Tools group might provide a new forum for this, but only in a few months and responsibilities will have to be discussed with XPOG.

[mariadalfonso]

@clelange and @lenzip you are admin of this cms-nanoAOD/nanoAOD-tools repository so that CAT and XPOG can decide how to go forward with this tools and other RDF and coffeea

[lenzip]

I have reviewed this PR, it only adds new flags to enable computing of SF for fatjets in UL campaigns. Defaults are unaffected.