User Details
- User Since
- Jul 3 2018, 2:12 PM (327 w, 17 h)
Mon, Oct 7
Ready to be merged.
Thu, Oct 3
Yes, this ready to be merged, assuming that a check has been made to make sure that all of the histogram tests produce something sensible, i.e. they produce non-zero histograms and all of the entries are inside the bin ranges etc.
Tue, Sep 17
Changes look good.
Jul 4 2024
Changes look OK.
Jun 24 2024
Thanks for spotting the parameter ordering error. We also need to fix this for D119 for the Sim10 branch "R02-02-00-patches". What is the best way to proceed?
- Set number of events as 5,000 for the D0ToKspipi JSON tests.
- Fix ordering of LASS parameters in D0ToKspipi. Updated History file.
Jun 21 2024
Validation plots comparing the EvtGen model with the Belle/BaBar paper:
This can be merged. Good to have this code cleanup.
Jun 20 2024
Code looks OK. Thanks for the info about the thread safety aspects.
Jun 19 2024
Changes look good.
Jun 18 2024
Jun 17 2024
- Use EvtDalitzReso const correctness from gitlab commit c51a5f79.
Yes, adding the const functions would be good, then we can simplify retrieving the resonances for the total amplitude as suggested. What is the specific git commit I should use?
Jun 3 2024
May 14 2024
This code is ready to be merged. We will leave the ability to change the Sherpa configuration via decay file parameters as a future enhancement.
May 2 2024
Only spotted a few minor things, otherwise this is ready to be merged.
I have a few minor suggestions about adding more comments and have suggested a way that we can define the Sherpa parameters via the decay file.
Apr 16 2024
I agree that these changes look OK.
Apr 11 2024
This is ready to be merged.
Feb 5 2024
Changes look OK.
Feb 3 2024
Changes look OK, although you will need to update the History file to include the latest changes made by D100.
These changes look OK, although I think you need to "undo" the fix for EvtBcVHad since this has been done in D100 that has been recently merged (which also makes significant changes to other parts of that class). So just update this class to the version in the current master, otherwise I think you will get a merge conflict (for those other parts that have changed).
- Merge branch 'master' of ssh://phab.hepforge.org/source/evtgen
Dec 20 2023
I've attached a comparison of some of the angular distributions that was used for the Ds -> phi pi, phi -> pi pi pi0 decay (comparing this MR with those from the LHCb Sim meeting talk). It needs new decay angle calculations in the test program.
- Add cosThetaResNorm & cosBetaRes angles to testDecayModel. Used in added test file for Ds -> phi pi, phi -> pi pi pi0. Make EvtVector4R::cross() function constant.
Dec 7 2023
The updated formatting looks good. This is ready to be merged for the new release.
Nov 24 2023
Formatting and CI check look OK.
Nov 22 2023
Yes, this is OK.
Nov 16 2023
Update looks OK.
Nov 7 2023
Clang formatting and CI check looks OK, so proceed with organising the appropriate branches.
Nov 6 2023
All of the changes look OK. Just have a query about some K-matrix code that is commented out. Otherwise this can be merged.
Oct 16 2023
Changes look OK.
Oct 12 2023
Code updated to address the first set of review comments.
- Edits for 1st review round for D100
Oct 10 2023
Here are the JSON test distributions for the EvtBcVPPHad and EvtPhiDalitz model updates, which includes some comparisons with tests done by the previous LHCb code review:
Oct 9 2023
Changes look OK. This can be merged.
Aug 24 2023
Code looks good.
Aug 21 2023
Thanks for updating the interface, Tom. I agree that it will be easier to release this as a patch release based on R02-02-00.
Jul 13 2023
Feb 3 2023
- Set default cosTheta to 1 for EvtSLBaryonAmp spin density matrix.
- Undo max prob changes to EvtSSD_DirectCP since we are going to try a more general approach.
Jan 26 2023
Many of the zero nDof histograms in problem-histos.txt occur when there is only 1 bin filled (the others are empty), e.g. a constant PDG mass value with essentially zero width or a fixed momentum in a 2-body decay. So we could add a check for non-zero entries within the histogram range if nDof = 0 in addition to the chiSq/nDof number.
- We should increase the tensor probability from 10 to 20 for EvtSSD_DirectCP (e.g. for B+ -> f_2 pi).
Result of the command grep "chiSq/nDof" *.out | grep -v "pValue = 0.000" > problem-histos.txt
Dec 16 2022
I have added some example tests for this, and have also created a task where we could add more tests for other models in the future: https://phab.hepforge.org/T205
- Add tests for Bc -> J/psi pi pi0 & psi(2S) pi BC_VHAD modes.
- Update History file to mention D89.
Dec 12 2022
Looks like the documents contain all of the relevant information.
Nov 21 2022
The new updates using the mass ranges look OK.
Nov 17 2022
- Yes, increasing the safety factor to 10% is sensible.
This clever solution for finding ASqMax looks OK overall, so I am approving these changes.
Sep 21 2022
Sep 14 2022
The maximum probability is now set at 10, but we may want to revisit this model (reopening this task) in order to reduce the small but very long probability distribution tail that extends beyond 10.
This now appears to be fixed in the master branch.
Fixed in the master branch.
Fixed in the master branch.
Aug 14 2022
The BTOSLLALI max prob has been increased by 25%, to try to avoid the warnings, in commit b162af83 in the GitLab branch 1-complete-test-coverage-of-all-decay-models.
Undefined max probability was set to 10.0 in commit a36bf4cd on the GitLab branch 1-complete-test-coverage-of-all-decay-models. However, this does not address the problem of larger probability values that can occur in the high tail region, as mentioned previously.
This has been fixed in commit a184c939 on the 1-complete-test-coverage-of-all-decay-models branch.
This has been corrected in commit 4c5d60c3 on the 1-complete-test-coverage-of-all-decay-models branch.
Aug 5 2022
Jul 18 2022
Jul 5 2022
Jun 30 2022
Changes look OK. It is good to see unnecessary pointers being removed. Just have a few minor suggestions.
May 11 2022
Thanks for the clarification about the BF values chosen, Michal. I agree that all of the changes are OK. I will accept this revision after we finalise any required copyright/documentation changes to the scripts according to Tom's suggestions.
I'm not an expert on cmake but all of the changes look OK and we can add more checks/features as time goes on. Let us go ahead and add this to the repository.
May 10 2022
Thanks for these changes, Michal. Just to double-check, the PDG webpage has the following BFs for the psi(2S) and eta_c modes: 5.8e-4 and 8.0e-4. Do we need to use different values (6.2e-4 and 9.0e-4) to balance things owing to the pythia hadronisation modes? The pythia modes also have slightly larger BFs than before.
May 9 2022
Yes, the default lambda1 parameter quoted in the documentation should be -0.3, as given by Eq 39 (not Eq 24) on page 24 of https://arxiv.org/pdf/hep-ph/9805303.pdf . The BTOXSGAMMA model description in the guide has various other grammar problems that also need correcting. We could update the latex for the guide in the repository, or put the correct info in the paper draft (or do both).