search

elaird / supy

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

supy depends on susycaf #153

Open elaird opened 11 years ago

elaird commented 11 years ago

In supy / samples / init.py, manageInclusive() uses a calculable (calculables.other.pthatLess) from susycaf.

betchart commented 11 years ago

I suppose it's a little less specific than that. It depends on the local library defining calculables.other.pthatLess. It could be generalized by making the calculable take a leaf name (and possibly an arbitrary prefix) and a value rather than just a value, but I'm not sure it is worthwhile to worry about it. The import is hidden inside the function, so it's only a problem if one wants to use the function, in which case it had better be defined.

I guess the problem that the function solves needs to be generalized if the solution is to fit well in supy.