Hi Nathan,
actually you are right, there are not so many differences related to ecal
- I was misled since many constant definitions moved their line in the
lcdd list.
However, I made a 1:1 check of these constants, and a few of them which are
present in the original version of lcdd (I'm referring to
HPS-EngRun2015-Nominal-v1, for instance) are missing in the recreated one.
They are (not sure I spotted them all):
<constant name="sensorLength" value="98.33" />
<constant name="z01b" value="153.815" />
<constant name="dipoleMagnetHeight" value="1000.0" />
<constant name="ecal_z" value="80.0" />
<constant name="sensorWidth" value="38.3399" />
<constant name="PI" value="3.14159265359" />
<constant name="moduleLength" value="100.0" />
not sure if they are really useful... but I find it strange they had been
removed in the new converter. Is this ok?
I cannot say if the reverse is also valid, that is, if there are new
constants which were not present on the original file (I'll check).
Moreover, in the original lcdd I find this stanza
<physvol>
<volumeref ref="tracking_volume" />
<positionref ref="identity_pos" />
<rotationref ref="identity_rot" />
</physvol>
while in the recreated one I read
<positionref ref="tracking_region_pos" />
is everything normal?
Thanks, cheers
Alessandra
On Mon, 11 Jul 2016, Nathan Baltzell wrote:
> Hi Alessandra,
>
> I think if the compact.xml is unchanged, then the lcdd should be
> identical for the calorimeter. Unless the HPSEcal3 model code
> was changed (the recent updates were only in HPSEcal4 I thought).
> What are the differences you see?
>
> -Nathan
>
>
> On Jul 11, 2016, at 5:34, Alessandra Filippi <[log in to unmask]> wrote:
>
>> Hi Jeremy,
>> I think I found the issue: the new GEOMCONVERTER file is now to be found in hps_trunk/detector-model and not in lcsim_trunk/detector-framework I used to set. Indeed, when I run from lcsim_trunk the command
>> java -jar java -jar ./detector-framework/target/lcsim-detector-framework-3.7-SNAPSHOT-bin.jar
>> and get the window with the input/output filenames to be filled in, I get the error I mentioned last Friday.
>>
>> Also, when I run from hps_trunk/detector-data the command
>> java -cp ../distribution/target/hps-distribution-3.10-SNAPSHOT-bin.jar org.lcsim.geometry.compact.converter.Main -o lcdd detectors/HPS-EngRun2015-Nominal-v1/compact.xml detectors/HPS-EngRun2015-Nominal-v1/HPS-EngRun2015-Nominal-v1_NEW.lcdd
>> everything runs smoothly.
>>
>> However, if I compare the old lcdd with the new one I get, I find some differences (I started from a standard v1/compact.xml file, without applying any change to the geometry, just to compare the lcdd files).
>> Should I worry about? Is this due to the changes in Ecal geometry, so in principle this recreated lcdd should be more compliant to out new hps_java code? (I'll try to run some tests to check).
>> thanks, cheers
>> Alessandra
>>
>>
>>
>>
>>
>>
>>
>>
>>
>>
>> On Fri, 8 Jul 2016, McCormick, Jeremy I. wrote:
>>
>>> Hi, Allessandra.
>>>
>>> Can you put exactly what command you are running?
>>>
>>> I’ll try to reproduce the issue….
>>>
>>> Thanks.
>>>
>>> —Jeremy
>>>
>>> On Jul 8, 2016, at 3:04 AM, Alessandra Filippi <[log in to unmask]> wrote:
>>>
>>>> Hi all,
>>>> I'm tryng to use again the lccim GeomConverter to produce some new geometries, but I get this sort of error:
>>>>
>>>>
>>>> Exception in thread "main" java.lang.RuntimeException: java.lang.RuntimeException: Subdetector is not an instanceof LCDDSubdetector.
>>>> at org.lcsim.geometry.compact.converter.Main.main(Main.java:107)
>>>> Caused by: java.lang.RuntimeException: Subdetector is not an instanceof LCDDSubdetector.
>>>> at org.lcsim.geometry.compact.converter.lcdd.LCDDDetector.writeSubdetectors(LCDDDetector.java:216)
>>>> at org.lcsim.geometry.compact.converter.lcdd.LCDDDetector.writeLCDD(LCDDDetector.java:133)
>>>> at org.lcsim.geometry.compact.converter.lcdd.Main.convert(Main.java:63)
>>>> at org.lcsim.geometry.compact.converter.Main.run(Main.java:115)
>>>> at org.lcsim.geometry.compact.converter.Main.main(Main.java:103)
>>>>
>>>>
>>>>
>>>> I suppose I should not have updated the lcsim trunk (now I'm running lcsim-detector-framework-3.7-SNAPSHOT-bin.jar), but I cannot recall which
>>>> org.lcsim version to retrieve, that is still compliant with our code.
>>>> Can you please help?
>>>> thanks, cheers
>>>> Alessandra
>>>>
>>>> ########################################################################
>>>> 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
