Skip to content

Conversation

@LeilyR
Copy link
Contributor

@LeilyR LeilyR commented Nov 16, 2020

fixed xml error brought up in #1003

@LeilyR
Copy link
Contributor Author

LeilyR commented Nov 16, 2020

Hi @dpryan79 Do you know why galaxy tests failed with this error:
ERROR: Your Python version is: 3.7 Galaxy is currently supported on Python 2.7 only. To run Galaxy, please download and install a supported version from python.org. If a supported version is installed but is not your default, getgalaxy.org contains instructions on how to force Galaxy to use a different version.

@dpryan79
Copy link
Collaborator

@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

@dpryan79
Copy link
Collaborator

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.

@LeilyR
Copy link
Contributor Author

LeilyR commented Nov 18, 2020

thanks a million @dpryan79!

@dpryan79
Copy link
Collaborator

I'm not sure what the issue is with github actions, but at least azure is running.

@LeilyR
Copy link
Contributor Author

LeilyR commented Nov 18, 2020

indeed, and now the errors are different and are specific to a single test.

@dpryan79
Copy link
Collaborator

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.

@LeilyR
Copy link
Contributor Author

LeilyR commented Nov 18, 2020

Thanks a lot! I will have a look at them.

@LeilyR LeilyR changed the base branch from develop to fix_1013 November 30, 2020 15:56
@LeilyR LeilyR merged commit 4eb79d5 into fix_1013 Nov 30, 2020
@LeilyR LeilyR deleted the fix_1003 branch February 15, 2021 12:45
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.

2 participants