Update wiki#466
Conversation
Updated item: PAR_TR_VEL_CORR, DT__PHASE, DT__HYBRID_CFL, DT__HYBRID_CFL_INIT, DT__HYBRID_VELOCITY, DT__HYBRID_VELOCITY_INIT, OPT__FLAG_ENGY_DENSITY, OPT__FLAG_INTERFERENCE, OPT__FLAG_SPECTRAL, OPT__FLAG_SPECTRAL_N, OPT__LB_EXCHANGE_FATHER, OPT__OVERLAP_MPI, MIN_TEMP, MIN_ENTR, EXT_POT_TABLE_DH_X, EXT_POT_TABLE_DH_Y, EXT_POT_TABLE_DH_Z, EXT_POT_TABLE_FLOAT8, OPT__GRAVITY_EXTRA_MASS, OPT__INT_PHASE.
Add new page for ELBDM runtime parameters Add spectral Interpolation
| @@ -81,6 +94,16 @@ set [[PAR_IMPROVE_ACC | Runtime Parameters:-Particles#PAR_IMPROVE_ACC]]=1. | |||
| must be disabled. In addition, it is currently recommended to disable | |||
| [[AUTO_REDUCE_DT | Runtime Parameters:-Timestep#AUTO_REDUCE_DT]]. | |||
|
|
|||
| <a name="OPT__LB_EXCHANGE_FATHER"></a> | |||
There was a problem hiding this comment.
I'm not sure where to place this parameter. Currently I put here.
There was a problem hiding this comment.
I think it's ok not to mention --model=ELBDM here?
| * **Restriction:** | ||
|
|
||
| OPT__GRAVITY_EXTRA_MASS | ||
| <a name="OPT__GRAVITY_EXTRA_MASS"></a> |
There was a problem hiding this comment.
Here I didn't write a detailed guide of how to edit this function. Is that ok?
There was a problem hiding this comment.
I think it might be better to make a brief tutorial such as:
- Go to file
src/SelfGravity/Poi_AddExtraMassForGravity.cpp. - Copy
Poi_AddExtraMassForGravity_Template()and make your own function in your test problem. - Set up
Poi_AddExtraMassForGravity_Ptr.
Asking the user to edit src/SelfGravity/Poi_AddExtraMassForGravity.cpp might be misleading.
Instead of writing the guide here I think it might be cleaner to make a new subsection under Adding-New-Simulations#vi-add-problem-specific-functionalities.
There was a problem hiding this comment.
I think it would be better to put the tutorial on the page gravity to match the existing pattern "external acceleration/potential". But I still added a new subsection under Adding-New-Simulations#vi-add-problem-specific-functionalities and give a link to page gravity.
|
Hi @vivi235711, I noticed that this PR is targeting the |
|
@jzuhone Hi, I saw that you changed the base branch from |
|
@vivi235711 I am not sure how this happened; I didn't intend to do this and I didn't actually hit the buttons for it. I use GitHub on my phone so something accidental may have happened. Sorry about that. |
| * **Restriction:** | ||
|
|
||
| OPT__GRAVITY_EXTRA_MASS | ||
| <a name="OPT__GRAVITY_EXTRA_MASS"></a> |
There was a problem hiding this comment.
I think it might be better to make a brief tutorial such as:
- Go to file
src/SelfGravity/Poi_AddExtraMassForGravity.cpp. - Copy
Poi_AddExtraMassForGravity_Template()and make your own function in your test problem. - Set up
Poi_AddExtraMassForGravity_Ptr.
Asking the user to edit src/SelfGravity/Poi_AddExtraMassForGravity.cpp might be misleading.
Instead of writing the guide here I think it might be cleaner to make a new subsection under Adding-New-Simulations#vi-add-problem-specific-functionalities.
| Mass introduced here is only used for gravity (e.g., Poisson equation) and | ||
| is excluded from other equations (e.g., hydro/MHD/Schrödinger) and output data. | ||
| To configure the extra mass, edit /src/SelfGravity/Poi_AddExtraMassForGravity.cpp. | ||
| Refer to [[Initialization Function | Adding-New-Simulations#initialization-function]] for details. |
There was a problem hiding this comment.
I think it might be better to refer to #vi-add-problem-specific-functionalities section instead of #initialization-function? Or maybe add a new subsection below #external-potential. See my other comment.
|
Add |
|
@vivi235711 When you have time, please update this PR to resolve the conflicts. Thanks! If you don't have time to address @HsunYeong's comments at the moment, please feel free to let us know. Hsun-Yeong can follow up on them in a separate PR. |
Add brief tutorial for OPT__GRAVITY_EXTRA_MASS Typo fix
Resolve conflicts Fix links
|
@HsunYeong
|
|
@vivi235711 I really appreciate you taking the time to update this PR! @HsunYeong Please help review this PR again after you are back from your trip. Thanks! |
HsunYeong
left a comment
There was a problem hiding this comment.
@vivi235711 Thank you for the update. I think the PR is ready once the typo is fixed.
| is just to avoid redundant assignments to the same field index variable. | ||
|
|
||
| The second parameter should be set to either `FIXUP_FLUX_YES` or `FIXUP_FLUX_NO`. | ||
| Tfuld be set to either `FIXUP_FLUX_YES` or `FIXUP_FLUX_NO`. |
There was a problem hiding this comment.
| Tfuld be set to either `FIXUP_FLUX_YES` or `FIXUP_FLUX_NO`. | |
| The second parameter should be set to either `FIXUP_FLUX_YES` or `FIXUP_FLUX_NO`. |
I think some words are accidentally removed here?
|
|
||
| The workflow is triggered when changes are pushed to any of the following files: | ||
| - `src/Init/Init_Load_Parameter.cpp` | ||
| - `example/input/Input__Paramter` |
There was a problem hiding this comment.
There is a typo Paramter, which is generated by line 187 of tool/wiki/sync_runtime_parameter.py. Maybe we can fix it in this PR?
There was a problem hiding this comment.
The typo is fixed in #521, you can ignore this comment.
Partially resolve #400.
The remaining part has been completed by @hsinhaoHHuang.
Changes:
Added links, pages, and descriptions for the following parameters:
Particles
Timesteps
Refinement
ELBDM (new page)
MPI and OpenMP
Hydro
Gravity
Interpolation