update JRA code from latest offline case to adapt online requirement #11
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi Alexx,
I have managed to modify the JRA code with the latest offline package for online use. There are several points conflicting:
vertex >=4 (offline) while vertex > = 1 (online), due to the HLT pixel vertices requirement.
(1) srcVtx = OfflinePrimaryVertices (offline) while srcVtx = hltPixelVertices (online)
(2) kt6Calojets and kt6PFjets objects are not compatible for HLT samples, therefore they have to be commented under the condition of "caloHLT" and "pfHLT"
(3) sentences corresponding to "parton" have to be commented since there is no parton module in HLT samples for JRA ntuple production
The version number for "JRAEvent" has to be incremented from 25 to 26 because of compilation requirement:
There are many other small changes with the optional parameters, such as the example below for "JetAnalyzers/bin/jet_apply_jec_x.cc". They are necessary for online case!
Some additional things:
In my opinion, it is quite difficult to synchronize the online and offline framework, since they seem to be not compatible. I'm not sure if they can be unified partly instead of all.
Thanks,
Feng