vermouth-martinize icon indicating copy to clipboard operation
vermouth-martinize copied to clipboard

Published 20 hours ago • verified publisher icon marrink-lab

GO IDP bugfix

Open csbrasnett opened this issue 10 months ago • 2 comments

Need to add secstruc to the water bias application, otherwise can't apply across a whole protein (starting resid has to be > 1). this should fix that. And add preprint citation.

csbrasnett avatar Apr 23 '24 13:04 csbrasnett

I'm not certain, but I think adding again there would be degenerate anyway? VirtualSiteCreator gets called both in the Go model and if args.water_bias is used, so I think just adding it there should be fine.

Happy to write an integration test. It strikes me this might be a good opportunity to deal with #585 at the same time, so I can do that as well?

csbrasnett avatar Apr 23 '24 15:04 csbrasnett

@csbrasnett I'd rather have 2 separate PRs, so that we can release this bug fix in near time

fgrunewald avatar Apr 23 '24 15:04 fgrunewald

This should bring things in line with polyply for generating topologies of idps. Maybe this is the right approach, maybe it isn't, but here's some kind of implementation either way. Happy to be advised whatever.

To do:

  1. properly check against polyply
  2. make some tests
  3. maybe propagate secondary structure for automatic id/suggestions of IDRs?

csbrasnett avatar Apr 24 '24 15:04 csbrasnett