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

Resampling #202

Closed
Closed

Conversation

NadezhdaKHACHAT
Copy link
Contributor

@NadezhdaKHACHAT NadezhdaKHACHAT commented Oct 26, 2023

Adds the option to resample the field onto a new grid during the propagation step. This can be useful for example when focusing the laser from a large beam size to a small spot.

The resampling option only works in the 'rt' geometry. In order to resample the fields, the user can simply pass a new grid object to the propagate() function. The new grid replaces the original grid.

lasy/examples/FBPICbeam.py Fixed Show fixed Hide fixed
lasy/examples/FBPICbeam.py Fixed Show fixed Hide fixed
lasy/examples/FBPICbeam.py Fixed Show fixed Hide fixed
lasy/examples/FBPICbeam.py Fixed Show fixed Hide fixed
lasy/examples/LASYbeam.py Fixed Show fixed Hide fixed
lasy/examples/LASYvsFBPIC.py Fixed Show fixed Hide fixed
@NadezhdaKHACHAT NadezhdaKHACHAT deleted the RESAMPLING branch October 26, 2023 12:24
@NadezhdaKHACHAT NadezhdaKHACHAT restored the RESAMPLING branch October 26, 2023 12:24
@NadezhdaKHACHAT NadezhdaKHACHAT deleted the RESAMPLING branch October 26, 2023 12:25
@NadezhdaKHACHAT NadezhdaKHACHAT restored the RESAMPLING branch October 26, 2023 16:26
@MKirchen MKirchen requested a review from MaxThevenet October 30, 2023 07:51
@RemiLehe
Copy link
Member

Thanks a lot for this PR! This looks very useful.
Could you add an automated test of this feature, for instance by adding one more test function in this file:
https://github.com/LASY-org/lasy/blob/development/tests/test_gaussian_propagator.py

@AngelFP AngelFP added the new feature request or implementation of a new feature label Jan 15, 2024
@RemiLehe
Copy link
Member

@hightower8083 mentioned that it would be good to first merge #227

@NadezhdaKHACHAT NadezhdaKHACHAT mentioned this pull request Jul 18, 2024
@RemiLehe
Copy link
Member

Superseeded by #269

@RemiLehe RemiLehe closed this Jul 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature request or implementation of a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants