Skip to content
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

Allow fitting procedure on a different cluster and make the fitting database accessible via the MongoDB #314

Merged
merged 148 commits into from
Jan 9, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
148 commits
Select commit Hold shift + click to select a range
6a3a342
add some drafts
JaGeo Dec 21, 2024
fc9e7a5
pre-commit auto-fixes
pre-commit-ci[bot] Dec 21, 2024
25a95b4
restructure and rely on flow instead
JaGeo Dec 21, 2024
4ca1d1c
restructure and rely on flow instead
JaGeo Dec 21, 2024
55562d5
restructure and rely on flow instead
JaGeo Dec 21, 2024
e5787c7
pre-commit auto-fixes
pre-commit-ci[bot] Dec 21, 2024
4f3d6c4
more restructuring of completeworkflow
JaGeo Dec 21, 2024
52deaa2
more restructuring of completeworkflow
JaGeo Dec 21, 2024
6ba11e0
pre-commit auto-fixes
pre-commit-ci[bot] Dec 21, 2024
855dccc
more restructuring of completeworkflow
JaGeo Dec 21, 2024
fcf0cda
bring the workflow more in shape
JaGeo Dec 22, 2024
1958654
pre-commit auto-fixes
pre-commit-ci[bot] Dec 22, 2024
3c0d6a8
fix the workflow stepwise
JaGeo Dec 22, 2024
f85baa3
pre-commit auto-fixes
pre-commit-ci[bot] Dec 22, 2024
8b54a5c
fix some more probles
JaGeo Dec 22, 2024
af16c12
pre-commit auto-fixes
pre-commit-ci[bot] Dec 22, 2024
5e09a4f
hard code displacement to 0.01 in benchmark, fix other more issues in…
JaGeo Dec 22, 2024
7094b93
pre-commit auto-fixes
pre-commit-ci[bot] Dec 22, 2024
1bd5d49
fix more logic problems
JaGeo Dec 22, 2024
295478e
make outputs nicer
JaGeo Dec 22, 2024
f7ad583
pre-commit auto-fixes
pre-commit-ci[bot] Dec 22, 2024
e17fc92
fix a bunch of tests in auto phonons after the new changes
JaGeo Dec 22, 2024
75de5db
fix more tests
JaGeo Dec 22, 2024
829e674
fix benchmark tests
JaGeo Dec 22, 2024
e2bc0eb
daza
JaGeo Dec 22, 2024
bad024c
fix more tests
JaGeo Dec 22, 2024
00e20cd
fix pre database position for all workflows
JaGeo Dec 22, 2024
c80a5ed
add more documentaion and fix number of jobs, addition of get_output
JaGeo Dec 22, 2024
9444190
pre-commit auto-fixes
pre-commit-ci[bot] Dec 22, 2024
d98727e
add more documentation
JaGeo Dec 22, 2024
215c0de
fix list comprehension
JaGeo Dec 22, 2024
1944eed
pre-commit auto-fixes
pre-commit-ci[bot] Dec 22, 2024
ec68520
mace stuff
JaGeo Dec 22, 2024
d2a1485
add to data
JaGeo Dec 23, 2024
834e175
add to data
JaGeo Dec 23, 2024
8c5aa8a
fix ranom seed for structures that are too similar
JaGeo Dec 24, 2024
13b1229
pre-commit auto-fixes
pre-commit-ci[bot] Dec 24, 2024
6eb620f
fix ranom seed for structures that are too similar
JaGeo Dec 24, 2024
bdf0c8b
fix ranom seed for structures that are too similar
JaGeo Dec 24, 2024
c59f853
pre-commit auto-fixes
pre-commit-ci[bot] Dec 24, 2024
453c75d
default random seed
JaGeo Dec 24, 2024
624f762
default random seed
JaGeo Dec 24, 2024
21f8ca2
fix write benchmark generation beyond runs in jobflow
JaGeo Dec 25, 2024
43f95d6
pre-commit auto-fixes
pre-commit-ci[bot] Dec 25, 2024
36af259
fix random seed and add strict tests
JaGeo Dec 25, 2024
a8e065b
fix lorbit problems
JaGeo Dec 25, 2024
6705f1b
fix test
JaGeo Dec 25, 2024
71c59d9
fix test
JaGeo Dec 25, 2024
82cd1e5
fix rms computation
JaGeo Dec 26, 2024
512cad7
pre-commit auto-fixes
pre-commit-ci[bot] Dec 26, 2024
1163b83
add one more test
JaGeo Dec 26, 2024
301a378
add documentation
JaGeo Dec 26, 2024
e562ce3
add hint on the default displacement
JaGeo Dec 26, 2024
74da25a
add hint on the default displacement
JaGeo Dec 26, 2024
1f89160
fix phonon data generation
JaGeo Dec 28, 2024
297de2d
pre-commit auto-fixes
pre-commit-ci[bot] Dec 28, 2024
899bd4e
clean up
JaGeo Dec 28, 2024
52175df
fix references
JaGeo Dec 29, 2024
c5f2ae7
add test for stratisfied split
JaGeo Dec 30, 2024
519b6d3
Merge branch 'iterative_phonon_flow.py' of github.com:autoatml/autopl…
JaGeo Dec 30, 2024
941ad78
fix doc issues and data issues
JaGeo Dec 30, 2024
2d456ac
pre-commit auto-fixes
pre-commit-ci[bot] Dec 30, 2024
ddb51c8
fix doc issues and data issues
JaGeo Dec 30, 2024
5530cd6
pre-commit auto-fixes
pre-commit-ci[bot] Dec 30, 2024
c3f87a4
fix import issues
JaGeo Dec 30, 2024
aff293d
Merge branch 'iterative_phonon_flow.py' of github.com:autoatml/autopl…
JaGeo Dec 30, 2024
1f2c72f
pre-commit auto-fixes
pre-commit-ci[bot] Dec 30, 2024
76e21e6
fix import issues
JaGeo Dec 30, 2024
937dec6
pre-commit auto-fixes
pre-commit-ci[bot] Dec 30, 2024
b61c4a7
add test for pre-database
JaGeo Dec 30, 2024
25085a7
deactivate filter* on test data
JaGeo Dec 31, 2024
44fc621
pre-commit auto-fixes
pre-commit-ci[bot] Dec 31, 2024
20ce3a0
Add tests for test file sizes
JaGeo Dec 31, 2024
6ee13d0
pre-commit auto-fixes
pre-commit-ci[bot] Dec 31, 2024
abe61bb
remove definition
JaGeo Dec 31, 2024
ed9d7ec
fix output of do_
JaGeo Dec 31, 2024
145c354
pre-commit auto-fixes
pre-commit-ci[bot] Dec 31, 2024
4e1c3c7
extend test to more generations and fix dft reference passing
JaGeo Jan 2, 2025
9b23188
conflicts
JaGeo Jan 2, 2025
1ad1d9c
pre-commit auto-fixes
pre-commit-ci[bot] Jan 2, 2025
ea15f67
fix doc
JaGeo Jan 2, 2025
22e1f88
Merge branch 'iterative_phonon_flow.py' of github.com:autoatml/autopl…
JaGeo Jan 2, 2025
9d9c514
fix doc
JaGeo Jan 2, 2025
79d2067
fix doc
JaGeo Jan 2, 2025
b6a7668
fix doc
JaGeo Jan 2, 2025
f768aad
pre-commit auto-fixes
pre-commit-ci[bot] Jan 2, 2025
4ea17d1
try increasing myst heading anchors
naik-aakash Jan 2, 2025
5e010cd
add option to fit on a different cluster
JaGeo Jan 3, 2025
bdb0bb9
pre-commit auto-fixes
pre-commit-ci[bot] Jan 3, 2025
4d6b226
try adding back _all__ block
naik-aakash Jan 3, 2025
d32bbba
fix serialization problem
JaGeo Jan 3, 2025
9ef5eb3
pre-commit auto-fixes
pre-commit-ci[bot] Jan 3, 2025
547c104
fix linting
JaGeo Jan 3, 2025
bd4f244
fix serialization problem
JaGeo Jan 3, 2025
8964bbe
fix glue xml, remove likely wrong database_dir
JaGeo Jan 3, 2025
52773a4
pre-commit auto-fixes
pre-commit-ci[bot] Jan 3, 2025
b7bf3a1
add try except block in rmse calculation
JaGeo Jan 4, 2025
2e05227
Merge branch 'iterative_phonon_flow.py' of github.com:autoatml/autopl…
JaGeo Jan 4, 2025
e582522
pre-commit auto-fixes
pre-commit-ci[bot] Jan 4, 2025
db273a1
fix some more None issues
JaGeo Jan 4, 2025
6781beb
Merge branch 'iterative_phonon_flow.py' of github.com:autoatml/autopl…
JaGeo Jan 4, 2025
2b4fd49
pre-commit auto-fixes
pre-commit-ci[bot] Jan 4, 2025
9d4036c
fix some more None issues
JaGeo Jan 4, 2025
9c83e05
Merge branch 'iterative_phonon_flow.py' of github.com:autoatml/autopl…
JaGeo Jan 4, 2025
3cfb845
fix some more None issues
JaGeo Jan 4, 2025
aa3cc7b
pre-commit auto-fixes
pre-commit-ci[bot] Jan 4, 2025
4696f69
slightly change default names
JaGeo Jan 4, 2025
647b056
fix test names
JaGeo Jan 4, 2025
0d74394
fix test names
JaGeo Jan 4, 2025
5f40db7
pre-commit auto-fixes
pre-commit-ci[bot] Jan 4, 2025
5472a4f
fix test names
JaGeo Jan 4, 2025
fc8b589
fix test names
JaGeo Jan 4, 2025
e902ec3
fix test names
JaGeo Jan 4, 2025
e0af456
fix test names
JaGeo Jan 4, 2025
aa89200
fix test names
JaGeo Jan 4, 2025
c794cf4
fix test names
JaGeo Jan 4, 2025
8a2210f
fix more errors
JaGeo Jan 4, 2025
e533209
fix some more remote issues
JaGeo Jan 5, 2025
53d48ab
pre-commit auto-fixes
pre-commit-ci[bot] Jan 5, 2025
82fb1b1
fix some more remote issues
JaGeo Jan 5, 2025
93106fe
fix some more remote issues
JaGeo Jan 5, 2025
7924bed
test iterative_phonon_flow.py
JaGeo Jan 6, 2025
c51b308
Merge branch 'iterative_phonon_flow.py' of github.com:autoatml/autopl…
JaGeo Jan 6, 2025
8d3fda8
test iterative_phonon_flow.py fix
JaGeo Jan 6, 2025
653bef3
fix output issue
JaGeo Jan 6, 2025
f92f264
Merge branch 'iterative_phonon_flow.py' of github.com:autoatml/autopl…
JaGeo Jan 6, 2025
cce6d1c
add some docu
JaGeo Jan 6, 2025
a8b6142
fix format
JaGeo Jan 6, 2025
790e172
add another test
JaGeo Jan 6, 2025
2984c2a
fix confusing list outputs
JaGeo Jan 6, 2025
d3a87df
pre-commit auto-fixes
pre-commit-ci[bot] Jan 6, 2025
dd0911a
fix doc bug
JaGeo Jan 6, 2025
5b05b43
pre-commit auto-fixes
pre-commit-ci[bot] Jan 6, 2025
e1f804f
fix doc bug
JaGeo Jan 6, 2025
8bb2b57
fix merge conflict
JaGeo Jan 7, 2025
eb060b2
remove debug prints
JaGeo Jan 7, 2025
f99e069
pre-commit auto-fixes
pre-commit-ci[bot] Jan 7, 2025
4816136
improve doc text
QuantumChemist Jan 9, 2025
e223279
improve doc text
QuantumChemist Jan 9, 2025
1649bc1
Merge branch 'main' into add_external_GPU_support
QuantumChemist Jan 9, 2025
bebe946
some improvements in the iterative unit test
QuantumChemist Jan 9, 2025
0e2522d
Merge branch 'add_external_GPU_support' of https://github.com/autoatm…
QuantumChemist Jan 9, 2025
be23d7b
some improvements in the iterative unit test
QuantumChemist Jan 9, 2025
61d090c
adjust doc to new DFT job names
QuantumChemist Jan 9, 2025
f6df57a
a few improvements of the if self.run_fits_on_different_cluster block
QuantumChemist Jan 9, 2025
1296335
small rearrangement
QuantumChemist Jan 9, 2025
874da5e
fix unit tests
QuantumChemist Jan 9, 2025
d788a98
fix unit tests
QuantumChemist Jan 9, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
123 changes: 123 additions & 0 deletions docs/user/phonon/flows/fitting/fitting.md
Original file line number Diff line number Diff line change
Expand Up @@ -395,3 +395,126 @@ Additional fit settings can again be passed using `fit_kwargs` or `**{...}`.
> containing at least one entry for "rattled_dir", "phonon_dir" and "isolated_atom" **VASP** calculations,
> otherwise the code will not finish successfully.


