-
Notifications
You must be signed in to change notification settings - Fork 4
OpenMS support #105
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
OpenMS support #105
Conversation
|
External library pyopenms will require. |
|
Hi @Arslan-Siraj, also thank you here for your contribution. I would also need a unit test so please add it or send me a minimal example file so that I can write one. |
|
I think the issue regarding the failing tests is most likely due to this OpenMS/OpenMS#7691. This was fixed in |
|
Hi @t0mdavid-m I cannot install version 3.2.0.1 as it doesn't exist on some package index it seems. I see it in pypi, but I cannot add it to the dependencies as I get: This is why I specifically avoided installing that version. Can you hint me towards the problem? |
|
It looks like we only rebuilt the wheels for windows: https://pypi.org/project/pyopenms/3.2.0.1/#files Would it be acceptable for you to solve it by setting the requirement to |
|
Ok perfect. I realized last time it downloaded version 3.2.0, which still had the problems. This time, it updated to use 3.2.1. I fixed all the remaining issues with tests except the windows issue, which still persists. This has to do with some dynamic DLL linking that doesn't work. Can you check this: |
|
@picciama can you check please? It seems version pyopenms 3.0.0 passed all checks for now. |
I am trying to add OpenMS support for search engines (.idXML format).
Target to add OpenMS in Oktoberfest.
what I did?
here is dependent pull request initiate OpenMS support spectrum_fundamentals#101