Robert T. McGibbon

Results 83 comments of Robert T. McGibbon

It looks to me like we're still seeing these warnings, and one error, when testing trio 0.19.0 for python3.10 in nixpkgs. Subscribing myself to this issue.

I thought that was fixed. What version of mdtraj do you have? Is the fix in https://github.com/mdtraj/mdtraj/pull/1706 not working, or do you have a version that doesn't include that?

I am indifferent. If you added it I wouldn't object.

@dwhswenson: is this still relevant? I'm happy to hit "merge".

Honestly I don't really know anything about the conventions for nucleic acid names, so I'm not really in a position to review much.

Hi Alex and Sander, I just wanted to give you a heads up re: https://github.com/mdtraj/mdtraj/pull/1738/files. This is one of those issues where I'm probably not able to help, and I...

This is probably the relevant code: https://github.com/mdtraj/mdtraj/blob/c0d3c985a61698d5a97c7772c373aa0a1a0300ee/mdtraj/formats/mol2.py#L170-L227 Based on the docstrings and the fact that this file has seen minimal updates in the last 7 years, I'm not particularly surprised....

For clarification: do VMD-produced mol2 files that contain systems with more than 1 atoms get loaded correctly? Is the issue specific to system with only 1 atom?

Seems like this bug probably still exists.