-
Notifications
You must be signed in to change notification settings - Fork 19
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
[WIP] Collective Effects Config in Dashboard #690
base: development
Are you sure you want to change the base?
[WIP] Collective Effects Config in Dashboard #690
Conversation
src/python/impactx/dashboard/Input/space_charge_configuration/spaceChargeFunctions.py
Fixed
Show resolved
Hide resolved
try: | ||
prob_relative_value = float(prob_relative_value) | ||
poisson_solver = state.poisson_solver | ||
|
||
if poisson_solver == "multigrid": |
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.
Validation looks good.
|
||
state.dynamic_size = False | ||
state.max_level = 3 | ||
state.n_cell = [0.0, 0.0, 0.0] |
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.
n_cell
are integers.
Validation: they MUST be >1
if space charge (any solver) is on.
A sensible, coarse default is:
state.n_cell = [0.0, 0.0, 0.0] | |
state.n_cell = [32, 32, 32] |
src/python/impactx/dashboard/Input/space_charge_configuration/spaceChargeMain.py
Outdated
Show resolved
Hide resolved
Accidently pulled in code from unit testing.. need to remove first 18 commits as they are not related to this PR. |
for more information, see https://pre-commit.ci
folder name is still called 'csrConfiguration'
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
This reverts commit 4456665.
d6d3cb1
to
c08a1be
Compare
for more information, see https://pre-commit.ci
…y30/impactx into dashboard/space_charge_configuration
if state.space_charge: | ||
sim.max_level = state.max_level | ||
sim.n_cell = state.n_cell | ||
sim.particle_shape = state.particle_shape | ||
sim.poisson_solver = state.poisson_solver | ||
sim.space_charge = state.space_charge | ||
sim.dynamic_size = state.dynamic_size | ||
sim.prob_relative = state.prob_relative | ||
|
||
# csr | ||
if state.csr: | ||
sim.csr = state.csr | ||
sim.csr_bins = state.csr_bins | ||
|
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.
Currently, particle_shape
is only displayed when space_charge
is set to True. However, I see that particle_shape
is also being used in run_chicane_csr.py
. Should we consider having particle_shape
displayed for CSR as well?
PR adds in Space Charge and CSR configuration for users.
For Space Charge:
For CSR
Resolves #665