-
Notifications
You must be signed in to change notification settings - Fork 23
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
Combined transverse & longitudinal laser profile #50
Conversation
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.
Looks good, thanks for this PR! The details of each profile are the same that were in different files before, right? Github is having a hard time, but I think they were reviewed already in previous PRs.
Yes, that's correct, the mathematical expressions are the same as before this PR. |
This PR finalizes the implementation of transverse and longitudinal laser profiles:
A new class
CombinedLongitudinalTransverseProfile
is created ; its usage is illustrated in the example.The previously-existing profiles (Gaussian, Hermite-Gauss, Laguerre-Gauss) have been moved to:
In general, it is expected that pulses will be constructed with
CombinedLongitudinalTransverseProfile
. However, because the 3D Gaussian pulse is so commonly used, I created a dedicated class for this type of laser ; its usage is illustrated in the example.