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
Right Norman, this is important to merge to the 4.0.1!!RafoOn 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?
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]>
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
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