Skip to content

Conversation

@roryyorke
Copy link
Collaborator

meta.yaml and conda_build_config.yaml copied and adapted from conda-forge/slycot-feedstock.

The failures in testing on OSX seem to be due to something with coveralls; the failures happen (at least on my repo - tests yet to run here) at coverage-collection time, and this happens for both conda and wheel tests. I didn't change the wheel build, and these OSX wheel tests passed completely in the most recent merged PR gh-294.

I tried to get rid of the various conda warnings encountered through the build, but ended up with a free-threaded python (!) when testing against 3.13; when I tried to change meta.yaml to prevent that, the build broke in a mysterious way.

@roryyorke roryyorke mentioned this pull request Sep 7, 2025
@roryyorke roryyorke closed this Oct 12, 2025
@roryyorke roryyorke reopened this Oct 12, 2025
@roryyorke
Copy link
Collaborator Author

closed and re-opened to get Github actions to rerun, in the hope that whatever was broken with coverage on macos is now fixed.

@roryyorke
Copy link
Collaborator Author

The CI failures are in both Windows test runs, in tests unrelated to Slycot (I think). Evidently these two optimization cases are expected to not converge, but in CI they do.

@murrayrm I'm not familiar with the optimization code, and in particular if this problem should never converge (in which case I guess there's a bug somewhere), or if it's that whatever optimizer is used can now handle this problem (but why should Windows optimization only have this property?).

I see python-control has @slycotonly markers - I'm going to see if pytest can be told to run only those; it seems like it would be a reasonable thing to do anyway in Slycot, to reduce CI time and decouple from non-Slycot python-control code.

  FAILED control/tests/optimal_test.py::test_optimal_doc[shooting-3-None-xfail] - assert not True
   +  where True =      message: Optimization terminated successfully\n     success: True\n      status: 0\n         fun: 98.05537313350811\n...74e+01  1.000e+02]\n               [-2.000e+00  3.419e+00  1.991e+00]\n               [ 0.000e+00  1.277e-01 -2.813e-01]].success
  FAILED control/tests/optimal_test.py::test_optimal_doc[shooting-3-zero-xfail] - assert not True
   +  where True =      message: Optimization terminated successfully\n     success: True\n      status: 0\n         fun: 98.05537313350811\n...74e+01  1.000e+02]\n               [-2.000e+00  3.419e+00  1.991e+00]\n               [ 0.000e+00  1.277e-01 -2.813e-01]].success
  = 2 failed, 4255 passed, 39 skipped, 7 deselected, 10 xfailed, 1 xpassed, 299 warnings in 431.86s (0:07:11) =

@roryyorke
Copy link
Collaborator Author

I've opened python-control/python-control#1182 to allow only Slycot-dependent tests to be selected in pytest.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant