-
Notifications
You must be signed in to change notification settings - Fork 0
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
SOF-7217: add templates and executables for SIMPLE.X calculation #116
Conversation
assets/espresso/gww_simple.j2.in
Outdated
ggwin%omega_fit=20 | ||
ggwin%n_grid_fit=240 | ||
ggwin%tau=9.8 | ||
ggwin%n_set_pola=16 |
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.
can we indent the ggwin
block for better visibility
assets/espresso/head_simple.j2.in
Outdated
second_grid_i=1 | ||
second_grid_n=10 | ||
niter_ph=1 | ||
nsteps_lanczos=30 |
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.
Let's use 4 spaces and unindent &inputph
assets/espresso/simple.j2.in
Outdated
num_val=16 | ||
num_cond=24 | ||
s_bands=0.1 | ||
s_product=1.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.
Maybe we could use https://github.com/cphyc/fortran-linter and pre-commit hook to format this
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.
Looks like the linter works, except for the 4 spaces. Let's use the fork in https://github.com/Exabyte-io/fortran-linter
1f4ac10
to
8721782
Compare
No description provided.