-
Notifications
You must be signed in to change notification settings - Fork 46
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 with odd python version leading to this error #523
Comments
Hello hello! |
The VerMouTH is 0.9.2 Thanks! |
And these are the same for both? Our tests run for every python version between 3.7 and 3.11... https://github.com/marrink-lab/vermouth-martinize/blob/master/.github/workflows/run_tests.yml#L24 |
@WillPPK is this issue still active? |
Yes - I will get back to you. Sorry I was quite busy.
Will
…______________________________
Dr Tanadet Pipatpolkai
Post-doctoral scholars, Delemotte Lab
Science for Life Laboratory
Department of Applied Physics, KTH
Sweden
On 27 Jun 2023, at 14:02, Fabian Grunewald ***@***.***> wrote:
@WillPPK<https://github.com/WillPPK> is this issue still active?
—
Reply to this email directly, view it on GitHub<#523 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ARXOA7OYKTHHS6R75O5TQJ3XNLDUBANCNFSM6AAAAAAZLDRZEQ>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
These didn't work on the workstation containing: But working fine on the workstation with: Maybe it's the more recent networkx? |
@WillPPK thanks for reporting the versions. I think the error is not related to the networkx version but the changes in vermouth going from 0.7.3 to 0.9.2. Can you share the combine.pdb with us? My working hypothesis is that in the newer version the moltypes are not correctly shared between protein copies. |
The command
were run on two independent workstations. However, one with the more recent Anaconda installation show this error.
The workstation with python3.8.5 works fine, whilst the laptop with python3.10 leads to this error.
Maybe a python version and dependency annotations are needed for clarities.
The text was updated successfully, but these errors were encountered: