search

elaird / supy

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

Fix example #137

Closed gerbaudo closed 12 years ago

gerbaudo commented 12 years ago

Hi Burt and Ted,

These are minor changes to fix the supy example (a friend of mine started to use supy, so I got a chance to test these again, and see that it was broken).

One comment: I'm not sure whether you want to have 'susyTree' instead of '/' in defaults.py. If not, we could drop 'examples' and point to https://github.com/davidegerbaudo/supy-minimal-example

davide

betchart commented 12 years ago

Hi Davide,

You can fix the configuration for a particular analysis by putting a specific configuration in the local directory with the analysis. So, you might want to move this example analysis to a subdirectory of its own and include its configuration file there.

Burt

gerbaudo commented 12 years ago

Thanks, Burt. This parameter is now specified in configuration.py. The example is now working again.

Davide

betchart commented 12 years ago

Merge is ok with me.

betchart commented 12 years ago

merging into 'devel' instead, will subsequently get merged to master.