Print

Print


Yes, that error was fixed on trunk last night.  tpass6 is the result.



On Mar 22, 2016, at 13:16, Bradley T Yale <[log in to unmask]> wrote:

> I get the same error when running standard MC readout as well.
> 
> -Brad
> 
> ________________________________________
> From: [log in to unmask] <[log in to unmask]> on behalf of McCormick, Jeremy I. <[log in to unmask]>
> Sent: Monday, March 21, 2016 8:26 PM
> To: Holly SzumilaVance; Nathan Baltzell
> Cc: hps-software
> Subject: RE: hps-java release
> 
> Hi,
> 
> There's two problems here I can see (sorry should have caught this in testing).
> 
> In EcalTimeWalk.java from ecal-recon around line 39, you should not do this...
> 
> private static final double[] par = {
>            DatabaseConditionsManager
>                    .getInstance()
>                    .getCachedConditions(
>                            org.hps.conditions.ecal.EcalTimeWalk.class,
>                            "ecal_time_walk").getCachedData().getP0(),
>        ...
> }
> 
> Because it is static code that may execute before the conditions system is initialized.  This type of code should only really go into the detectorChanged method of Driver.
> 
> But I think the real problem is that you're not using the right class which is actually EcalTimeWalkCollection and not EcalTimeWalk, because the manager returns collections and not single objects.
> 
> You want to do something like this in EcalRawConverter:
> 
> // parameters array class var
> private double[] pars = new double[5];
> 
> public void detectorChanged(Detector detector) {
>    EcalTimeWalk timeWalk = getConditionsManager().getCachedConditions(EcalTimeWalkCollection.class, "ecal_time_walk").getCachedData().get(0);
>    pars[0] = timeWalk.getP0();
>    // set rest of parameters
> }
> 
> Then you can pass the pars array to the time walk correction function.
> 
> --Jeremy
> 
> -----Original Message-----
> From: Holly SzumilaVance [mailto:[log in to unmask]]
> Sent: Monday, March 21, 2016 5:03 PM
> To: Nathan Baltzell
> Cc: McCormick, Jeremy I.
> Subject: Re: hps-java release
> 
> Looks like time walk from the database isn't working. I had this working on Friday. Maybe my copy is different. I'll check.
> 
>> On Mar 21, 2016, at 7:50 PM, Nathan Baltzell <[log in to unmask]> wrote:
>> 
>> I tried 3.7 and hit this error:
>> 
>> 2016-03-21 19:44:39 [INFO] org.hps.evio.EvioToLcio run :: first
>> physics event time: 1431855398 - Sun May 17 05:36:38 EDT 2015 Exception in thread "main" java.lang.ExceptionInInitializerError
>>   at org.hps.recon.ecal.EcalRawConverter.HitDtoA(EcalRawConverter.java:602)
>>   at org.hps.recon.ecal.EcalRawConverterDriver.process(EcalRawConverterDriver.java:477)
>>   at org.lcsim.util.Driver.doProcess(Driver.java:261)
>>   at org.lcsim.util.Driver.processChildren(Driver.java:271)
>>   at org.lcsim.util.Driver.process(Driver.java:187)
>>   at org.lcsim.util.DriverAdapter.recordSupplied(DriverAdapter.java:74)
>>   at org.lcsim.job.JobControlManager.processEvent(JobControlManager.java:826)
>>   at org.hps.evio.EvioToLcio.run(EvioToLcio.java:605)
>>   at org.hps.evio.EvioToLcio.main(EvioToLcio.java:98)
>> Caused by: org.lcsim.conditions.ConditionsManager$ConditionsSetNotFoundException: No converter registered for type: org.hps.conditions.ecal.EcalTimeWalk
>>   at org.lcsim.conditions.ConditionsManagerImplementation.getCachedConditions(ConditionsManagerImplementation.java:92)
>>   at org.hps.recon.ecal.EcalTimeWalk.<clinit>(EcalTimeWalk.java:39)
>> 
>> Holly, do you know what is up with this?
>> 
>> 
>> 
>>> On Mar 21, 2016, at 7:12 PM, "McCormick, Jeremy I." <[log in to unmask]> wrote:
>>> 
>>> Hi,
>>> 
>>> I made the hps-java 3.7 release which uses the lcsim 3.4 version.
>>> 
>>> I didn't deploy the site as there are some file system issues I need
>>> to sort out first.  (I may be able to do it manually later for this
>>> version.)
>>> 
>>> It should show up as the current hps-distribution version here.
>>> 
>>> http://srs.slac.stanford.edu/nexus/index.html#nexus-search;quick~hps-
>>> distribution
>>> 
>>> Please download and test this.  It is based on the trunk from around 3:30 PST today.
>>> 
>>> Thanks.
>>> 
>>> --Jeremy
>>> 
>>> #####################################################################
>>> ###
>>> 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