Remove some warnings (#343)

* remove unnecessary warning about mpi4py

* remove very old deprecation warnings

* fix flake8

* really struggling today

* version bump

pyoptsparse/__init__.py

@@ -1,4 +1,4 @@
-__version__ = "2.9.4"
+__version__ = "2.10.0"
 
 from .pyOpt_history import History
 from .pyOpt_variable import Variable

pyoptsparse/pyOpt_MPI.py

@@ -8,9 +8,6 @@
 
 # Standard Python modules
 import os
-import warnings
-
-# isort: off
 
 
 class COMM:
@@ -50,20 +47,15 @@ def __init__(self):
 # and raise exception on failure. If set to anything else, no import is attempted.
 if "PYOPTSPARSE_REQUIRE_MPI" in os.environ:
     if os.environ["PYOPTSPARSE_REQUIRE_MPI"].lower() in ["always", "1", "true", "yes"]:
+        # External modules
         from mpi4py import MPI
     else:
         MPI = myMPI()
 # If PYOPTSPARSE_REQUIRE_MPI is unset, attempt to import mpi4py, but continue on failure
 # with a notification.
 else:
     try:
+        # External modules
         from mpi4py import MPI
     except ImportError:
-        warn = (
-            "mpi4py could not be imported. mpi4py is required to use "
-            + "the parallel gradient analysis and parallel objective analysis for "
-            + "non-gradient based optimizers. Continuing using a dummy MPI module "
-            + "from pyOptSparse."
-        )
-        warnings.warn(warn, stacklevel=2)
         MPI = myMPI()

pyoptsparse/pyOpt_optimization.py

@@ -4,7 +4,6 @@
 import os
 import pickle
 from typing import Callable, Dict, Iterable, List, Optional, Tuple, Union
-import warnings
 
 # External modules
 import numpy as np
@@ -227,11 +226,6 @@ def addVarGroup(
                 f"The 'nVars' argument to addVarGroup must be greater than or equal to 1. The bad DV is {name}."
             )
 
-        # we let type overwrite the newer varType option name
-        if "type" in kwargs:
-            varType = kwargs["type"]
-            # but we also throw a deprecation warning
-            warnings.warn("The argument `type=` is deprecated. Use `varType` in the future.", stacklevel=2)
         # Check that the type is ok
         if varType not in ["c", "i", "d"]:
             raise Error("Type must be one of 'c' for continuous, 'i' for integer or 'd' for discrete.")
@@ -816,10 +810,6 @@ def finalize(self):
             self._finalizeConstraints()
             self.finalized = True
 
-    def finalizeDesignVariables(self):
-        warnings.warn("finalizeDesignVariables() is deprecated, use _finalizeDesignVariables() instead.", stacklevel=2)
-        self._finalizeDesignVariables()
-
     def _finalizeDesignVariables(self):
         """
         Communicate design variables potentially from different
@@ -844,10 +834,6 @@ def _finalizeDesignVariables(self):
             dvCounter += n
         self.ndvs = dvCounter
 
-    def finalizeConstraints(self):
-        warnings.warn("finalizeConstraints() is deprecated, use _finalizeConstraints() instead.", stacklevel=2)
-        self._finalizeConstraints()
-
     def _finalizeConstraints(self):
         """
         There are several functions for this routine: