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

Update argument handling in PDBList.py, possibly by creating a new module. #3989

Open
JoaoRodrigues opened this issue Jul 18, 2022 · 2 comments
Labels
Enhancement good first issue This should be an easy fix, suitable for beginners help wanted

Comments

@JoaoRodrigues
Copy link
Member

The command-line interface for the PDBList.py module is quite clunky, using a series of complicated ifs and sys.argv calls to parse input from the user. It would be super nicer to simply use argparse, although this will break compatibility with previous versions.

The solution here might pass through copying the code to a new module and along with other issues (#3988, #3987) upgrade PDBList.py and break backwards compability. We could then slap a deprecation warning on PDBList to warn users to move to the new code.

@RUCHI21CODES
Copy link

Can you please assign me this issue

@peterjc
Copy link
Member

peterjc commented Oct 25, 2022

@RUCHI21CODES No, there are already people working on this with open pull requests. If you can spot improvements or otherwise usefully comment on those PRs, that could be helpful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement good first issue This should be an easy fix, suitable for beginners help wanted
Projects
None yet
Development

No branches or pull requests

3 participants