-
Notifications
You must be signed in to change notification settings - Fork 7
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
Issue when trying to run test #64
Comments
hello Michael, how did you install longstitch? If you haven't done so, I'd recommend installing longstitch+dependencies using conda & make a new environment https://anaconda.org/bioconda/longstitch thanks for your interest in ntlink/longstitch conda install -c bioconda longstitch |
Hi, I initially tried to conda install in a fully fresh environment but kept getting an error saying it required python 3.1, whereas the conda install installed 3.10. I tried force installing 3.1 into the environment but it wouldn't resolve. I replicated this error on two machines. I then tried the brew install method, which failed with a missing ntlinks_bottle_manifest, leading me to try the source code install. The source install seemingly works, and runs both longstitch and ntlinks tests up until the error I mentioned above |
Accidentally closed, sorry. |
Ive tried again to do the conda install But it hung for hours solving the environment:
i then tried installing it using mamba: Which returned:
I then tried installing ntlink from conda: Which returned:
I then tried with mamba which returned this:
I then force installed python 3.9: This worked, and after this i could succesfully install both ntlink and longstitch using mamba. I ran longstitch on my assembly and it ran to completion. |
Thank you for documenting your install attempts. I am glad to hear of the successful install+execution of longstitch on your system. For posterity (&other users), would you be able to also record the spec of your system on here? |
Sure. I'm running on Ubuntu 20.4 CPU:Ryzen9 3900X |
Thanks for the info @mrmrwinter, and I'm glad you got it working! To confirm, dependencies of LongStitch do require python 3.8 or 3.9 (as you found when installing ntLink separately), which I believe is leading to the initial error you saw with Just to reiterate for any future users who have the issue, if using
|
Hi @lcoombe, I'm running into a similar issue. Do you know what parts of ntLink are actually incompatible with Python >= 3.10? Can they easily be upgraded? |
Hi @Adamtaranto, Thanks for bringing up this issue again. At least on the ntLink side, that restriction was added by a bioconda maintainer in 2022 - there shouldn't be any ntLink code that is incompatible. |
I can't see anywhere in the bioconda recipe or in setup.py that is specifically enforcing python <= 3.10. Maybe coming from a dependency? I also noticed that ntLink can't be installed from bioconda on an M-series mac as there is no osx-arm64 build of the package. You might be able to make the package conda installable on Apple silicon macs by including "noarch: python" in the build section. I see that the bioconda recipe requires " - {{ compiler('cxx') }}", but it's not clear to me if anything is getting compiled. So might be fine as noarch? |
Hi @Adamtaranto, The python3 requirement is in the meta.yaml for the recipe: https://github.com/bioconda/bioconda-recipes/blob/86d1f45fa2731bb8a3fbdb6ef157b38a04254885/recipes/ntlink/meta.yaml#L13 Thanks for pointing out the compiler part - you're right, previous versions of ntLink did require compilation, but not since we refactored to make btllib a dependency (vs. packaged in ntLink). I have a busy week, but will aim to open a bioconda PR to update the ntLink recipe this week - I will let you know when I do. |
I have opened a PR on bioconda (bioconda/bioconda-recipes#45522) to update the recipe to noarch (thus removing the python version restriction). Hopefully it will be merged soon! |
@Adamtaranto - the update to the ntLink conda recipe has now been merged. Let me know if you are still seeing any issues! |
Hi @lcoombe - Nearly there! No error from ntlink itself, but still fails to install on M-series mac as Abyss has no osx-arm64 build. The Python <= 3.10 limitation seems to be coming from btllib, though I can't see where that is being enforced. Thanks for looking into this. |
Hi @Adamtaranto, Thanks for your patience with all this! Looking into it more, the issue is really on the bioconda side. They are working on rolling out python 3.11 and ARM builds - but the last update from them is that it is still a work in progress: bioconda/bioconda-recipes#33333 (comment) At this point, unfortunately for packages that cannot be |
Hi @Adamtaranto - In case it helps you in the meantime - I saw this issue on bioconda while keeping an eye on the repository for ARM build updates: bioconda/bioconda-recipes#45586 |
Just to update in case anyone is still interested - bioconda recently began to support builds on the osx-arm64 architecture. |
Yay! That's awesome! Have you found any good examples/tutorials for how to add osx-arm64 builds to existing recipes? |
Yeah, so it's opt-in right now, and a pretty easy addition to the Of course, the more complicated part comes when dependencies of the recipe don't yet have the appropriate builds, or if certain recipes fail the builds for whatever reason on those architectures - but hopefully over the coming months more of those problem recipes will be sorted out! |
Hi,
I am looking to run Longstitch on my assembly, and need ntlinks installed to do so. I have installed from source code, but when running the test script i receive the following error:
I can't find any mention of this error in relation to ntlinks on Google, so any help would be appreciated.
Many thanks
Edit: i am also getting the same error when trying to run longstitch on my own data
The text was updated successfully, but these errors were encountered: