Looks good to me, many thanks @abudinen! But let's give the others some time to check through again just to be sure.
Open Tasks
Needs Triage (7)
- Restricted Project
- Tue, Apr 9, 10:10 AM
- T222 Investigate the use of JSON schema in view of introducing and eventually migrating to JSON dec files
- Restricted Project
- Jul 10 2023, 6:20 PM
- Restricted Project
- Jun 6 2023, 5:17 PM
- Restricted Project
- Jun 8 2021, 2:55 PM
Normal (8)
Active Repositories
- rWHIZARDSVN whizardsvn
- Tue, Apr 30, 11:11 PM2024-04-30 23:11:37 (UTC+1)
- Subversion
- rEVTGEN evtgen
- 477 Commits
- ·
- Restricted Project
- Wed, Apr 24, 3:56 PM2024-04-24 15:56:15 (UTC+1)
- Git
- rHERWIGHG herwighg
- 9,432 Commits
- ·
- Restricted Project
- Mar 28 2024, 2:14 PM2024-03-28 14:14:11 (UTC+0)
- Mercurial
- View All Results
Recent Activity
Today
- Moved internal Sherpa configurations for RNG interface into a dedicated cpp file.
Yesterday
Mon, Apr 29
Thanks for the replies tlatham. I am happy with the implementations of them. Since the results of the tests look, as expected, numerically identical, except for the few cases where the changes are understood (and where the models need anyway fixing), I think that this differential is ready to be landed.
I have run all the tests on master and on the branch with these changes on top.
- Comments from Fernando
Thanks for the check through @abudinen - I've replied to your comments inline, and will shortly push up the corresponding changes.
Fri, Apr 26
I spotted a few places where we could get rid of commented code and I also have a few questions out of curiosity. Presumably, the output of the tests will be identical with the master branch. In which case, I think that this differential is ready to be landed.
Rebase on master branch:
- Rename member variables to avoid leading underscores
- Enable shadowing and overloaded-virtual compiler warnings
- Doxygen fixes
- Add check of member variable names in CI
Thu, Apr 25
Wed, Apr 24
As discussed in !16, this switch was now used only in examples and hence it is fine to remove it.
Tue, Apr 23
Closing this revision after the changes have been landed in the commit 71807eba8af5.
Thanks @abudinen, looks good to me
Mon, Apr 22
- Updated History.md
Many thanks @abudinen, looks great, please go ahead.
- Removed training space in History.md
Fri, Apr 19
I added a few points to consult with Sherpa experts as we discussed.