Fix formatting issues noted in review

Author: Bartek Florczyk Vik (CCI RPT RES1)

docs/make_plots.py

@@ -552,9 +552,6 @@ def make_gasoil_condensate_cid2(show: bool = True) -> None:
     if show:
         plt.show()
 
-    # check_table(wog.gasoil.table)
-    # check_table(wog.wateroil.table)
-
 
 def make_wateroil_condensate_idc2(show: bool = True) -> None:
     plt.xkcd()
@@ -699,12 +696,6 @@ def make_gaswater_condensate_icd2(show: bool = True) -> None:
         arrowprops={"arrowstyle": "->"},
         xytext=(swl + 0.1, krgend - 0.1),
     )
-    # axes.annotate(
-    #     "KRWEND",
-    #     xy=(1 - sgrw, krwend),
-    #     arrowprops={"arrowstyle": "->"},
-    #     xytext=(1 - sgrw - 0.13, krwmax - 0.15),
-    # )
     axes.annotate(
         "KRWEND",
         xy=(1 - sgrw, krwend),

docs/modelling.rst

@@ -134,9 +134,10 @@ dropout can occur it should be modelled as a three-phase simulation.
       -
 
 Wet gas/condensate field (ICD2, CID2, IDC2)
----------------------------------------------------
+-------------------------------------------
 
-This field scenario is for a three-phase Eclipse simulation allowing for condendsate dropout (oil displacing gas) in addition to water displacing both gas and oil. 
+This field scenario is for a three-phase Eclipse simulation allowing for condendsate
+dropout (oil displacing gas) in addition to water displacing both gas and oil. 
 The figure below illustrates different regions and processes in a wet gas/condensate field.
 
 .. image:: images/wet-condensate_field_illustration.png
@@ -157,7 +158,7 @@ Following processes are associated with the regions depicted in the figure above
     * - 3
       - CID2 & IDC2
 
-Three-Phase Consistency Requirements:
+Three-phase consistency requirements:
 
 - **SWL**: The same input must be used for all three flow scenarios.
 - The following relationships must hold: **SGRW** = **SGCR** = **SGRO** = **SORW** 
@@ -207,7 +208,7 @@ This is the main process for a gas/condensate field.
 
 Oil displacing gas (CID2)
 ~~~~~~~~~~~~~~~~~~~~~~~~~~
-This is the process for condendsate dropout.
+This is the process for condensate dropout.
 
 .. image:: images/gasoil-condensate-cid2.png
     :width: 600

src/pyscal/factory.py

@@ -1411,12 +1411,12 @@ def check_deprecated(params: dict[str, Any]) -> None:
     # Remove this block in pyscal 1.x
     if "krowend" in params and "kroend" not in params:
         raise ValueError(
-            "krowend is not supported by pyscal for WaterOil and GasOil objects."
+            "krowend is not supported by pyscal for WaterOil and GasOil objects. "
             "Use kroend"
         )
     if "krogend" in params and "kroend" not in params:
         raise ValueError(
-            "krogend is not supported by pyscal for WaterOil and GasOil objects."
+            "krogend is not supported by pyscal for WaterOil and GasOil objects. "
             "Use kroend"
         )
 

tests/test_factory.py

@@ -1128,10 +1128,10 @@ def test_check_deprecated_krowgend():
     """Up until pyscal 0.5.x, krogend and krowend were parameters
     to the oil curve parametrization for WaterOil and GasOil. From
     pyscal 0.6.0, krogend and krowend are merged to kroend.
-    After pyscal 0.8 presence of krogend and krowend is a ValueError
+    After pyscal 0.8 presence of krogend and krowend is a ValueError.
     After pyscal 0.16 presence krogend and krowend is accepted as
     parameters for create_wateroilgas, but is still ValueError for
-    create_water_oil and create_gas_oil is a
+    create_water_oil and create_gas_oil.
     """
     with pytest.raises(ValueError):
         create_water_oil({"swl": 0.1, "nw": 2, "now": 2, "krowend": 0.4})