[Hps-analysis] test pass1

Wed May 31 02:23:42 EDT 2017

Okay, thanks for the explanation.  It makes more sense now that I look at your PR.  You’re just setting alignment constant values for the tilt angle in case they are not read from the db.

On May 30, 2017, at 7:33 PM, Omar Moreno <omoreno1 at ucsc.edu<mailto:omoreno1 at ucsc.edu>> wrote:

On Tue, May 30, 2017 at 7:27 PM, McCormick, Jeremy I. <jeremym at slac.stanford.edu<mailto:jeremym at slac.stanford.edu>> wrote:
"I had to add the SVT title angles to the compact.”

I assume you meant “tilt” here.  I don’t understand what this means or why you had to do it.  Can you please explain further?  (I thought this was already supported by alignment constants being read from the compact.xml or reading this info from the conditions db.)

The tilt angles weren't in the DB or the compact.xml.  Since I have no if the default is reasonable, the tilt angles were added to both in order to ensure that it's being set properly.  It was set to 0 for both top and bottom.

On May 30, 2017, at 7:24 PM, Omar Moreno <omoreno1 at ucsc.edu<mailto:omoreno1 at ucsc.edu>> wrote:

Hi All,

Alignment constants are now in the DB.  The valid run range for these constants is 7219-8100.  For those interested, the collection ID in the 'svt_alignments' table 2982.  I tested locally using the following command

java -cp hps-jar org.hps.evio.EvioToLcio -d HPS-PhysicsRun2016-v5-3-fieldmap_globalAlign -l test_out -n 10 -r -x /org/hps/steering/recon/PhysicsRun2016FullRecon.lcsim -DoutputFile=test ../data/physics2016/evio/hps_007796.evio.0

and it looks like they are being loaded correctly:

2017-05-30 19:07:19 [INFO] org.lcsim.geometry.compact.converter.HPSTrackerBuilder <init> :: alignment conditions will be read from database
2017-05-30 19:07:19 [INFO] org.hps.conditions.database.AbstractConditionsObjectConverter getData :: loading conditions set...
id: 2398
name: svt_alignments
runStart: 7219
runEnd: 8100
tableName: svt_alignments
collectionId: 2982
updated: 2017-05-30 18:52:44.0
created: 2017-05-30 18:52:44.0
tag: null
createdBy: omoreno
notes: 2016 physics run alignment constants.

I had to add the SVT title angles to the compact.  The changes are on a branch and  I created a pull request for Jeremy to review and merge.  However, now that the alignment constants are in the DB, changes to the compact shouldn't change anything.

As for the svt_calibrations, it looks like only two non-specialized calibration runs were ever loaded into the database.  I'm verifying with Pelle and MattS that this is correct because looking through the logs, it looks like we took several calibration runs throughout the physics run.

As for bad channels,  I'm not going to put them into the DB for now.  If I do, then hits on those channels will be removed from the reconstruction and we haven't tested enough to understand what the repercussions would be.  Specifically, the hit clustering doesn't know what to do when a channel has a bad channel as a neighbor.  This may lead to strange behavior so I rather not risk it and wait for the next pass.

Best,

--Omar Moreno

On Sun, May 28, 2017 at 3:20 PM, McCormick, Jeremy I. <jeremym at slac.stanford.edu<mailto:jeremym at slac.stanford.edu>> wrote:
I only checked the svt_calibrations but all the other SVT conditions should be confirmed as well....
________________________________
From: hps-software at slac.stanford.edu<mailto:hps-software at slac.stanford.edu> <hps-software at slac.stanford.edu<mailto:hps-software at slac.stanford.edu>> on behalf of Rafayel Paremuzyan <rafopar at jlab.org<mailto:rafopar at jlab.org>>
Sent: Sunday, May 28, 2017 2:16:21 PM
To: Omar Moreno; Graf, Norman A.
Cc: hps-software; hps-analysis at jlab.org<mailto:hps-analysis at jlab.org>
Subject: Re: test pass1

Hi Omar,

