Okay, I misunderstood. I think that everything in ecal-recon needs to be ported. But you might check with Sho on this and also see what Drivers are called from steering files. > On Apr 29, 2014, at 11:18 PM, "Gabriel CHARLES" <[log in to unmask]> wrote: > > Hi, > > I was indeed refering only to ecal.recon, thank you Jeremy for > mentionning this point. > > I agree with you on the steps to be done, nevertheless, I will be > able to attend the next software meeting only in three weeks from now > because of vacations and diverse personnal involvment. I think that > meanwhile I should keep working on the modification of the database. > > That is why if I could have a list of not needed files in recon.ecal > it would be nice, so please let me know which files are not used there. > > Of course, I am going to change them one by one but I am not sure the > method should be explained here as we rely on each other. > > Best regards, > --- > Gabriel CHARLES > Institut de Physique Nucléaire d'Orsay > >> On Tue, 29 Apr 2014 16:58:34 -0700, McCormick, Jeremy I. wrote: >> Hi, Gabriel. >> >> Shouldn't we only worry about modifying recon code once ecal-recon >> and ecal-readout-sim are working properly with the new conditions >> system on the branches? I believe there should be at least one >> presentation from you to the software group that proves this is the >> case before anything should be touched in recon. In fact, it would >> be >> nice to prove to everyone that the results of the readout simulation >> are exactly the same as before. >> >> I think we also need to establish that the performance of the new >> code is acceptable. It is not particularly well optimized in the >> conditions module, so we should probably figure out exactly how much >> slower it is than Sho's (relatively fast) implementation and improve >> the "hot spots" accordingly before forcing everyone to use it. >> >> --Jeremy >> >> -----Original Message----- >> From: [log in to unmask] >> [mailto:[log in to unmask]] On Behalf Of Gabriel CHARLES >> Sent: Tuesday, April 29, 2014 3:01 PM >> To: hps-software >> Subject: To everyone writing hps.org.recon >> >> Hello everybody, >> >> As you may know I am modifying hps-java for it to use the database >> based conditions. I would like to change only the code that should be >> updated. So could you indicate me which files in org.hps.recon are >> not >> used, please? It will also permit some cleaning on the same time and >> make the work more efficient. >> >> Thanks a lot, >> -- >> Gabriel CHARLES >> Institut de Physique Nucléaire d'Orsay >> >> >> ######################################################################## >> 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 > ######################################################################## 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