Fix parsers

workflowparsers/aflow/parser.py

@@ -163,7 +163,7 @@ def init_quantities(self):
     def parse(self, key=None):
         super().parse(key)
 
-        if self.get('poscar') is not None and self._results.get('geometry') is None:
+        if self._results.get('poscar') is not None and self._results.get('geometry') is None:
             try:
                 atoms = vasp.read_vasp(StringIO(self._results['poscar'][-1]))
                 self._results['cell'] = atoms.get_cell()
@@ -515,9 +515,10 @@ def parse(self, filepath, archive, logger):
         for n, specie in enumerate(species):
             atom_labels += [specie] * self.aflow_data['composition'][n]
         sec_system.atoms.labels = atom_labels
-        sec_system.atoms.positions = (
-            self.aflow_data.get('positions_cartesian', []) * ureg.angstrom
-        )
+        if self.aflow_data.get('positions_cartesian') is not None:
+            sec_system.atoms.positions = (
+                self.aflow_data.get('positions_cartesian') * ureg.angstrom
+            )
 
         # parse system metadata from aflow_data
         system_quantities = [

workflowparsers/elastic/parser.py

@@ -259,6 +259,7 @@ def deformation_dirs(self):
                 for d in self._dirs
                 if d.startswith(self._deform_dir_prefix)
             ]
+            self._deform_dirs.sort()
 
         return self._deform_dirs
 
@@ -336,7 +337,11 @@ def get_strain_energy(self):
             # the peculiarity of the x_elastic_strain_diagram_values metainfo that it does
             # not have the energy unit
             energies.append((data[1] * ureg.hartree).to('J').magnitude)
-        if len(np.shape(energies)) != 2:
+        try:
+            energies = np.array(energies)
+            if len(np.shape(energies)) != 2:
+                strains, energies = [], []
+        except Exception:
             strains, energies = [], []
         return strains, energies