Skip to content

Conversation

@solmazimi
Copy link

Included receptor and ligand files, and common scripts directory that supports RBFE and swapping calculations

@solmazimi solmazimi requested a review from egallicc April 8, 2025 05:24
@solmazimi solmazimi self-assigned this Apr 8, 2025
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please remove .gitignore

LIGAND2_ATOMS = <LIG2ATOMS>
LIGAND1_CM_ATOMS = <LIG1CMATOMS>
LIGAND2_CM_ATOMS = <LIG2CMATOMS>
DISPL0_LIG1 = ' 0.0, 0.0, 0.0 '
Copy link
Collaborator

Choose a reason for hiding this comment

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

DISPL0_LIG1 and DISPL0_LIG2 are not relevant for this example. Please remove them.

ACORE = 0.062500
UBCORE = 100.0
FRICTION_COEFF = 0.500000
HMASS = 1.5
Copy link
Collaborator

Choose a reason for hiding this comment

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

Hydrogen mass repartitioning is carried out in the make_atm_system_from_rcpt_lig.py utility. Please remove HMASS from the input control file.

@@ -0,0 +1 @@
localhost,0:0,1,OpenCL,,/tmp
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove nodefile. It is created automatically by the "run" script.

#SBATCH --no-requeue
#SBATCH -t 12:00:00

. /nfs/sazimi-d/miniforge3/bin/activate atm8.2.0
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please generalize to:

. $HOME/miniforge3/bin/activate atm8.2.1

the latest release is 8.2.1

for pair in <LIGPAIRS> ; do
jobname=<RECEPTOR>-$pair
echo "Prepping $jobname"
( cd ${jobname} && python <ASYNCRE_DIR>/rbfe_structprep.py ${jobname}_asyncre.cntl ) || exit 1
Copy link
Collaborator

Choose a reason for hiding this comment

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

If the user follows the default installation instructions, the rbfe_structprep.py script is installed in the conda environment as an executable:

( cd ${jobname} &&  rbfe_structprep.py ${jobname}_asyncre.cntl )  || exit 1


jobname=<JOBNAME>

. /nfs/sazimi-d/miniforge3/bin/activate atm8.2.0
Copy link
Collaborator

Choose a reason for hiding this comment

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

Similarly to the above:

. $HOME/miniforge3/bin/activate atm8.2.1
rbfe_structprep.py ${jobname}_asyncre.cntl || exit 1
rbfe_explicit.py ${jobname}_asyncre.cntl

#displacement vector
displacement=("0.0" "70.0" "0.0")

#residue ids of the receptor that define the center of the binding site
Copy link
Collaborator

Choose a reason for hiding this comment

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

Comment that the example does not use binding site restraints

@@ -0,0 +1,289 @@
# cat r*/*.out > data
# R CMD BATCH uwham_analysis.R

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think we have improved versions of the uwham_analysis.R script that properly label axes in plots and flag large perturbation energy gaps.


# Usage: python make_atm_rbfe_system_frompdb.py <options>
# Emilio Gallicchio, 5/2023 adapted from code by Bill Swope, 11/2021
# modified by Solmaz Azimi 6/2024
Copy link
Collaborator

Choose a reason for hiding this comment

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

Update date

bbox = boundingBoxSizes(rcpt_positions + lig1_positions + lig2_positions)
bboxsizes = [ bbox[i][1]-bbox[i][0] for i in range(3) ]
padding = 2. * 1.0*nanometer
padding = 5. * 1.0*nanometer
Copy link
Collaborator

Choose a reason for hiding this comment

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

The padding is significantly larger. Why is this big? Should it be an optional parameter?

Copy link
Collaborator

Choose a reason for hiding this comment

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

@solmazimi, please address the comment above.

Solmaz Azimi added 2 commits April 9, 2025 11:22
Signed-off-by: Solmaz Azimi <solmazimi@Solmazs-MacBook-Air.local>
@egallicc
Copy link
Collaborator

@solmazimi please sync your branch with the current master branch and resolve the conflict with make_atm_system_from_rcpt_lig.py

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants