Skip to content
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

[4] Implement frame based backmapping #341

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

jan-stevens
Copy link
Member

In this pull request, the frame-based backmapping protocol's first version will be implemented. The most basic idea is that users provide internal reference frames for each residue, which will be applied to the (user-provided) residue templates.

Copy link
Member

@fgrunewald fgrunewald left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On first pass looks good!

@fgrunewald fgrunewald changed the title Implement frame based backmapping [4] Implement frame based backmapping Aug 16, 2023
@fgrunewald fgrunewald added the DNA_extension Part of the DNA extension project label Aug 16, 2023
@cesar-lopez-bau
Copy link

Is the backmapping implemented yet?

@jan-stevens
Copy link
Member Author

Is the backmapping implemented yet?

Hi Cesar! We're still wrapping up the final implementation of this protocol, but most of the functionality is implemented. Can you tell us which molecule you're working with? Perhaps we can assist you in using this branch?

@cesar-lopez-bau
Copy link

cesar-lopez-bau commented Dec 14, 2023 via email

@fgrunewald
Copy link
Member

@cesar-lopez-bau that sounds exciting! Are you going to share your CHARMM implementation at some point?

To address your problem I've created a discussion thread (#353) with an example on how to backmap dextran to charmm using the current functionality without the need for a frame-based backmapping. The generated coordinates look reasonable, even though I think the frame-based method can be a bit more precise. @jan-stevens will show you the frame-based version based on that example.

In case you are going to test the regular backmapping strategy, let us know if there are any problems you might encounter.

@cesar-lopez-bau
Copy link

cesar-lopez-bau commented Dec 21, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DNA_extension Part of the DNA extension project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants