Hi Jeremy,

I 100% agree that this makes more sense. 

There are a few classes of codes where I've added IC to the end to indicate that they work with this clustering. We may think to merge these codes and make sure everything works. It's in the HPSEcalClusterIC class and the HPSReconParticle and ReconParticle (I think Omar's codes). 

~Holly

On Thu, Oct 16, 2014 at 3:39 PM, McCormick, Jeremy I. <[log in to unmask]> wrote:
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


--
BEGIN-ANTISPAM-VOTING-LINKS
------------------------------------------------------

NOTE: This message was trained as non-spam.  If this is wrong,
please correct the training as soon as possible.


Teach CanIt if this mail (ID 01N3TDVwa) is spam:

Spam:        https://www.spamtrap.odu.edu/canit/b.php?i=01N3TDVwa&m=4f519760aa3f&t=20141016&c=s

Not spam:    https://www.spamtrap.odu.edu/canit/b.php?i=01N3TDVwa&m=4f519760aa3f&t=20141016&c=n

Forget vote: https://www.spamtrap.odu.edu/canit/b.php?i=01N3TDVwa&m=4f519760aa3f&t=20141016&c=f
------------------------------------------------------
END-ANTISPAM-VOTING-LINKS




--
Respectfully,
Holly 



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