Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multiple additions + forward parameterisation + mvl addons #100

Merged
merged 12 commits into from
Aug 2, 2021

Conversation

preghenella
Copy link
Collaborator

No description provided.

@preghenella
Copy link
Collaborator Author

@njacazio please have a look.
@marcovanleeuwen I merged your development from your fwdpara branch locally, fixed some conflicts that originated from parallel development and this should be how it goes into the DelphesO2 repo. thanks

@preghenella
Copy link
Collaborator Author

I actually notice that is does not work, there was a piece I missed.
will fix it, sorry, coming back soon

@preghenella
Copy link
Collaborator Author

@njacazio
a new way to defined the efficiency is encoded in the LUT in the eff2 member.
a method has been added to the TrackSmearer to decide which efficiency to be used setWhatEfficiency.
by default we use the old efficiency definition, but this might change

@njacazio
Copy link
Collaborator

njacazio commented Aug 1, 2021

Ciao @preghenella very large amount of work. I'm fine with the changes!

@preghenella
Copy link
Collaborator Author

ok thanks @njacazio
I merge it. hopefully we will manage to test it ;)

@preghenella preghenella merged commit 188605a into AliceO2Group:master Aug 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants