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

get_ligands #53

Open
pauljmclaughlin opened this issue Jul 20, 2022 · 5 comments
Open

get_ligands #53

pauljmclaughlin opened this issue Jul 20, 2022 · 5 comments

Comments

@pauljmclaughlin
Copy link

I find that "get_ligands" is not defined (running in Jupyter Notebooks- use "!pip install pypdb" and "from pypdb import * "). On github pypdb.py the function is in comments, but it is not actually in code. Am I missing something? If not, is there another way to do this?
Thanks.

@williamgilpin
Copy link
Owner

thank you, let me check on this. are other functions running, or is just this one giving errors?
if it's just this one function, then it's an API issue I need to fix. Otherwise, I probably broke the pathing in the latest update.

@pauljmclaughlin
Copy link
Author

Yes, "get_all_info" works, for example. I am not expert enough to know if the code on GitHub is that which is picked up with "import pypdb " within Jupyter Notebooks. But in the python code on GitHub the function looks like a stub .

@williamgilpin
Copy link
Owner

Thank you! It looks like that function has been commented out in the code for the last few releases---my guess is that the function broke when RCSB changed their API last year. Do you know which version of the code you are using, or the date on which the function last worked?

@williamgilpin
Copy link
Owner

the date that the function last worked for you, I mean

@pauljmclaughlin
Copy link
Author

This is the first time I have tried to use it. In fact, I was alerted to it by a post made three years ago when someone suggested its use, in response to a question.

(https://bioinformatics.stackexchange.com/questions/8583/retrieve-id-ligand-from-pdb-file)

From this, we can surmise, but not be absolutely sure, that the respondent had used it successfully.

In the interim, I have managed to pull out the ligand names by using the API for PDB. So this is not so crucial to me now. If you are pushed for time, this is not that urgent.

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

No branches or pull requests

2 participants