## Is it possible to run the DFT calculations and the MLIP fitting step on different machines?

Very often, we might have the situation that our GPU does not share a hard drive with the compute cluster where we
perform the VASP runs. In such situations, it is convenient to split up the computations.

This can be done by e.g. using jobflow-remote and the following settings for VASP and fitting jobs.
The `local_worker` is the local machine (e.g., a GPU without slurm queue).

```python
from autoplex.auto.phonons.flows import CompleteDFTvsMLBenchmarkWorkflow, IterativeCompleteDFTvsMLBenchmarkWorkflow
from jobflow_remote import submit_flow, set_run_config
from atomate2.vasp.powerups import update_user_incar_settings
from atomate2.vasp.powerups import update_vasp_custodian_handlers

from atomate2.vasp.jobs.phonons import PhononDisplacementMaker
from atomate2.settings import Atomate2Settings

autoplex_flow = IterativeCompleteDFTvsMLBenchmarkWorkflow(max_iterations=3, rms_max=0.2,
complete_dft_vs_ml_benchmark_workflow_0=CompleteDFTvsMLBenchmarkWorkflow(
symprec=1e-3,
run_fits_on_different_cluster=True,
add_dft_phonon_struct=False,
path_to_hyperparameters="/local_machine/mlip-phonon-defaults.json",
apply_data_preprocessing=True,
add_dft_rattled_struct=True,
volume_custom_scale_factors=[1.0,1.0, 1.0],
rattle_type=0, distort_type=0,
rattle_std=0.1, # maybe 0.1
benchmark_kwargs={"relax_maker_kwargs": {
"relax_cell": False}},
supercell_settings={"min_length": 5,
"max_length": 15,
"min_atoms": 10,
"max_atoms": 300,
"fallback_min_length": 9},
# settings that worked with a GAP
split_ratio=0.33,
regularization=False,
separated=False,
num_processes_fit=48,
displacement_maker=phonon_displacement_maker,
phonon_bulk_relax_maker=phonon_bulk_relax_maker,
phonon_static_energy_maker=phonon_static_energy_maker,
rattled_bulk_relax_maker=phonon_bulk_relax_maker,
isolated_atom_maker=static_isolated_atom_maker),
complete_dft_vs_ml_benchmark_workflow_1=CompleteDFTvsMLBenchmarkWorkflow(
symprec=1e-3,
run_fits_on_different_cluster=True,
path_to_hyperparameters="/local_machine/mlip-phonon-defaults.json",
apply_data_preprocessing=True,
add_dft_phonon_struct=False,
add_dft_rattled_struct=True,
volume_custom_scale_factors=[1.0],
rattle_type=0, distort_type=0,
rattle_std=0.1,
benchmark_kwargs={"relax_maker_kwargs": {
"relax_cell": False}},
supercell_settings={"min_length": 5,
"max_length": 15,
"min_atoms": 10,
"max_atoms": 300,
"fallback_min_length": 9},
split_ratio=0.33,
regularization=False,
separated=False,
num_processes_fit=48,
displacement_maker=phonon_displacement_maker,
phonon_bulk_relax_maker=phonon_bulk_relax_maker,
phonon_static_energy_maker=phonon_static_energy_maker,
rattled_bulk_relax_maker=phonon_bulk_relax_maker,
isolated_atom_maker=static_isolated_atom_maker)).make(
structure_list=structure_list, mp_ids=mpids, benchmark_structures=benchmark_structure_list,
benchmark_mp_ids=mpbenchmark,
rattle_seed=0,
fit_kwargs_list=[{
"soap": {"delta": 1.0, "l_max": 12, "n_max": 10,
"atom_sigma": 0.5, "zeta": 4, "cutoff": 5.0,
"cutoff_transition_width": 1.0,
"central_weight": 1.0, "n_sparse": 6000, "f0": 0.0,
"covariance_type": "dot_product",
"sparse_method": "cur_points"},
"general": {"two_body": True, "three_body": False, "soap": True,
"default_sigma": "{0.001 0.05 0.05 0.0}", "sparse_jitter": 1.0e-8, }}]
)

resources = {"nodes": 1, "partition": "micro", "time": "00:55:00", "ntasks": 48, "qverbatim": "#SBATCH --get-user-env",
"mail_user": "your_email@adress", "mail_type": "ALL", "account": "xxxxxx"}

resources_phon = {"nodes": 3, "partition": "micro", "time": "0:55:00", "ntasks": 144,
"qverbatim": "#SBATCH --get-user-env",
"mail_user": "your_email@adress", "mail_type": "ALL", "account": "xxxxxx"}

resources_ratt = {"nodes": 3, "partition": "micro", "time": "0:55:00", "ntasks": 144,
"qverbatim": "#SBATCH --get-user-env",
"mail_user": "your_email@adress", "mail_type": "ALL", "account": "xxxxxx"}

resources_mlip = {}
autoplex_flow = set_run_config(autoplex_flow, name_filter="dft static", resources=resources, worker="supermuc_worker")
autoplex_flow = set_run_config(autoplex_flow, name_filter="stat_iso_atom", resources=resources, worker="supermuc_worker")

autoplex_flow = set_run_config(autoplex_flow, name_filter="dft phonon static", resources=resources_phon, worker="supermuc_worker")
autoplex_flow = set_run_config(autoplex_flow, name_filter="static", resources=resources_phon, worker="supermuc_worker")

autoplex_flow = set_run_config(autoplex_flow, name_filter="dft rattle static", resources=resources_ratt, worker="supermuc_worker")
autoplex_flow = set_run_config(autoplex_flow, name_filter="tight relax", resources=resources, worker="supermuc_worker")
autoplex_flow = set_run_config(autoplex_flow, name_filter="dft tight relax", resources=resources, worker="supermuc_worker")

autoplex_flow = set_run_config(autoplex_flow, name_filter="machine_learning_fit", resources=resources_mlip, worker="local_worker")
autoplex_flow = set_run_config(autoplex_flow, name_filter="gap phonon static", resources=resources_mlip, worker="local_worker")
autoplex_flow = set_run_config(autoplex_flow, name_filter="Force field", resources=resources_mlip, worker="local_worker")
autoplex_flow = set_run_config(autoplex_flow, name_filter="data_preprocessing_for_fitting", resources=resources, worker="supermuc_worker")


autoplex_flow = update_user_incar_settings(autoplex_flow, {"NPAR": 4})

autoplex_flow = update_vasp_custodian_handlers(autoplex_flow, custom_handlers={})

autoplex_flow.name = "small Sn test, test without phonon2"

# submit the workflow to jobflow-remote
print(submit_flow(autoplex_flow, worker="local_worker", resources=resources_mlip, project="phonons_qha"))
```
12 changes: 6 additions & 6 deletions docs/user/phonon/flows/generation/data.md
Original file line number Diff line number Diff line change
Expand Up @@ -189,8 +189,8 @@ INFO Starting job - reduce_supercell_size_job_mp-22905
INFO Finished job - reduce_supercell_size_job_mp-22905
INFO Starting job - rattled supercells_mp-22905
INFO Finished job - rattled supercells_mp-22905
INFO Starting job - tight relax_mp-22905
INFO Finished job - tight relax_mp-22905
INFO Starting job - dft tight relax_mp-22905
INFO Finished job - dft tight relax_mp-22905
INFO Starting job - generate_randomized_structures_mp-22905
INFO Finished job - generate_randomized_structures_mp-22905
INFO Starting job - run_phonon_displacements_mp-22905
Expand All @@ -202,10 +202,10 @@ INFO Finished job - dft rattle static 2/12_mp-22905
...
INFO Starting job - single-atom displaced supercells_mp-22905
INFO Finished job - single-atom displaced supercells_mp-22905
INFO Starting job - tight relax_mp-22905
INFO Finished job - tight relax_mp-22905
INFO Starting job - static_mp-22905
INFO Finished job - static_mp-22905
INFO Starting job - dft tight relax_mp-22905
INFO Finished job - dft tight relax_mp-22905
INFO Starting job - dft static_mp-22905
INFO Finished job - dft static_mp-22905
INFO Starting job - generate_phonon_displacements_mp-22905
INFO Finished job - generate_phonon_displacements_mp-22905
INFO Starting job - run_phonon_displacements_mp-22905
Expand Down
7 changes: 7 additions & 0 deletions src/autoplex/auto/phonons/flows.py
Original file line number Diff line number Diff line change
Expand Up @@ -176,6 +176,9 @@ class CompleteDFTvsMLBenchmarkWorkflow(Maker):
Name of the glue.xml file path.
use_defaults_fitting: bool
Use the fit defaults.
run_fits_on_different_cluster: bool
Allows you to run fits on a different cluster than DFT (will transfer
fit database via MongoDB, might be slow).
"""

name: str = "add_data"
Expand Down Expand Up @@ -226,6 +229,7 @@ class CompleteDFTvsMLBenchmarkWorkflow(Maker):
glue_xml: bool = False
glue_file_path: str = "glue.xml"
use_defaults_fitting: bool = True
run_fits_on_different_cluster: bool = False

def make(
self,
Expand Down Expand Up @@ -400,6 +404,7 @@ def make(
separated=self.separated,
regularization=self.regularization,
distillation=self.distillation,
run_fits_on_different_cluster=self.run_fits_on_different_cluster,
).make(
species_list=isoatoms.output["species"],
isolated_atom_energies=isoatoms.output["energies"],
Expand Down Expand Up @@ -826,6 +831,7 @@ class CompleteDFTvsMLBenchmarkWorkflowMPSettings(CompleteDFTvsMLBenchmarkWorkflo
phonon_bulk_relax_maker: BaseVaspMaker = field(
default_factory=lambda: MPGGADoubleRelaxMaker.from_relax_maker(
MPGGARelaxMaker(
name="dft tight relax",
run_vasp_kwargs={"handlers": ()},
input_set_generator=MPRelaxSet(
force_gamma=True,
Expand All @@ -849,6 +855,7 @@ class CompleteDFTvsMLBenchmarkWorkflowMPSettings(CompleteDFTvsMLBenchmarkWorkflo
rattled_bulk_relax_maker: BaseVaspMaker = field(
default_factory=lambda: MPGGADoubleRelaxMaker.from_relax_maker(
MPGGARelaxMaker(
name="dft tight relax",
run_vasp_kwargs={"handlers": ()},
input_set_generator=MPRelaxSet(
force_gamma=True,
Expand Down
15 changes: 10 additions & 5 deletions src/autoplex/auto/phonons/jobs.py
Original file line number Diff line number Diff line change
Expand Up @@ -275,7 +275,6 @@ def complete_benchmark( # this function was put here to prevent circular import

for path in ml_path:
suffix = Path(path).name
print(suffix)
if suffix == "without_regularization":
suffix = "without_reg"
if suffix not in ["phonon", "rattled"]:
Expand Down Expand Up @@ -541,6 +540,7 @@ def dft_phonopy_gen_data(
if phonon_bulk_relax_maker is None:
phonon_bulk_relax_maker = DoubleRelaxMaker.from_relax_maker(
TightRelaxMaker(
name="dft tight relax",
run_vasp_kwargs={"handlers": {}},
input_set_generator=TightRelaxSetGenerator(
user_incar_settings={
Expand All @@ -567,6 +567,7 @@ def dft_phonopy_gen_data(

if phonon_static_energy_maker is None:
phonon_static_energy_maker = StaticMaker(
name="dft static",
input_set_generator=StaticSetGenerator(
auto_ispin=False,
user_incar_settings={
Expand All @@ -584,13 +585,17 @@ def dft_phonopy_gen_data(
# to be removed
"NPAR": 4,
},
)
),
)

# always set autoplex default as job name
phonon_displacement_maker.name = "dft phonon static"
phonon_bulk_relax_maker.name = "tight relax"
phonon_static_energy_maker.name = "static"
phonon_static_energy_maker.name = "dft static"
try:
phonon_bulk_relax_maker.relax_maker1.name = "dft tight relax"
phonon_bulk_relax_maker.relax_maker2.name = "dft tight relax"
except AttributeError:
phonon_bulk_relax_maker.name = "dft tight relax"

for displacement in displacements:
dft_phonons = DFTPhononMaker(
Expand Down Expand Up @@ -725,7 +730,7 @@ def dft_random_gen_data(

# always set autoplex default as job name
displacement_maker.name = "dft rattle static"
rattled_bulk_relax_maker.name = "tight relax"
rattled_bulk_relax_maker.name = "dft tight relax"

# TODO: decide if we should remove the additional response here as well
# looks like only the output is changing
Expand Down
1 change: 0 additions & 1 deletion src/autoplex/data/common/jobs.py
Original file line number Diff line number Diff line change
Expand Up @@ -237,7 +237,6 @@ def generate_randomized_structures(
"""
if supercell_matrix is None:
supercell_matrix = [[2, 0, 0], [0, 2, 0], [0, 0, 2]]

supercell = get_supercell(
unitcell=get_phonopy_structure(structure),
supercell_matrix=supercell_matrix,
Expand Down
1 change: 1 addition & 0 deletions src/autoplex/data/common/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -658,6 +658,7 @@ def force_plot(

"""
in_atoms = ase.io.read(in_file, ":")

for atoms in in_atoms[:-1]:
kwargs = {
"forces": atoms.arrays["REF_forces"],
Expand Down
Loading
Loading