- User Since
- Jul 3 2018, 2:12 PM (125 w, 4 d)
Fri, Nov 27
Thu, Nov 26
Mon, Nov 23
Fri, Nov 20
Wed, Nov 18
Fri, Oct 30
Hi @jback, many thanks for the changes and apologies for the delay in responding to them.
I think this looks good to go now.
Unless @kreps has any comments I think you can go ahead and "land" it onto master.
Oct 15 2020
Oct 13 2020
Oct 8 2020
Oct 6 2020
Oct 2 2020
Hi @jback, sorry it's taken me so long to review this.
You can find here some technical points.
I'll try to compare the physics against the referenced paper next week, although I guess this was checked during the LHCb review?
You mention also testing it using testDecayModel but I can't find the json file(s) for this model. Could you include them in the next revision?
Sep 21 2020
Sep 18 2020
Sep 17 2020
Sep 16 2020
Sep 14 2020
Sep 9 2020
I suppose at least 'worker' should imply someone being paid for their services.
I think the first 3 you list don't really work very well in this context (as you say). However, 'coordinator/worker' describes rather well the relationship of the objects here.
A few alternatives to replace 'master' along the same lines could be 'manager', 'director', or 'controller'.
But I'm struggling to come up with other good alternatives to 'worker', which could maybe still be taken to be a bit pejorative? Maybe 'runner'? Any other ideas?
Sep 3 2020
OK great, yes confirm the changes in rLAURA696d6d44a70e fix these concerns.
Sep 2 2020
Aug 21 2020
The CI build in the gitlab mirror of EvtGen is now working.
I've also committed the necessary changes to switch to using jsoncpp from LCG.
The build and test run successfully and you can see the details of the CI pipeline here:
I'm running everything in LCG_98 now because that includes the new versions of Photos and Tauola that include HepMC3 (btw it also includes EvtGen 2.0.0). There are now 4 builds:
and a test that uses the gcc9-dbg build. At present the test just runs your testDecayModel on one of the example json files you provided.
One thing I've noted is that the lack of a reference file does not result in the test exiting with a failure code, so that's something we need to look at.
The next step will be to analyse the contents of the push to see what files have changed and based on that work out what tests need to be run, etc.