You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently Beam has it's minimum width at Z = 0. However, I came across a problem where a focused beam was injected into the plasma, having a minimum width in the region of the central plasma. Since it seems natural that Z=0 in the beam coordinates corresponds to the entry point of the beam into the plasma, I suggest adding the focal_length property, which defaults to 0.
This will change how the beam width is calculated. Instead of this:
Having beam foci inside plasma is something this model doesn't support and is it's limitation.
I would personally wait with such changes for the new beam model which won't have backwards compatibility.
In the new architecture of the beam I implement the beam description through the DistributionFunction. One of the implementation will allow users to implement simply their profiles through Raysect's function framework, as it is done in the laser's case.
The basic beam profile model will be based on the Gaussian beam, which is in fact generalisation of the model we have now and allows users to place beam waists in arbitrary positions.
I think this is just an error, but if we allow an arbitrary beam waist position, then we have to fix that too, which means it will be a different beam model.
Currently
Beam
has it's minimum width at Z = 0. However, I came across a problem where a focused beam was injected into the plasma, having a minimum width in the region of the central plasma. Since it seems natural that Z=0 in the beam coordinates corresponds to the entry point of the beam into the plasma, I suggest adding thefocal_length
property, which defaults to 0.This will change how the beam width is calculated. Instead of this:
we'll have this:
This change also affects the
_generate_geometry()
method.I already have this implemented in the fork, so I can make a PR with this feature and the the changes discussed in #414.
@Mateasek, @jacklovell, what do you think of this feature?
The text was updated successfully, but these errors were encountered: