-
Notifications
You must be signed in to change notification settings - Fork 12
FDM Zoom-in simulation #132
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
base: psidm
Are you sure you want to change the base?
Conversation
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.
Hi @jchan192, I left some comments.
Major comments
- The comments with
[Bug]or[Potential bug]need to be fix. - The comments with
[Style]need to be updated to gamer style. - Please remove all the trailing spaces. You can search them by this command:
grep -r -n "[[:blank:]]$" ./Minor comments
- The comments with
[Optimize]are some optimization or generalization of the code. Let me know if you disagree any of them.
The rest of the comments without tags are some minor change suggestions or requests.
Please let me know if any of my comments is unclear.
Once all of the conversations are resolved, I will try to reproduce your production run.
example/test_problem/ELBDM/LSS_Hybrid_Zoomin/plot_script/plot_projection_grid.py
Show resolved
Hide resolved
example/test_problem/ELBDM/LSS_Hybrid_Zoomin/plot_script/plot_projection_grid.py
Outdated
Show resolved
Hide resolved
example/test_problem/ELBDM/LSS_Hybrid_Zoomin/plot_script/plot_projection_grid.py
Outdated
Show resolved
Hide resolved
ChunYen-Chen
left a comment
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.
ChunYen-Chen
left a comment
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.
example/test_problem/ELBDM/LSS_Hybrid_Zoomin/plot_script/Compute_profiles.py
Outdated
Show resolved
Hide resolved
example/test_problem/ELBDM/LSS_Hybrid_Zoomin/plot_script/VelocityDisp.py
Outdated
Show resolved
Hide resolved
example/test_problem/ELBDM/LSS_Hybrid_Zoomin/plot_script/Profile_Functions.py
Outdated
Show resolved
Hide resolved
example/test_problem/ELBDM/LSS_Hybrid_Zoomin/plot_script/Profile_Functions.py
Outdated
Show resolved
Hide resolved
example/test_problem/ELBDM/LSS_Hybrid_Zoomin/plot_script/Profile_Functions.py
Outdated
Show resolved
Hide resolved
example/test_problem/ELBDM/LSS_Hybrid_Zoomin/plot_script/plot_slice_grid.py
Outdated
Show resolved
Hide resolved
|
@jchan192 , and the link to Examples of the download script: #!/bin/bash
LOCAL_FILENAME="Zoomin_IC_heavyhalo"
FILE_ID="677cc1c3999605c485c8de80"
# 1. download
curl https://hub.yt/api/v1/item/${FILE_ID}/download -o "${LOCAL_FILENAME}.tar.gz"
# 2. unzip
tar zxvf ${LOCAL_FILENAME}.tar.gz
rm ${LOCAL_FILENAME}.tar.gz
ln -fs UM_IC_hybrid_N1024 UM_IC#!/bin/bash
LOCAL_FILENAME="Zoomin_IC_lighthalo"
FILE_ID="677cbeb4999605c485c8de7a"
# 1. download
curl https://hub.yt/api/v1/item/${FILE_ID}/download -o "${LOCAL_FILENAME}.tar.gz"
# 2. unzip
tar zxvf ${LOCAL_FILENAME}.tar.gz
rm ${LOCAL_FILENAME}.tar.gz
ln -fs UM_IC_hybrid_N256_third UM_ICSee also gamer-project#408. |
Updated
src/TestProblem/ELBDM/LSS/Init_TestProb_ELBDM_LSS.cppto read a table of parameters to control the zoom-in volume
Added a new example in
example/test_problem/ELBDM/LSS_Hybrid_Zoomin/for testing a smaller zoom-in simulation,while including the IC that can produce the results in (Chan+24, submitted to MNRAS).