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

DTF via LoKi functors #3

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

FedericaLionetto
Copy link

No description provided.

@FedericaLionetto
Copy link
Author

Here we are! So, you said that the refit is done once per variable...is there a way to do it only once? I don't know whether that is the reason, but I noticed that it gets terribly slow when you have something like 20 variables...

@apuignav
Copy link
Contributor

Unfortunately, the way things are coded now it doesn't look like there's a way of only doing one refit. This is in fact one of the reasons why we decided against including DTF functors in the first starterkit. I'm sure it's the reason why your code is slow.

Having said that, I have some time scheduled with Vanya to discuss the LoKi functors part, so I'll try to see if there's some magic incantation that I haven't thought of to bypass this problem.

@FedericaLionetto
Copy link
Author

Ok, so for the moment I write a comment in the text, explaining that the refit is done several times and that it might take some time, and then if things change we update it accordingly ;)

@FedericaLionetto
Copy link
Author

I added a comment on pros and cons of the two approaches. Feel free to modify it and let me know if you think it's fine ;)

@seneubert
Copy link
Contributor

@apuignav I thought about this for a bit. Actually the MVADictTools might be a foundation to solve this. At least this is a LoKi-aware way to write a whole set of variables to the ntuple. Will require some coding, though to make the DTF produce a dictionary of all it's output and plug this into the dict tools (not a project for the starterkit)

@apuignav
Copy link
Contributor

I agree @sneubert, but the solution is not obvious, because it requires to pass the Decay Tree Fit from functor to functor, ie, save some kind of state. I guess this is the real challenge.

@seneubert
Copy link
Contributor

I guess what we would do is to not actually call the DTF from a functor (just like the BDT is not called in the functor but in the DictTool). It would actually be similar to the current TupleTool but with the added benefit that the syntax to write variables to your tree would look like LoKi functors.

@ibab
Copy link
Contributor

ibab commented Jul 22, 2015

Yes, someone write this please! 👍

@betatim betatim mentioned this pull request Nov 26, 2015
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.

4 participants