-
Notifications
You must be signed in to change notification settings - Fork 0
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
Python3.10 breaks GoldenHinges #5
Comments
Thank you for the notification, I'll look into this. Looks like it can be solved with a simple |
@mcrone can you please provide the command or a reproducible example that causes the error? The tests pass in Python 3.10, so I was not able to reproduce the error. I confirm that in v3.10, your example call to |
@veghp I just ran the example code from the repo:
This is the error:
|
Thank you. The provided code runs fine for me. I use
We previously agreed that neither pkg makes a call to Biopython has recently made a possibly related change: https://github.com/biopython/biopython/blob/master/NEWS.rst#18-november-2022-biopython-180 I suspect there may be a mismatch between some of the installed packages or Python version. The above installations that worked for me are in a new conda environment. Would updating the packages solve the problem? |
Turns out that it was a bokeh problem, I still had an older version that was a dependency of another library. However, I now have an arm64 problem. I can't install Numberjack because I don't believe Mistral can be compiled for arm64.
Confirmed working on python3.11 on an x86_64 (my old laptop). |
I'm glad that updating the packages solved the problem. Yes unfortunately Numberjack is an old dependency, and we also test and use for x86 architecture and Ubuntu only. I'll close this one as the original problem is solved, and this is also a separate issue which I see you have filed here: eomahony/Numberjack#62 . |
Yeah I'm not sure Numberjack is still maintained unfortunately. There is Google ORTools that looks still active and could do the job (see the end of this page) |
There seems to be an implicit call of
from collections import Sequence
in the code. This works up until Python3.9, but then returns the errorImportError: cannot import name 'Sequence' from 'collections'
.This is because collections has been renamed to collections.abc and so this can be fixed with
import collections.abc as collections
in Python3.10.Found reference to this issue here: https://stackoverflow.com/questions/69596494/unable-to-import-freegames-python-package-attributeerror-module-collections
As I mentioned this seems to be an implicit import as there is no reference to collections in the code of either GoldenHinges or DnaFeaturesViewer.
The text was updated successfully, but these errors were encountered: