Apply review

aiidateam · Jun 10, 2023 · 159d67c · 159d67c
1 parent f38792f
commit 159d67c
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 13 deletions.
diff --git a/docs/source/howto/calculations/pw.md b/docs/source/howto/calculations/pw.md
@@ -296,7 +296,7 @@ When a {{ PwCalculation }} is completed, there are quite a few possible analyses
 
 During the self-consistent field cycle, the difference in energy of the newly computed charge density and the starting one is computed and stored.
 It can easily be retrieved through the {meth}`~aiida_quantumespresso.tools.calculations.pw.PwCalculationTools.get_scf_accuracy` method.
-This method can be accessed directly through the ``tools`` of a completed `{{ PwCalculation }}` node:
+This method can be accessed directly through the `tools` of a completed {{ PwCalculation }} node:
 
 ```python
 In [1]: node = load_node(IDENTIFIER)

diff --git a/docs/source/tutorials/first_pw.md b/docs/source/tutorials/first_pw.md
@@ -1,18 +1,14 @@
 
-(tutorials-first-pw)=
+(tutorials-running-pw)=
 
-# Your first `pw.x` calculation
+# Running a `pw.x` calculation
 
 (tutorials-pw-through-cli)=
 
 ## Running a `pw.x` calculation through the CLI
 
 The simplest way to run a first `pw.x` calculation with `aiida-quantumespresso` is through the [command line interface (CLI)](#reference-cli) that ships with the package.
 
-:::{margin} ❗️Important
-This is a margin note.
-:::
-
 ```console
 aiida-quantumespresso calculation launch pw -X pw@locahost -F SSSP/1.1/PBE/efficiency
 ```

diff --git a/src/aiida_quantumespresso/tools/calculations/pw.py b/src/aiida_quantumespresso/tools/calculations/pw.py
@@ -56,13 +56,10 @@ def get_scf_accuracy(self, index=0):
 
     def get_magnetic_configuration(self, atol: float = 0.5, ztol: float = 0.05) -> AttributeDict:
         """Get the final magnetic configuration of a ``pw.x`` calculation."""
-
-        calculation = self._node.inputs.parameters.get_dict()['CONTROL'].get('calculation', 'scf')
-
-        if calculation in ['scf', 'nscf', 'bands']:
-            structure = self._node.inputs.structure
-        else:
+        try:
             structure = self._node.outputs.output_structure
+        except AttributeError:
+            structure = self._node.inputs.structure
 
         try:
             magnetic_moments = self._node.outputs.output_trajectory.get_array('atomic_magnetic_moments')[-1].tolist()