-
Notifications
You must be signed in to change notification settings - Fork 18
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
Ionized electron momentum using Tomassini's model #1210
Ionized electron momentum using Tomassini's model #1210
Conversation
…ipace into laser_ion_momentum_2
Co-authored-by: Alexander Sinn <64009254+AlexanderSinn@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this PR! Do we expect, from theory, that Tomassini's expression should always be more accurate than what is in Massimo's? Probably yes, but if not we could have both implemented and available. Let's have a quick chat.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this PR! Just minor comment, see below.
This pull request follows the PR #1195.
2 errors have been fixed:
laser_dp_second_prefactor
) :The model for the transverse momentum spread
This new PR implemented the result from Tomassini's article ( #https://doi.org/10.1017/hpl.2021.56, equation (12)), including the terms of order
width_p
in the src code):with