Enable initial pulse front tilt set for lasers #1100
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
huixingjian
changed the title
huixingjian
changed the title
src/laser/Laser.H
Outdated
| @@ -25,9 +26,11 @@ public: | |||
| amrex::Real m_CEP {0.}; /**< Laser carrier-envelope phase (CEP) */ | |||
| /** Propagation angle of the pulse in the yz plane (0 is the along the z axis) */ | |||
| amrex::Real m_propagation_angle_yz {0.}; | |||
| /**Pulse front tilt angle of the pulse in yz plane (pi/2 is no PFT)*/ | |||
There was a problem hiding this comment.
Suggested change
| /**Pulse front tilt angle of the pulse in yz plane (pi/2 is no PFT)*/ | |
| /**Pulse front tilt angle of the pulse in yz plane (pi/2 is no PFT)*/ |
Can you add 4 spaces in front of the comments so it lines up with the other stuff.
src/laser/Laser.H
Outdated
| amrex::Real m_L0 {0.}; /**< Laser length (HW 1/e in amplitude) */ | ||
| amrex::Real m_tau {0.}; /**< Laser duration (HW 1/e in amplitude) */ | ||
| /** Focal distance of the laser pulse */ | ||
| /** Focal distance of the laser pulse */ |
There was a problem hiding this comment.
Suggested change
| /** Focal distance of the laser pulse */ | |
| /** Focal distance of the laser pulse */ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The lasers could have initial pulse front tilt angle on yoz plane by
<laser name>.PFT_yz—the angle between the pulse front (maximum intensity contour)and the propagation direction defined by [Selcuk Akturk Opt. Express 12 (2004)]