Skip to content
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.

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

Mc high ka corrections #5977

Merged
merged 1 commit into from
Jun 14, 2024

Conversation

ffoucart
Copy link
Contributor

@ffoucart ffoucart commented May 9, 2024

Proposed changes

Add corrections to high absorption opacity regions needed for implicit Monte-Carlo

Upgrade instructions

NA

Code review checklist

  • The code is documented and the documentation renders correctly. Run
    make doc to generate the documentation locally into BUILD_DIR/docs/html.
    Then open index.html.
  • The code follows the stylistic and code quality guidelines listed in the
    code review guide.
  • The PR lists upgrade instructions and is labeled bugfix or
    new feature if appropriate.

@ffoucart
Copy link
Contributor Author

ffoucart commented May 9, 2024

Depends on #5902

@ffoucart ffoucart added the dependent Needs a different PR to be merged in first label May 9, 2024
@ffoucart ffoucart force-pushed the MCHighKaCorrections branch 2 times, most recently from 10bc6dc to c8c0f92 Compare May 10, 2024 13:30
@ffoucart ffoucart force-pushed the MCHighKaCorrections branch 2 times, most recently from 7424311 to 0939cef Compare May 24, 2024 14:18
@ffoucart ffoucart marked this pull request as ready for review May 24, 2024 14:19
@ffoucart
Copy link
Contributor Author

Rebased against 5902 and ready for review.

@ffoucart ffoucart removed the dependent Needs a different PR to be merged in first label May 24, 2024
Copy link
Member

@kidder kidder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can squash the change when rebasing

#include <array>

#include "Evolution/Particles/MonteCarlo/NeutrinoInteractionTable.hpp"
#include "Evolution/Particles/MonteCarlo/TemplatedLocalFunctions.hpp"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

make this the first include followed by a blank line

// d(radiation_energy_density)/d(fluid_energy_density)
// at constant Ye, and
// d(neutrino_lepton_number_density)/d(proton_number_density)
// at constant T as measured of how 'impactfull' neutrino
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as a measure of how impactful

@ffoucart ffoucart force-pushed the MCHighKaCorrections branch 2 times, most recently from 5f00b37 to a52c6fa Compare June 6, 2024 20:07
@ffoucart ffoucart added the dependent Needs a different PR to be merged in first label Jun 6, 2024
@ffoucart
Copy link
Contributor Author

ffoucart commented Jun 6, 2024

Latest PR addressed code review requests, but also rebased against #6037 to avoid future conflicts. This PR now depends on #6037 (only for minor changes to the interface of the neutrino table when adding GZs)

@ffoucart ffoucart removed the dependent Needs a different PR to be merged in first label Jun 13, 2024
@kidder kidder merged commit d44c6b1 into sxs-collaboration:develop Jun 14, 2024
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants