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
|