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

Adapt protein mutation setup pipeline to handle glycosylated RBD:ACE2 #777

Closed
wants to merge 33 commits into from

Conversation

zhang-ivy
Copy link
Contributor

@zhang-ivy zhang-ivy commented Mar 15, 2021

This is a temporary fix that allows generation of htfs for glycosylated RBD:ACE2, since the amber glycan forcefield is not currently compatible with OpenMM.

Overview of the setup pipeline for glycosylated RBD:ACE2:

  1. Obtain apo RBD and apo ACE2 PDBs (does not need glycan bonding to be there). Edit the PDBs for tleap if necessary
  2. Solvate and parametrize 1) apo RBD and 2) RBD:ACE2 in tleap
  3. Load prmtop and inpcrd files from 2) into OpenMM and correct chain/residue ids in topology
    Output: old topology (corrected prmtop), old positions (inpcrd), old system (prmtop)
  4. Load the tleap output prmtop/inpcrd from 2) into MDAnalysis and correct chain/residue ids and remove solvent —> save as PDB
  5. Mutate PDB from 4) using pymol mutate.py
  6. Load mutated PDB and renumber the atoms based on the atom ordering in new_topology. Then copy the solvent positions from the old positions —> save as mutated and solvated PDB
  7. Correct the mutated and solvated PDB for tleap
  8. Parametrize in tleap (do not solvate)
    Output: new topology (created from old topology), new positions (inpcrd), new system (prmtop)

Summary of changes
In rjmc/topology_proposal.py:

  • Created PointMutationEngineRBD (subclass of PointMutationEngine)
  • Sped up _construct_atom_map

In app/relative_point_mutation_setup.py:

  • Created PointMutationExecutorRBD (subclass of PointMutationExecutor)

Created utils/rbd.py:

  • Contains utility functions used in PointMutationEngineRBD and PointMutationExecutor

TO DO:

  • Create /data/rbd_ace2/ dir and add template tleap.in and mutate.py (+ other necessary scripts/files)
  • Add examples to docstrings

@zhang-ivy zhang-ivy requested a review from dominicrufa March 15, 2021 21:14
@codecov
Copy link

codecov bot commented Mar 16, 2021

Codecov Report

Merging #777 (885879f) into master (82dd766) will decrease coverage by 1.37%.
The diff coverage is 6.84%.

@@ -2043,6 +2047,7 @@ def _construct_atom_map(self,
new_oemol_res_copy : openeye.oechem.oemol object
copy of modified new oemol
"""
_logger.debug("In construct atom map")
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@zhang-ivy delete this

@jchodera jchodera added the do not merge Do not merge this PR label May 24, 2021
@jchodera
Copy link
Member

Just noting that we don't want to merge this PR, and instead want to finish the work converting th glycam force field converted to OpenMM's ffxml in the short term.

Longer term, the Open Force Field Initiative will provide fully self-consistent general force fields that work for glycosylated residues transparently.

@mikemhenry
Copy link
Contributor

mikemhenry commented Jul 11, 2023

Since openmm/openmmforcefields#150 got merged, do we need this any more @zhang-ivy @jchodera ?

@zhang-ivy
Copy link
Contributor Author

Closing this as we've converted the glycam ff to an ffxml, so this hack is no longer necessary

@zhang-ivy zhang-ivy closed this Jul 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge Do not merge this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants