Print

Print


Hi,

I think perhaps we should cleanup some of the names of the classes for the clustering.

Here is my proposal:

EcalClusterer -> LegacyEcalClusterer <= old cluster maker

EcalClusterIC -> EcalClusterer <= current cluster maker

EcalClusterICBasic -> BasicEcalClusterer <= current cluster maker (basic version)

I think this will make it clearer what these classes actually do.  The last two are not actually "Cluster" implementations as it seems from reading the current names.

Any thoughts/objections?

I can cleanup all relevant steering files if I make this change but I thought it should be discussed first, as I don't want to step on any toes.  :)

--Jeremy

########################################################################
Use REPLY-ALL to reply to list

To unsubscribe from the HPS-SOFTWARE list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=HPS-SOFTWARE&A=1