LISTSERV mailing list manager LISTSERV 16.5

Help for HPS-SOFTWARE Archives


HPS-SOFTWARE Archives

HPS-SOFTWARE Archives


HPS-SOFTWARE@LISTSERV.SLAC.STANFORD.EDU


View:

Message:

[

First

|

Previous

|

Next

|

Last

]

By Topic:

[

First

|

Previous

|

Next

|

Last

]

By Author:

[

First

|

Previous

|

Next

|

Last

]

Font:

Proportional Font

LISTSERV Archives

LISTSERV Archives

HPS-SOFTWARE Home

HPS-SOFTWARE Home

HPS-SOFTWARE  October 2015

HPS-SOFTWARE October 2015

Subject:

Re: Ecal Geometry

From:

Nathan Baltzell <[log in to unmask]>

Reply-To:

Software for the Heavy Photon Search Experiment <[log in to unmask]>

Date:

Thu, 1 Oct 2015 18:57:30 -0400

Content-Type:

text/plain

Parts/Attachments:

Parts/Attachments

text/plain (193 lines)

Hi Everyone,

One option could be to develop the new model by modifying the existing HPSEcal3
in lcsim (without checking it in).  This may make immediate testing easier since you
don't have to worry about modifying the various places in hps-java that have hardcoded
references to the existing HPSEcal3 detector model, nor bother with understanding what
needs to change when you move it from existing in lcsim to hps-java.  Then, when finished
testing, maybe Jeremy or another expert can help convert it over to existing in hps-java
with a new model name.

The proposed alternative, starting with a new ecal geometry model in hps-java
instead of lcsim, would be much easier if a working ecal example existed.  Currently
we're basing our understanding of how the software works on the existing
implementation of ecal geometry model in lcsim, HPSEcal3, while just the
compact.xml and lcdd reside in hps-java.  Ok, maybe this is really easy to move
to hps-java, but it's not obvious.

-Nathan



On Oct 1, 2015, at 6:09 PM, Holly Vance <[log in to unmask]> wrote:

> Dear Jeremy,
> 
> Annie and I modified the HPSEcal3 geometry so that it can be backwards compatible with everything that exists as well as accept new values of shifts in x and y and z if they are specified. 
> 
> I have already tested these changes (which of course required modifying the HPSEcal3 detector in detector/converter/compact/... as well as the one in geometry/compact/converter/lcdd/...
> 
> I am also aware that any new detector needs to access geometry/subdetector/....
> 
> So I see the directories in the hps-java/detector-model are the same. Are you suggesting we simply copy the HPSEcal3 necessary files to this file location? We could make a new HPSEcal4, but the 3 is backwards compatible by simply accepting new values. 
> 
> Please advise where we are to commit changes. Thanks!
> 
> -Holly and Annie
> 
> 
> 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
> 
> 
> 
> 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

Top of Message | Previous Page | Permalink

Advanced Options


Options

Log In

Log In

Get Password

Get Password


Search Archives

Search Archives


Subscribe or Unsubscribe

Subscribe or Unsubscribe


Archives

June 2024
May 2024
April 2024
March 2024
February 2024
January 2024
December 2023
November 2023
October 2023
September 2023
August 2023
July 2023
June 2023
May 2023
April 2023
March 2023
February 2023
January 2023
December 2022
November 2022
October 2022
September 2022
August 2022
June 2022
April 2022
March 2022
February 2022
January 2022
December 2021
November 2021
October 2021
September 2021
August 2021
July 2021
June 2021
May 2021
April 2021
March 2021
February 2021
January 2021
December 2020
November 2020
October 2020
September 2020
August 2020
July 2020
June 2020
May 2020
April 2020
March 2020
February 2020
January 2020
December 2019
November 2019
October 2019
September 2019
August 2019
July 2019
June 2019
May 2019
April 2019
March 2019
February 2019
January 2019
December 2018
November 2018
October 2018
September 2018
August 2018
July 2018
June 2018
May 2018
April 2018
March 2018
February 2018
January 2018
December 2017
November 2017
October 2017
September 2017
August 2017
July 2017
June 2017
May 2017
April 2017
March 2017
February 2017
January 2017
December 2016
November 2016
October 2016
September 2016
August 2016
July 2016
June 2016
May 2016
April 2016
March 2016
February 2016
January 2016
December 2015
November 2015
October 2015
September 2015
August 2015
July 2015
June 2015
May 2015
April 2015
March 2015
February 2015
January 2015
December 2014
November 2014
October 2014
September 2014
August 2014
July 2014
June 2014
May 2014
April 2014
March 2014
February 2014
January 2014
December 2013
November 2013
October 2013
September 2013
August 2013
July 2013
June 2013
May 2013
April 2013
March 2013
February 2013
January 2013
December 2012
November 2012
October 2012
September 2012
August 2012
July 2012
June 2012
May 2012
April 2012
March 2012
February 2012
January 2012
December 2011
November 2011
October 2011
September 2011
August 2011
July 2011
June 2011
May 2011
April 2011
March 2011

ATOM RSS1 RSS2



LISTSERV.SLAC.STANFORD.EDU

Secured by F-Secure Anti-Virus CataList Email List Search Powered by the LISTSERV Email List Manager

Privacy Notice, Security Notice and Terms of Use