CI: explicitly include defaults channel#1129
Conversation
This change is motivated by that of commit 814f414
7499bd5 to
3079721
Compare
|
@slivingston Despite the claim that all checks completed, in fact the conda-based tests did not complete. See OS/BLAS test matrix run. It looks like the conda test matrix gets created but does not run. The error is ??? |
Good catch. Yes, I am investigating this and a similar change for Slycot on which I am working. |
|
It looks like the same hidden error happened 2 weeks ago in a different PR: https://github.com/python-control/python-control/actions/runs/13107250760 |
8f33d82 to
9b228b1
Compare
|
Commit 1ceeeea deleted required imports in a test matrix generation script, resulting in an uncaught exception. The backtrace from Python would then be used as the saved output in a step of the workflow file. One of the next jobs tries to parse this output as JSON, hence the error from JsonReader. This is treated as a syntax error in the workflow file, thus an incomplete rather than a failed job. I modified the workflow files so that errors from the test matrix generation scripts surface clearly as failed jobs. |
7d4a01f to
baaae00
Compare
|
...actually, still debugging. I will finish it in the morning. |
baaae00 to
04d729e
Compare
04d729e to
6c628f0
Compare
|
This is ready for review. |
|
Sorry for the extra delete in 1ceeeea |
@bnavigator No worries! I only intended to find the origin, not accuse anyone. |
This change is motivated by that of commit 814f414 (from PR #1128)