On Thu, Oct 1, 2015 at 4:38 PM, McCormick, Jeremy I. <[log in to unmask]> wrote:

We might as well just make a new model in hps-java as it was planned that any new development on these ECAL geometry classes would occur in hps-java and not lcsim, as we've now done with the tracker. This means people can work on this geometry code without needing access to the lcsim SVN, which most people in the collaboration do not have (it is in a separate SVN repository from hps-java with different user accounts).

So I would suggest making a new "HPSEcalEngRun2015" class in trunk/detector-model based on HPSEcal3, rather than changing the existing class in lcsim, which is known to work well and should essentially be frozen, as there has been an enormous amount of data generated using it. The slightest deviation there in the output geometry compared to the existing version could cause very nasty bugs in our software, so I believe it would be best to avoid this entirely by not changing it.

Also, one quirk of our framework is that you need two classes that implement the same geometry model, one for the recon "Java" geometry and the other which outputs the LCDD files for the simulation. If it is not clear to you how this works, please let me know and I'll point you to examples of each. I have generally worked on the simulation output model first, as you can easily visualize it in Geant4.

--Jeremy

-----Original Message-----
From: [log in to unmask] [mailto:[log in to unmask]]
Sent: Thursday, October 01, 2015 1:24 PM
To: McCormick, Jeremy I.

Cc: Holly Vance; hps-software
Subject: RE: Ecal Geometry

Holly,

I think it is better to have separate parameters for the shifts and don't mix them with them beam gap parameters. I can do this changes too.

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
>
>
> --
>
>
>
>
>
> ######################################################################
> ##
> 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 01PnUDOif) is spam:

Spam: https://www.spamtrap.odu.edu/canit/b.php?i=01PnUDOif&m=4bfb8e90c713&t=20151001&c=s

Not spam: https://www.spamtrap.odu.edu/canit/b.php?i=01PnUDOif&m=4bfb8e90c713&t=20151001&c=n

Forget vote: https://www.spamtrap.odu.edu/canit/b.php?i=01PnUDOif&m=4bfb8e90c713&t=20151001&c=f
------------------------------------------------------
END-ANTISPAM-VOTING-LINKS