It has been checked, at least Jeremy confirmed that 2016 calibs should have been used.
2017-05-27 02:29:51 [INFO] org.hps.conditions.database.AbstractConditionsObjectConverter getData :: loading conditions set...
id: 1397
name: svt_calibrations
runStart: 7566
runEnd: 99999
tableName: svt_calibrations
collectionId: 20
updated: 2016-02-26 14:42:17.0
created: 2016-02-26 14:42:17.0
tag: eng_run
createdBy: phansson
notes: Pedestals and noise. Loaded using SvtConditionsLoader.

However, I agree I should have been asking you to confirm if we are ready to make a recon svt wise.

Rafo

On 05/28/2017 12:01 AM, Omar Moreno wrote:
Hi Rafo,

I'm not sure what Jeremy fixed, but at this point It's probably a good idea to check if the correct SVT calibrations are being picked up for a given run.  Also, the alignment constants aren't in the DB yet so I'm not sure what its currently using. Just a couple of things to keep in mind.

--Omar Moreno

On Sat, May 27, 2017 at 1:54 PM, Graf, Norman A. <ngraf at slac.stanford.edu<mailto:ngraf at slac.stanford.edu>> wrote:
Thanks for the update Rafo.
Good to hear that the performance is back to what it was. Although we should still make some effort to speed things up beyond that.

I'm not sure how much I'll get done this weekend, but I'll start doing some checks as soon as I have some time.

Enjoy the weekend,
Norman
________________________________________
From: hps-software at slac.stanford.edu<mailto:hps-software at slac.stanford.edu> <hps-software at slac.stanford.edu<mailto:hps-software at slac.stanford.edu>> on behalf of Rafayel Paremuzyan <rafopar at jlab.org<mailto:rafopar at jlab.org>>
Sent: Saturday, May 27, 2017 1:03 PM
To: hps-software; hps-analysis at jlab.org<mailto:hps-analysis at jlab.org>
Subject: test pass1

Hi all,

After Jeremy fixed the code to choose the latest created condition,
the job running speed recovered. It is now similar to pass0 speed.

A new test pass1 is being processed now, some of files are already finished.

I would encourage users who deeply involved in the analysis to take a look,
and find bugs, that will help to have less bugs when we start pass1.
Some details on this test pass can be found in the link below
https://urldefense.proofpoint.com/v2/url?u=https-3A__confluence.slac.stanford.edu_pages_viewpage.action-3FpageId-3D223224540&d=DwIGaQ&c=lz9TcOasaINaaC3U7FbMev2lsutwpI4--09aP8Lu18s&r=J4PP6Zl8IyGHpsqWaKegORCYw8hoCHePTw5O95a5lqQ&m=-oTREucgtr2aq8Mjc-5PiQO0EuDGBKg5EyMRQLPu6fM&s=wdIpdSLdV1NSWzFUmCD5xcWqxazbiMiIdS2bbCkVQoM&e= <https://urldefense.proofpoint.com/v2/url?u=https-3A__confluence.slac.stanford.edu_pages_viewpage.action-3FpageId-3D223224540&d=DwMFaQ&c=lz9TcOasaINaaC3U7FbMev2lsutwpI4--09aP8Lu18s&r=0HDJrGO9TZQTE97J9Abt2A&m=NV9g2V6qlGckfePvJLd8oP54iibkmSCo6OaOmubdcGg&s=i6Y5fW6BYz8F8GAGLepncdBFsxRJEK69dgrl-ZT0648&e=>

Rafo

########################################################################
Use REPLY-ALL to reply to list

To unsubscribe from the HPS-SOFTWARE list, click the following link:
https://urldefense.proofpoint.com/v2/url?u=https-3A__listserv.slac.stanford.edu_cgi-2Dbin_wa-3FSUBED1-3DHPS-2DSOFTWARE-26A-3D1&d=DwIGaQ&c=lz9TcOasaINaaC3U7FbMev2lsutwpI4--09aP8Lu18s&r=J4PP6Zl8IyGHpsqWaKegORCYw8hoCHePTw5O95a5lqQ&m=-oTREucgtr2aq8Mjc-5PiQO0EuDGBKg5EyMRQLPu6fM&s=CFPeJM7rx-KusW6hBhxRLjpd5359OPYD04NoEitPLvQ&e= <https://urldefense.proofpoint.com/v2/url?u=https-3A__listserv.slac.stanford.edu_cgi-2Dbin_wa-3FSUBED1-3DHPS-2DSOFTWARE-26A-3D1&d=DwMFaQ&c=lz9TcOasaINaaC3U7FbMev2lsutwpI4--09aP8Lu18s&r=0HDJrGO9TZQTE97J9Abt2A&m=NV9g2V6qlGckfePvJLd8oP54iibkmSCo6OaOmubdcGg&s=ZSKfqkZiOo_Rn6SbwKeVWKYoFce8mK-ML0o5ke69RFI&e=>

