Print

Print


Hi,

Can I remove this class?  (I think you added it?)

src/main/java/org/hps/recon/ecal/HPSCalorimeterHit.java

It doesn't seem to be doing anything special, and I would like to merge it into the LCSim BaseCalorimeterHit class.

I'll make sure everything still works properly of course with the changes.

The problem in general with using "custom" classes that persist to LCIO is that when they are read back, LCSim will
only see the CalorimeterHit collection.  So any code that is written expecting HPSCalorimeterHit objects will then 
mysteriously fail.

It would be preferable to do the same thing for clusters too but it is more complicated as there is added functionality
not actually present in the LCSim class, so that requires some more thought.

--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