Print

Print


Hi,

So I just checked in a big scary commit to cleanup the tracking code.

Here's what I did:

1) Moved all classes not having to do with core tracking reconstruction to other packages (mostly analysis and users).  There weren't that many of them but doing so cleaned up the base tracking package significantly.

2) Renamed all classes that had an unnecessary "HPS" of them.  As far as I could tell, this was done semi-randomly.  The only one that survived in the core package was "HPSTrack" which I left because I didn't have a better name for it.

3) Added "TODO" and "FIXME" comments to classes, mostly about missing class documentation, proposed cleanup, functionality questions, etc.

4) Added a package org.hps.readout.svt where I put readout-related classes, that are used mostly by the EVIO data converters in the evio module.  This package at the moment contains FpgaData, SVTData, and SimpleSvtReadout.  Classes that are more about readout than reconshould go here from now on.  I think it is fine to have it in the tracking module along with the recon. 

5) Formatted all the source code according to my Formatter convention in Eclipse.

6) Cleaned up class comments and author statements.

7) Updated all Java and steering files external to the tracking module within the trunk to work with the new changes.

The full trunk build is working fine now for me after this.  Let me know if you can't get it working.

Have a good weekend!

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