-
Notifications
You must be signed in to change notification settings - Fork 218
fixed xml error brought up in 1003 #1017
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Hi @dpryan79 Do you know why galaxy tests failed with this error: |
|
@LeilyR Hmm, my guess is that this switched to the 20.09 release of Galaxy and that has a bug in it. Let me check |
|
OK, I had hard-coded Galaxy version 18.05 into the tests a few years ago since I knew that was working. For some reason that version tag must have just been updated to remove python 3 support, even though it used to be allowed (presumably someone found a show-stopper bug). I'll try updating the tests to use a more recent version of Galaxy and see if that solves the problem. |
|
thanks a million @dpryan79! |
|
I'm not sure what the issue is with github actions, but at least azure is running. |
|
indeed, and now the errors are different and are specific to a single test. |
|
The plotPCA error is due to changes in matplotlib (or maybe numpy) versions and it's best to just edit the test file so the test passes (basically small non-zero values will change from time to time). The other failure deserves investigation. |
|
Thanks a lot! I will have a look at them. |
fixed xml error brought up in #1003