Hi All, It should still have a different name. Otherwise, there will certainly be problems. Also, as jeremy points out, a new lcdd converter needs to be made if you ever want to simulate this new detector. Tim > On Oct 1, 2015, at 1:22 PM, [log in to unmask] wrote: > > HI Jeremy, Holly, > > Since we are not changing the ECal geometry a lot, just adding parameters. > May be it is a good idea to keep everything as it is, and make the changes > so, that when the new parameters are 0, it is behaves just like used to > be, without this shifts and fan out parameters. > > I think I can figure out how to do that for both classes. > > Annie. > > >> Don't change the existing HPSEcal3 detector model, as this could have some >> unintended side effects like causing a mismatch between the simulated and >> recon geometry models. >> >> You should instead make new classes in trunk/detector-model and we can go >> from there. You'll have a lot more freedom to change things this way by >> making a new set of classes. >> >> If you want to copy and paste the existing HPSEcal3 detector model into a >> new class in hps-java that's a reasonable place to start I think. >> >> -----Original Message----- >> From: Holly Vance [mailto:[log in to unmask]] >> Sent: Thursday, October 01, 2015 12:26 PM >> To: McCormick, Jeremy I. >> Cc: hps-software >> Subject: Re: Ecal Geometry >> >> We are making the changes we discussed in the workshop last week. These >> changes include (at the moment): x,y,z shifts to the top and bottom >> crystals in the Ecal. Ani is working on alterations to how we can space >> the crystals and angling. >> >> At the moment, the only place this happens is in HPSEcal3 in LCSIM. I see >> that this can be done in a similar manner in hps-java/detector-model. So I >> think I can copy the relevant code changes to there? >> >> On Thu, Oct 1, 2015 at 3:05 PM, McCormick, Jeremy I. >> <[log in to unmask]> wrote: >> >> >> What changes are you trying to make to HPSEcal3? Can you send me a diff? >> (output from 'svn diff' would be fine) >> >> It would be preferable to move development over to the hps-java >> trunk/detector-model package as we've done for the SVT detector models, >> as you won't need to checkin anything to lcsim then. >> >> The package names were left the same so it should be clear what needs to >> go where. >> >> We should probably make something new like an "HPSEcalEngRun2015" model >> where you can make siginificant changes to how the model works based on >> survey from the Eng Run etc., without any chance of affecting the >> existing classes. >> >> >> -----Original Message----- >> From: [log in to unmask] >> [mailto:[log in to unmask]] On Behalf Of Holly Vance >> Sent: Thursday, October 01, 2015 12:01 PM >> To: hps-software >> Subject: Ecal Geometry >> >> Hello, >> >> I've created a new HPSEcal3 detector in LCSIM. I do not currently have >> credentials to commit to this repository-how can I proceed to commit? Or >> can we branch the LCSIM trunk while we develop this detector further? >> >> Thanks, >> 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 >> >> >> ######################################################################## >> 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 01PnT6oUJ) is spam: >> Spam: >> https://www.spamtrap.odu.edu/canit/b.php?i=01PnT6oUJ&m=a4845caedb9a&t=20151001&c=s >> Not spam: >> https://www.spamtrap.odu.edu/canit/b.php?i=01PnT6oUJ&m=a4845caedb9a&t=20151001&c=n >> Forget vote: >> https://www.spamtrap.odu.edu/canit/b.php?i=01PnT6oUJ&m=a4845caedb9a&t=20151001&c=f >> ------------------------------------------------------ >> END-ANTISPAM-VOTING-LINKS >> >> >> >> >> >> ######################################################################## >> 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 ######################################################################## 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