-
Notifications
You must be signed in to change notification settings - Fork 6
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
Conda executable crashes #2
Comments
Hi @fplaza, Thanks for bringing this to my attention. Since the executable works, I assume you're on linux? I just tried the conda install on my Ubuntu 16 system and it appears to be fine. If you could provide your distribution and any system specifications that would be helpful for me to try and reproduce the crash. Thanks! |
I am using CentOS 8. |
I'm honestly not sure what's going on here... here's what I've tried
I need to track down a machine with no AVX2 running on x86-64. If the binary works for you, then I'd be surprised if compiling doesn't work. I'll leave this open until I figure it out, or if someone else has a similar problem |
I also seem to have the same issue on CentOS 7 without AVX2. The executable works for me as well, but the bioconda installation crashes immediately with "Illegal instruction (core dumped)". |
@edwardbirdlab thanks for confirming. Does the pre-built binary work for you too? |
Yes, the pre-built binary works without any issue. |
Hi, I am testing sylph 0.6.0 installed via conda on my Linux server, but got the same error |
@dawnmy thanks for bringing this up. I can reproduce this bug as well even on my computer. I think this is a bioconda issue, not a sylph issue: I did not change any hardware instruction behaviour between versions. I will update to version v0.6.1 and hope that bioconda fixes this. If not, it looks like I will have to ask the conda folks for help. |
@dawnmy I updated sylph to v0.6.1 on conda. It works okay on my computer now. Let me know on your end |
Thank you for the quick update. It works! |
I figured out what the issue is: -target=native rust flag is being passed to the bioconda build. So the binary will use native instructions on the bioconda machine that is building sylph. This is why AVX2 is needed for the current conda build. In the future, I will probably have to disable -target=native for conda. This will maybe make sylph slower, unfortunately... I would prefer AVX2 to be required for conda, but not AVX512, which -target=native may force. |
Hi @bluenote-1577, |
Hi @bluenote-1577,
I installed sylph using bioconda but it crashed immediately with the following error "Illegal instruction (core dumped)".
Notably, the executable available on github works.
Best,
Florian
The text was updated successfully, but these errors were encountered: