`Masker` is a duplicate of `Subtractor`

HEP-FCC / heppy

[deprecated] A python analysis framework for high energy physics

Other

11 stars 32 forks source link

`Masker` is a duplicate of `Subtractor` #78

Open vvolkl opened 6 years ago

vvolkl commented 6 years ago

There is not much code duplication because they are small clases, but it could lead to confusion nonetheless. In FCChhAnalyses, only Subtractor is used, so I propose to remove Masker.

cbernet commented 6 years ago

Subtractor has been added after Masker, which is more general. So we should instead keep Masker, and move Subtractor to the FCChh analysis packages. Could you please let me know when i can do that?