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

Ionized electron momentum using Tomassini's model #1210

Open
wants to merge 100 commits into
base: development
Choose a base branch
from

Conversation

EyaDammak
Copy link
Contributor

@EyaDammak EyaDammak commented Feb 18, 2025

This pull request follows the PR #1195.
2 errors have been fixed:

  • error in the second prefactor (laser_dp_second_prefactor) :
    $\alpha = 2n*-1 = 2Z \sqrt{(U_H/U_{ion})} - 1$
  • Model not physically correct for our case
    The model for the transverse momentum spread $\sigma_{u_x}$ in linear polarization was taken from equation [14] of Schroeder article which results from an average on the transverse plan (what he calls "effects of off-axis ionisation of electrons") and only works for a gaussian laser model.
    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 $\Delta^3$ and $\Delta^5$ for the rms electron momentum ($\sigma_{u_x}$ also called width_p in the src code):
    $\sigma_{u_x} = a_0 \Delta ( 1 + s1 * \Delta^2 + s2 * \Delta^4)$
    with $s1 = -7/4 + \alpha / 2$ and $s2 = (8\alpha^2 - 68\alpha+131)/16$
Screenshot 2025-02-20 at 14 20 40

@EyaDammak EyaDammak changed the title [WIP] Ionized electron momentum Tomassi model Ionized electron momentum using Tomassini's model Feb 18, 2025
Copy link
Member

@MaxThevenet MaxThevenet left a 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.

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.

3 participants