New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix to implicit keff for MG scattering #146

Merged

valeriaRaffuzzi merged 1 commit into CambridgeNuclear:main from ChasingNeutrons:develop

Jan 22, 2025

Collaborator

ChasingNeutrons commented Jan 22, 2025

This addresses issue #145. Also fixes a few typos around keffImplicitClerk_class


          Fix to keff for MG scattering

83df212

ChasingNeutrons requested review from Mikolaj-A-Kowalski and valeriaRaffuzzi

January 22, 2025 17:18

valeriaRaffuzzi approved these changes

View reviewed changes

Member

valeriaRaffuzzi left a comment

Looks good, so weird we missed this before! Thanks for this

valeriaRaffuzzi merged commit 23814ae into CambridgeNuclear:main

5 checks passed

ChasingNeutrons mentioned this pull request

Accounting for multiplicative scatter in MG implicit keff estimator #145

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet