search

elaird / supy

analyze events stored in TTrees in parallel
8 stars 7 forks source link

Debug nentries #147

Closed gerbaudo closed 12 years ago

gerbaudo commented 12 years ago

Hi Burt and Ted,

I see that, when I don't specify the number of entries and run in batch, my jobs don't run on any event. This seems to be due to wrappedChain.entries, for which I see at the first event

entries(nEntries=-1 ) = -1
nEntries : -1, self.entry: 0

so the condition at L68 is satisfied and it exits from the loop. I believe that this is a bug, and that it can be fixed with the additional "-1<". Please let me know if this sounds reasonable.

Thanks,

davide

P.S. I don't know why I did not see the problem when the branch below was merged...maybe I tested the branch with a specific n of events... https://github.com/elaird/supy/pull/112

gerbaudo commented 12 years ago

My bad. I see now that the default argument for nEntries is 'None', which I was overriding with '-1' in my analysis file. This is actually not a bug. Sorry for the noise.