Print

Print


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