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

Updated linux vina tutorial #3

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Updated linux vina tutorial #3

wants to merge 4 commits into from

Conversation

DaniDelHoyo
Copy link
Collaborator

Updated version of the linux vina tutorial

@@ -53,14 +53,14 @@ Running the following commands will install meeko and ringtail in
the currently active micromamba environment.

```
pip install meeko==0.6.0a3
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

seems like ringtail install a previous version of meeko, so we should probably install the right version after (or update it from ringtail release).
Also, now this would still not work until the Meeko release from develop

Choose a reason for hiding this comment

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

ringtail does not install meeko (?). It seems like the current ringtail works with meeko develop. Maybe you already talked to someone in the lab, but I can help with specific things that didn't work for you

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

my bad, seems like it does install meeko 0.5.1 but when installing it with micromamba, not with pip. We can remove this change then

mkdir results
./vina_1.2.5_linux_x86_64 --receptor receptor.pdbqt --config receptor.box.txt --ligand phenol.pdbqt --out results/phenol.pdbqt
rt_process_vs.py write -o results.db -fp results -m vina -ai -rf receptor.pdbqt
rt_process_vs write -o results.db -fp results -m vina -ai -rf receptor.pdbqt -sr
mkdir passing_sdf
rt_process_vs.py read -i results.db -e -2 -sdf passing_sdf
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Here there seems to be a bug with the "-sdf", Diogo is aware of it

@@ -35,6 +35,7 @@ micromamba install python=3.11 -y
micromamba install pymol-open-source -y
micromamba install rdkit numpy scipy -y
micromamba install chemicalite matplotlib=3.7 pandas -y
micromamba install importlib_resources gemmi -y
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

this couple of packages are missing

Copy link
Contributor

Choose a reason for hiding this comment

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

will be fixed in release meeko v0.6

```

Running the following commands should display help messages:
```
mk_prepare_ligand.py
rt_process_vs.py
rt_process_vs -h
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Without the -h it will return an error (help message still visible but we may want to just remove the error message)

Copy link
Contributor

Choose a reason for hiding this comment

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

Good point, just opened forlilab/Ringtail#57

@@ -35,7 +35,7 @@ micromamba install python=3.11 -y
micromamba install pymol-open-source -y
micromamba install rdkit numpy scipy -y
micromamba install chemicalite matplotlib=3.7 pandas -y
micromamba install importlib_resources gemmi -y
micromamba install gemmi -y

Choose a reason for hiding this comment

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

Removed backport because forlilab/Meeko#225

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.

3 participants