Print

Print


Hello Maurik,


Thanks for your prompt response, and apologies for not being more clear in my original message. Once we have solutions to the situations which throw Exceptions we would only process the file partitions which crashed.


Norman


________________________________
From: Maurik Holtrop <[log in to unmask]>
Sent: Friday, January 19, 2018 6:18 PM
To: Graf, Norman A.
Cc: McCormick, Jeremy I.; Rafayel Paremuzyan; hps-software
Subject: Re: Pull request #285

Hello Norman,

I donít think we need to do a full new pass for a few events where the current version of the software throws an exception. Canít we simply only re-process the files that failed before, once we have a solution on how to deal with those few events?

Unless of course these exceptions indicate that we are doing something something wrong and the fix changes the output in a real way.

Best,
Maurik


On Jan 19, 2018, at 9:12 PM, Graf, Norman A. <[log in to unmask]<mailto:[log in to unmask]>> wrote:

Hello Jeremy,

Your proposal sounds fine to me. We should merge the detector into the 4.0.1 release without any other code changes.

There are two outstanding issues (dphi and bordered band matrix) which throw uncaught Exceptions. But these are rare and have not yet been resolved (Matt, do you have any updates?). As inelegant as it may sound, I am proposing that we make another pass once we have patches for those two issues. I do not want to hold up production reconstruction of the 1.5mm data any longer.

Norman


________________________________
From: [log in to unmask]<mailto:[log in to unmask]> <[log in to unmask]<mailto:[log in to unmask]>> on behalf of McCormick, Jeremy I. <[log in to unmask]<mailto:[log in to unmask]>>
Sent: Friday, January 19, 2018 3:37 PM
To: Rafayel Paremuzyan
Cc: hps-software
Subject: Re: Pull request #285

Hi,

I moved this conversation to the software listÖ

If we really need to use the exact same version of recon from 4.0.1 with these updates, I propose the following:

- create a branch 4.0.2-pre from the 4.0.1 tag

- merge in the new detector update to this branch (can use the commit hash from the PR if the branch was deleted)

- merge in any other fixes/updates for 4.0.2 (are there any?)

- change hps-java version to 4.0.2

- make a 4.0.2 git tag from the branch

- build the java jars from the tag

Then we can run the 2015 pass from this jar which will contain the 4.0.1 recon with the new detector data.

I donít think we really need a proper release for 4.0.2, though if needed it could be done from the command line if required (the default Jenkins build uses master for releases).

óJeremy

On Jan 19, 2018, at 9:00 AM, Rafayel Paremuzyan <[log in to unmask]<mailto:[log in to unmask]>> wrote:

Right Norman,

this is important to merge to the 4.0.1!!

Rafo


On 01/19/2018 11:54 AM, Graf, Norman A. wrote:
There have been changes to the master which will change the results. We should merge this detector into the branch that was used for 4.0.1 and tag that as 4.0.2.

Norman

________________________________
From: McCormick, Jeremy I.
Sent: Thursday, January 18, 2018 2:32 PM
To: Graf, Norman A.
Cc: Maurik Holtrop; Graham, Mathew Thomas; Nathan Baltzell; Rafayel Paremuzyan
Subject: Re: Pull request #285

I can make a 4.0.2 release with this detector but Iím not sure that the physics results have not changed since 4.0.1 was tagged - can someone confirm/deny?

On Jan 18, 2018, at 2:20 PM, Graf, Norman A. <[log in to unmask]<mailto:[log in to unmask]>> wrote:

Hello Maurik,

Thanks for taking care of this. This will also need to be included in the production branch (4.0.2?). Could you please coordinate with Jeremy and Rafo?

Thanks,
Norman


________________________________
From: Maurik Holtrop <[log in to unmask]<mailto:[log in to unmask]>>
Sent: Thursday, January 18, 2018 2:19 PM
To: Graf, Norman A.
Cc: Graham, Mathew Thomas; Nathan Baltzell; McCormick, Jeremy I.; Rafayel Paremuzyan
Subject: Re: Pull request #285

Hi Norman,

I approved the changes and then merged the code and deleted the branch.
Sorry for the delay. Thank you for pointing me to this.

Best,
Maurik


On Jan 18, 2018, at 2:20 PM, Graf, Norman A. <[log in to unmask]<mailto:[log in to unmask]>> wrote:

Hi Matt, Maurik,

What's holding up my pull request #285? This is on the critical path for running pass8 over the 1.5mm SVT data. I've tested it and Alessandra has signed off on it. Please let me know what you are waiting on.

Thanks,
Norman





________________________________
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