Skip to content

Commit 001610a

Browse files
author
Hancheng Zhao
committed
Previously phonopy.save.yaml was shared across different qha calcs, which is not ideal. Also deleted the requirement of phonopy_save_path in qha_properties because it is never used
1 parent 5bf4fde commit 001610a

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

asimtools/asimmodules/phonopy/full_qha.py

+4-4
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ def full_qha(
8080
},
8181
'supercell': supercell,
8282
'distance': 0.02,
83-
'phonopy_save_path': phonopy_save_path,
83+
'phonopy_save_path': '../'+phonopy_save_path,
8484
},
8585
},
8686
'step-1': {
@@ -99,15 +99,15 @@ def full_qha(
9999
'env_id': process_env_id,
100100
'args': {
101101
'supercell_image_file_pattern': '../step-1/id-*/image_output.xyz',
102-
'phonopy_save_path': phonopy_save_path,
102+
'phonopy_save_path': '../'+phonopy_save_path,
103103
'use_seekpath': True,
104104
},
105105
},
106106
'step-3': {
107107
'asimmodule': 'phonopy.thermal_properties',
108108
'env_id': process_env_id,
109109
'args': {
110-
'phonopy_save_path': phonopy_save_path,
110+
'phonopy_save_path': '../'+phonopy_save_path,
111111
't_max': t_max,
112112
'suffix': {}
113113
},
@@ -127,7 +127,7 @@ def full_qha(
127127
'env_id': process_env_id,
128128
'args': {
129129
'ev_csv': '../step-0/eos.csv',
130-
'phonopy_save_path': phonopy_save_path,
130+
# 'phonopy_save_path': phonopy_save_path,
131131
'thermal_properties_file_pattern': '../step-1/id-*/step-3/thermal_properties-*.yaml',
132132
't_max': t_max - 10,
133133
'pressure': pressure,

asimtools/asimmodules/phonopy/qha_properties.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313

1414
def qha_properties(
1515
ev_csv: str,
16-
phonopy_save_path: str,
16+
# phonopy_save_path: str,
1717
thermal_properties_file_pattern: str,
1818
pressure: Optional[float] = None,
1919
t_min: float = 0,

0 commit comments

Comments
 (0)