I added a few points to consult with Sherpa experts as we discussed.

Thanks for checking! Ok, will land this differential.

Great, thanks for checking it over @jback!

I agree that these changes look OK.

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

Great, many thanks @abudinen! I have no further comments following the thorough review already conducted in EvtGen!10. Please go ahead and land this.

• Reverted doRadCorr function to not be const for semantic correctness in Vincia and Sherpa interfaces.

This was solved in D107 and D108.

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

This is ready to be merged.

The changes look good to me.

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!

I have search for each removed model and class via grep in the MR branch and found entries only in the history and release-note files. Therefore, I think that an explicit mention of the removed models and classes in the history file might be useful. I also checked whether the deleted models are used at Belle II or LHCb and it seems they are not using them. I remember that @kreps mentioned that PROPSLPOLE is used at LHCb, but I did not find any decay file in the DecFiles project.

• Update History.md