########################################################################
Use REPLY-ALL to reply to list

To unsubscribe from the HPS-SOFTWARE list, click the following link:
https://urldefense.proofpoint.com/v2/url?u=https-3A__listserv.slac.stanford.edu_cgi-2Dbin_wa-3FSUBED1-3DHPS-2DSOFTWARE-26A-3D1&d=DwIGaQ&c=lz9TcOasaINaaC3U7FbMev2lsutwpI4--09aP8Lu18s&r=J4PP6Zl8IyGHpsqWaKegORCYw8hoCHePTw5O95a5lqQ&m=-oTREucgtr2aq8Mjc-5PiQO0EuDGBKg5EyMRQLPu6fM&s=CFPeJM7rx-KusW6hBhxRLjpd5359OPYD04NoEitPLvQ&e= <https://urldefense.proofpoint.com/v2/url?u=https-3A__listserv.slac.stanford.edu_cgi-2Dbin_wa-3FSUBED1-3DHPS-2DSOFTWARE-26A-3D1&d=DwMFaQ&c=lz9TcOasaINaaC3U7FbMev2lsutwpI4--09aP8Lu18s&r=0HDJrGO9TZQTE97J9Abt2A&m=NV9g2V6qlGckfePvJLd8oP54iibkmSCo6OaOmubdcGg&s=ZSKfqkZiOo_Rn6SbwKeVWKYoFce8mK-ML0o5ke69RFI&e=>

________________________________

Use REPLY-ALL to reply to list

To unsubscribe from the HPS-SOFTWARE list, click the following link:
https://urldefense.proofpoint.com/v2/url?u=https-3A__listserv.slac.stanford.edu_cgi-2Dbin_wa-3FSUBED1-3DHPS-2DSOFTWARE-26A-3D1&d=DwIGaQ&c=lz9TcOasaINaaC3U7FbMev2lsutwpI4--09aP8Lu18s&r=J4PP6Zl8IyGHpsqWaKegORCYw8hoCHePTw5O95a5lqQ&m=-oTREucgtr2aq8Mjc-5PiQO0EuDGBKg5EyMRQLPu6fM&s=CFPeJM7rx-KusW6hBhxRLjpd5359OPYD04NoEitPLvQ&e= <https://urldefense.proofpoint.com/v2/url?u=https-3A__listserv.slac.stanford.edu_cgi-2Dbin_wa-3FSUBED1-3DHPS-2DSOFTWARE-26A-3D1&d=DwMFaQ&c=lz9TcOasaINaaC3U7FbMev2lsutwpI4--09aP8Lu18s&r=0HDJrGO9TZQTE97J9Abt2A&m=NV9g2V6qlGckfePvJLd8oP54iibkmSCo6OaOmubdcGg&s=ZSKfqkZiOo_Rn6SbwKeVWKYoFce8mK-ML0o5ke69RFI&e=>

________________________________

Use REPLY-ALL to reply to list

To unsubscribe from the HPS-SOFTWARE list, click the following link:
https://urldefense.proofpoint.com/v2/url?u=https-3A__listserv.slac.stanford.edu_cgi-2Dbin_wa-3FSUBED1-3DHPS-2DSOFTWARE-26A-3D1&d=DwIGaQ&c=lz9TcOasaINaaC3U7FbMev2lsutwpI4--09aP8Lu18s&r=J4PP6Zl8IyGHpsqWaKegORCYw8hoCHePTw5O95a5lqQ&m=-oTREucgtr2aq8Mjc-5PiQO0EuDGBKg5EyMRQLPu6fM&s=CFPeJM7rx-KusW6hBhxRLjpd5359OPYD04NoEitPLvQ&e= 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.jlab.org/pipermail/hps-analysis/attachments/20170531/43f63737/attachment-0001.html>