continuing

bluemira/codes/process/equation_variable_mapping.py

@@ -679,7 +679,7 @@ class Constraint(ConstraintSelection, Model):
     "fplhsep": 137,
     "rebco_thickness": 138,
     "copper_thick": 139,
-    "dr_tf_wp": 140,  # TODO: WTF
+    "dr_tf_wp": 140,
     "fcqt": 141,
     "nesep": 142,
     "f_coppera_m2": 143,

bluemira/codes/process/mapping.py

@@ -127,7 +127,7 @@
     "tk_ts": ("thshield_ib", "m"),
     # "tk_ts": ("thshield_ob", "m"),
     # "tk_ts": ("thshield_vb", "m"),
-    # TODO: q is not properly put in the MFILE output
+    # TODO @je-cook: q is not properly put in the MFILE output
     # This should be ok OK most of the time as q_95 is input and then
     # used as the lower bound of the q iteration variable, but this
     # should be fixed as soon as PROCESS deal with this issue on

bluemira/magnetostatics/biot_savart.py

@@ -101,7 +101,7 @@ def _check_discretisation(d_l: npt.NDArray[np.float64]):
         max_d_l = np.max(lengths)
         if max_d_l > 0.03 * total:
             bluemira_warn("Biot-Savart discretisation possibly insufficient.")
-        # TODO: Improve check and modify discretisation
+        # TODO @CoronelBuendia: Improve check and modify discretisation
 
     @process_xyz_array
     def potential(
@@ -202,7 +202,7 @@ def inductance(self) -> float:
         You probably shouldn't use this if you are actually interested in the
         inductance of an arbitrarily shaped Coordinates...
         """  # noqa: W505, E501
-        # TODO: Validate inductance calculate properly and compare stored
+        # TODO @CoronelBuendia: Validate inductance calculate properly and compare stored
         # energy of systems
         inductance = 0
         for _i, (x1, dx1) in enumerate(

bluemira/magnetostatics/finite_element_2d.py

@@ -167,7 +167,7 @@ def define_g(
             dofs = locate_dofs_topological(self.V, tdim - 1, facets)
             bcs = [dirichletbc(ScalarType(0), dofs, self.V)]
         else:
-            # TODO: we should pass directly the BCs, not the functions since
+            # TODO @je-cook: we should pass directly the BCs, not the functions since
             # dolfinx wants functions and dofs.
             bcs = (
                 dirichlet_bc_function

bluemira/materials/material.py

@@ -428,7 +428,7 @@ def __str__(self) -> str:
     def __eq__(self, other: object) -> bool:
         """Equality Check"""
         if not isinstance(other, type(self)):
-            # TODO a mixutre with only one component could be the same
+            # TODO @je-cook: a mixutre with only one component could be the same
             return False
         raise NotImplementedError("Material equality not implemented")
 
@@ -623,7 +623,7 @@ def Jc(self, B: float, temperature: float) -> float:  # noqa: N802
         # when raising a negative number to a fractional power, which in this
         # parameterisation only occurs if a non-physical (<0) current density
         # is returned.
-        # TODO: Check the above..
+        # TODO @DanShort12: Check the above..
         c = (1 - ii) ** self.beta if 1 - ii > 0 else 0
         return a * b * c
 

pyproject.toml

@@ -325,7 +325,6 @@ ignore-names = [
 "eudemo/eudemo/reactor.py" = ["ARG001"]
 "bluemira/codes/plasmod/mapping.py" = ["TD"]
 "bluemira/balance_of_plant/calculations.py" = ["TD"]
-"bluemira/fuel_cycle/*" = ["TD"]
 "{tests,eudemo/eudemo_tests}/*" = [
     "INP001",
     "PLR2004",
@@ -361,7 +360,7 @@ ignore-names = [
 "tests/geometry/test_private_tools.py" = ["E501"]
 "tests/utilities/test_opt_variables.py" = ["E501"]
 # TODO remove
-"eudemo/eudemo/*" = ["DOC201"]  #186
+"eudemo/eudemo/*" = ["DOC201", "TD"]  #186
 # see above for actual line
 # "examples/*" = ["DOC201"]  # 51
 "bluemira/utilities/*" = ["DOC201"]  # 38
@@ -373,7 +372,7 @@ ignore-names = [
 "bluemira/materials/*" = ["DOC201"]  # 62
 "bluemira/magnetostatics/*" = ["DOC201"]  # 45
 "bluemira/geometry/*" = ["DOC201"]  # 109
-"bluemira/fuel_cycle/*" = ["DOC201"]  # 19
+"bluemira/fuel_cycle/*" = ["DOC201", "TD"]  # 19
 "bluemira/equilibria/*" = ["DOC201"]  # 217
 "bluemira/display/*" = ["DOC201"]  # 25
 "bluemira/codes/*" = ["DOC201"]  # 242