fix tests, figured out issue with matplotlib on windows 3.13, unpinned OSQP (#200)

* fix tests

* Update test.yml

* Update pyproject.toml

* Update __main__.py

* Update __main__.py

* Update __main__.py

* Update __init__.py

* test warnings

* retry

* retry

* retry

* seems to have worked

Author: enzbus

.github/workflows/test.yml

@@ -117,7 +117,7 @@ jobs:
       fail-fast: false
       matrix: # https://github.com/actions/runner-images
         os: [ubuntu-latest, windows-latest, macos-latest, macos-13] # last 2 are aarch64 and amd64
-        python-version: ['3.8', '3.9', '3.10', '3.11', '3.12', '3.13.3'] # bug in 3.13.4 affecting Windows: github.com/python/cpython/issues/135151
+        python-version: ['3.8', '3.9', '3.10', '3.11', '3.12', '>=3.13.0 <=3.13.13 || >=3.13.5 <3.14.0'] # bug in 3.13.4 affecting Windows: github.com/python/cpython/issues/135151
         exclude:
           - os: macos-latest # aarch64
             python-version: '3.8'
@@ -164,6 +164,10 @@ jobs:
         if: ${{ matrix.python-version == '3.12'}}
         run: python -m pip install numpy==2.1.2 scipy==1.14.1 matplotlib==3.9.2 cvxpy==1.5.3 pandas==2.2.3 osqp==0.6.7.post3 ecos==2.0.14 scs==3.2.7 pillow==11.2.1
 
+      - name: If running on python 3.13 install any dependency whose latest version is causing upstream issues
+        if: ${{ matrix.python-version == '>=3.13.0 <=3.13.13 || >=3.13.5 <3.14.0'}}
+        run: python -m pip install pillow==11.2.1 # 11.3 throws a deprecationwarning from matplotlib; works fine, https://github.com/matplotlib/matplotlib/pull/30221
+        
       - name: Install package
         run: |
               python -m pip install .[test]
@@ -214,4 +218,4 @@ jobs:
         if: startsWith(github.ref, 'refs/tags/')
         uses: pypa/gh-action-pypi-publish@release/v1
         with:
-          skip-existing: true
+          skip-existing: true

cvxportfolio/policies.py

@@ -948,6 +948,11 @@ def values_in_time_recursive( # pylint: disable=arguments-differ
             warnings.filterwarnings(
                 "ignore", category=UserWarning,
                 message='Solution may be inaccurate')
+            # this one is thrown by OSQP 1.0
+            warnings.filterwarnings(
+                "ignore", category=PendingDeprecationWarning,
+                message='The default value of raise_error ' +
+                'will change to True in the future.')
             # cvxpy 1.4 and 1.5 FutureWarnings about ECOS deprecation
             if cp.__version__[:3] in ['1.4', '1.5']:
                 warnings.filterwarnings("ignore", category=FutureWarning)

cvxportfolio/tests/__main__.py

@@ -97,9 +97,9 @@ def parseArgs( # pylint: disable=missing-param-doc,missing-type-doc
             self.testRunner = TextTestRunnerAllowDownloadError
 
 if __name__ == '__main__': # pragma: no cover
+    # We only test turning warnings to exceptions
+    # on current Python versions
     if sys.version_info.minor > 9:
-        # DeprecationWarning's may be thrown
-        # when running with old versions
         mainOptionallyAllowDownloadError(warnings='error')
     else:
         mainOptionallyAllowDownloadError()

pyproject.toml

@@ -32,9 +32,6 @@ dependencies = [
     "requests; python_version<'3.9'",
     # "multiprocess", # robustifies usage w/ 3rd party modules, used if installed
     "scs", # it's hardcoded as fallback solver if numerical errors
-    # following is sub-dependency; need to pin it b/c w/ recent release
-    # problematic intf changes, confusing warnings, not resolved by caller yet
-    "osqp<1.0.0"
     ]
 
 [project.optional-dependencies]