Fix mypy

src/nomad_simulations/outputs.py

@@ -68,17 +68,19 @@ class Outputs(ArchiveSection):
     # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # #
     # List of properties
     # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # #
-    electronic_band_gap = SubSection(sub_section=ElectronicBandGap.m_def, repeats=True)
 
     fermi_level = SubSection(sub_section=FermiLevel.m_def, repeats=True)
 
     chemical_potential = SubSection(sub_section=ChemicalPotential.m_def, repeats=True)
 
+    electronic_band_gap = SubSection(sub_section=ElectronicBandGap.m_def, repeats=True)
+
     electronic_dos = SubSection(
         sub_section=ElectronicDensityOfStates.m_def, repeats=True
     )
 
     xas_spectra = SubSection(sub_section=XASSpectra.m_def, repeats=True)
+
     # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # #
     # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # #
 

src/nomad_simulations/physical_property.py

@@ -212,7 +212,9 @@ def _new_value(self) -> Quantity:
                     description=quant.description,
                 )
 
-    def __init__(self, m_def: Section = None, m_context: Context = None, **kwargs):
+    def __init__(
+        self, m_def: Section = None, m_context: Context = None, **kwargs
+    ) -> None:
         super().__init__(m_def, m_context, **kwargs)
         if self.iri is None:
             logger.warning(

src/nomad_simulations/properties/band_gap.py

@@ -74,7 +74,9 @@ class ElectronicBandGap(PhysicalProperty):
         """,
     )
 
-    def __init__(self, m_def: Section = None, m_context: Context = None, **kwargs):
+    def __init__(
+        self, m_def: Section = None, m_context: Context = None, **kwargs
+    ) -> None:
         super().__init__(m_def, m_context, **kwargs)
         self.name = self.m_def.name
         self.rank = []

src/nomad_simulations/properties/energies.py

@@ -38,7 +38,9 @@ class FermiLevel(PhysicalProperty):
         """,
     )
 
-    def __init__(self, m_def: Section = None, m_context: Context = None, **kwargs):
+    def __init__(
+        self, m_def: Section = None, m_context: Context = None, **kwargs
+    ) -> None:
         super().__init__(m_def, m_context, **kwargs)
         self.rank = []
 
@@ -61,7 +63,9 @@ class ChemicalPotential(PhysicalProperty):
         """,
     )
 
-    def __init__(self, m_def: Section = None, m_context: Context = None, **kwargs):
+    def __init__(
+        self, m_def: Section = None, m_context: Context = None, **kwargs
+    ) -> None:
         super().__init__(m_def, m_context, **kwargs)
         self.rank = []
 

src/nomad_simulations/properties/spectral_profile.py

@@ -55,7 +55,9 @@ class SpectralProfile(PhysicalProperty):
     #     """,
     # )
 
-    def __init__(self, m_def: Section = None, m_context: Context = None, **kwargs):
+    def __init__(
+        self, m_def: Section = None, m_context: Context = None, **kwargs
+    ) -> None:
         super().__init__(m_def, m_context, **kwargs)
         self.rank = []  # ? Is this here or in the attrs instantiation better?
 
@@ -138,7 +140,9 @@ class ElectronicDensityOfStates(SpectralProfile):
         """,
     )
 
-    def __init__(self, m_def: Section = None, m_context: Context = None, **kwargs):
+    def __init__(
+        self, m_def: Section = None, m_context: Context = None, **kwargs
+    ) -> None:
         super().__init__(m_def, m_context, **kwargs)
         self.name = self.m_def.name
 
@@ -152,7 +156,7 @@ def resolve_fermi_level(self) -> Optional[np.float64]:
         fermi_level = self.fermi_level
         # if fermi_level is None:
         #     fermi_level = resolve_output_value(self, FermiLevel)
-        # return fermi_level
+        return fermi_level
 
     def check_spin_polarized(self) -> bool:
         """
@@ -224,7 +228,9 @@ class XASSpectra(SpectralProfile):
         repeats=False,
     )
 
-    def __init__(self, m_def: Section = None, m_context: Context = None, **kwargs):
+    def __init__(
+        self, m_def: Section = None, m_context: Context = None, **kwargs
+    ) -> None:
         super().__init__(m_def, m_context, **kwargs)
         # Set the name of the section
         self.name = self.m_def.name