removed some comments, fixed up OpacitySolver initialization

tardis/opacities/macro_atom/macroatom_solver.py

@@ -5,9 +5,7 @@
 from tardis.opacities.macro_atom.macroatom_state import MacroAtomState
 
 
-class MacroAtomSolver(
-    object
-):  # Possibly make two classes, one for normal and one for continuum
+class MacroAtomSolver(object):
 
     initialize: bool = True
     normalize: bool = True
@@ -111,8 +109,6 @@ def solve(
             State of the macro atom ready to be placed into the OpacityState
         """
 
-        # TODO: Figure out how to calculate p_combined, Check TransitionProbabilitiesProperty in assemble_plasma, properties/base.py
-        # Make the combined transition probabilities something that is configurable in the class
         transition_probabilities = (
             self.solve_non_markov_transition_probabilities(
                 atomic_data,

tardis/opacities/opacity_solver.py

@@ -31,9 +31,10 @@ def __init__(
 
         self.line_interaction_type = line_interaction_type
         self.disable_line_scattering = disable_line_scattering
-        if (
-            self.line_interaction_type != "scatter"  # TODO: Fix this
-        ):  # Need a switch to use the continuum solver
+        if self.line_interaction_type in (
+            "downbranch",
+            "macroatom",
+        ):
             if montecarlo_globals.CONTINUUM_PROCESSES_ENABLED:
                 self.macro_atom_solver = (
                     None  # in the future will be the MacroAtomContinuum solver
@@ -87,7 +88,7 @@ def solve(self, legacy_plasma) -> OpacityState:
         elif self.line_interaction_type in (
             "downbranch",
             "macroatom",
-        ):  # Need to pass though the continuum species
+        ):
             macroatom_state = self.macro_atom_solver.solve(
                 legacy_plasma,
                 atomic_data,