User Details
- User Since
- Jul 3 2018, 2:12 PM (303 w, 2 d)
Wed, Apr 24
Tue, Apr 23
Thanks @abudinen, looks good to me
Mon, Apr 22
Many thanks @abudinen, looks great, please go ahead.
Tue, Apr 16
Great, thanks for checking it over @jback!
Mon, Apr 15
Looks good to me. However, I think it would be good to have @jback check the changes introduced since his review on EvtGen!9 last Monday, in particular the neverRadCorr fix. The changes introduced in the last week can be seen here:
https://gitlab.cern.ch/evtgen/evtgen/-/compare/7e56b47604d68f89d00842409aeab2f9f1414b04...9-replace-photos-flag
Fri, Apr 12
Great, many thanks @abudinen! I have no further comments following the thorough review already conducted in EvtGen!10. Please go ahead and land this.
Thu, Apr 11
Thanks both, I'll go ahead and land this. Unusually I'm going to use the "merge" strategy in this case rather than the usual "squash" strategy, since I think having the intermediate commits in the history will make it simpler to restore any individual models in the future if we need to do so.
Rebase on master
Wed, Apr 10
Thanks @abudinen, I've added the lists you requested to the History file.
- Add lists of removed models and EvtGenBase classes
Looks good, many thanks!
Tue, Apr 9
Mar 5 2024
- Update History.md
Mar 4 2024
Feb 19 2024
Updated History.md after landing of D107
Rebase after landing of D100
Feb 5 2024
Issues with PROPSLPOLE turn out to be more involved. The model appears to be doing things it shouldn't. The SLPOLE model should be used in preference. As such the decision is to remove PROPSLPOLE.
Jan 11 2024
The implementation in D101 has caused problems with being able to run the python example GenFit3pi.py because to be able to use our classes in python they need to have dictionaries generated.
As such, I'm going to revert the changes from landing D101 and make new commits that only apply the following parts of those changes:
- Removal of the deprecated ClassImp macro in all source files
- The improvements to the CMake modules to avoid using old-style CMake stuff like file-level include_directories, etc.
Dec 20 2023
Thanks John, this looks great!
Once I'm back in the office in January I'll run it through the CI and then merge it if all is well.
Dec 19 2023
Dec 7 2023
Hi @jback, just a quick note to explain the new changes.
- Updates to clang-format style for clang 16
- Modify clang-format style to 4-space indentation
- Fix missing newline in README
- Scripts to fix indentation of Doxygen comment blocks
- Further improve formatting style
- Re-apply formatting to source files
Nov 22 2023
Nov 21 2023
- Apply formatting in v3-patches branch
@jback, does this look ok? It's the same as the code we now have in #evtgen.
Nov 20 2023
- Add warning if requested C++ standard is too old
Nov 16 2023
Thanks John!
Nov 15 2023
Thanks. @abudinen, I've tried to improve the clarity of the description following our discussion.
Nov 7 2023
It looks like it is going to be a nightmare to merge timedep-branch into master if we apply this formatting first (even after applying the same formatting there first).
So what I'd propose is to merge the last actual code changes into master - will create the differential for that shortly - then create a new v3-patches branch, to which this formatting can then be applied, and release v3r6 from that branch. This Differential can be updated to accomplish that part.
The formatting can then be applied to master after merging in timedep-branch - will require a separate Differential at a later date.
Does that sound reasonable?
Nov 6 2023
Thanks to D101, this is now in master but needs to also be propagated to the time-dependent branch, so not yet closing this.
Closed by rLAURA9dffea19ce7b
(not sure why this didn't happen automatically)
In parallel I need to check what happens if I apply the same formatting in the timedep branch and then try and merge the branches! If that doesn’t go so well we might have to postpone this change untill after merging timedep into master.
Since this is purely formatting changes, I think the main thing to review is that the formatting style looks reasonable and that the CI check looks ok. Up to you if you want to look through all the files!!