search

KIT-CMS / ntuple_processor

Submodule of ntuple-analysis containing the actual software to run the analysis
0 stars 3 forks source link

Remove `build_info` from `Dataset` #20

Closed stwunsch closed 4 years ago

stwunsch commented 4 years ago

Please remove the build_info from the Dataset class. It makes life much more complicated:

https://github.com/KIT-CMS/ntuple_processor/blob/master/utils/_booking.py#L70

I think you introduced it to have the ChangedDataset variation, is this correct? Why can't you just loop over the NTuples there and fiddle around with the info?

stwunsch commented 4 years ago

Fixed by #29