NonConvergingErrorHandler add doc string comment suggesting setting A…

…MIX/BMIX doesn't seem to help much

src/custodian/vasp/handlers.py

@@ -1560,6 +1560,9 @@ class NonConvergingErrorHandler(ErrorHandler):
     Check if a run is hitting the maximum number of electronic steps at the
     last nionic_steps ionic steps (default=10). If so, change ALGO using a
     multi-step ladder scheme or kill the job.
+
+    In some cases (ALGO=All or ALGO=Normal and ISMEAR < 0), this handler also changes AMIX
+    and BMIX but unsure if this helps much. Some anecdotal evidence suggests it doesn't.
     """
 
     is_monitor = True

src/custodian/vasp/utils.py

@@ -18,6 +18,7 @@ def _estimate_num_k_points_from_kspacing(structure: Structure, kspacing: float)
     Inputs:
         structure (Structure): structure used in the calculation
         kspacing (float): KSPACING used in the calculation
+
     Returns:
         tuple[int,int,int] : the number of estimated k-points on each axis.