picongpu icon indicating copy to clipboard operation
picongpu copied to clipboard

Arbitrary order field solver documentation may need some clarification

Open pordyna opened this issue 3 years ago • 5 comments

The current documentation shows figures for the phase velocity error for a very low resolution (3 cells per laser wavelength) that suggest that the new solver requires an order of magnitude shorter timestep, but according to @sbastrakov this is not true for more realistic resolutions. So, maybe we should also include figures for typical resolutions in typical Wakefield and solid-density simulations. @steindev would it be also possible to attach your python scripts in the documentation, like we do it in Models/Field Ionization, so that a user can make an estimate for the optimal time step duration themself?

pordyna avatar Jul 13 '22 08:07 pordyna

Perhaps in this section in addition to existing plots for $N_\lambda = 3$ also add a better resolved case.

Note that now PIConGPU outputs phase velocity for the currently used parameters. Of course, this does not mean we shouldn't also provide a script as requested - as PIConGPU output is for informing a user and controlling wrong setups, while a script could help to choose parameters wisely.

sbastrakov avatar Jul 13 '22 09:07 sbastrakov

cc @steindev just in case

sbastrakov avatar Jul 14 '22 10:07 sbastrakov

@steindev What is the status of this issue?

PrometheusPi avatar Nov 01 '22 09:11 PrometheusPi

On the list.

steindev avatar Nov 03 '22 15:11 steindev

Is this still planned?

chillenzer avatar Feb 06 '24 10:02 chillenzer