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

Unify get_residue_contact_pairs() from buildgraph.py by get_IRCs() from irc.py #394

Open
DaniBodor opened this issue Mar 17, 2023 · 1 comment
Labels
good first issue Good for newcomers nice to have low priority issues stale issue not touched from too much time utils internally used modules
Projects

Comments

@DaniBodor
Copy link
Collaborator

The buildgraph function (exclusively used in some unit tests) already existed before I worked on irc feature. Later I discovered that I largely recreated that function.

The one used created for irc features is more elegantly/concisely written, so it would be nice to replace the buildgraph function with what is needed from the irc one and then call the function both for the tests and irc features

@DaniBodor DaniBodor created this issue from a note in Development (To do) Mar 17, 2023
@DaniBodor DaniBodor added good first issue Good for newcomers nice to have low priority issues utils internally used modules labels Mar 17, 2023
@DaniBodor DaniBodor moved this from To do to Nice to have in Development Mar 17, 2023
@github-actions
Copy link

This issue is stale because it has been open for 30 days with no activity.

@github-actions github-actions bot added the stale issue not touched from too much time label Apr 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers nice to have low priority issues stale issue not touched from too much time utils internally used modules
Projects
Development
Nice to have
Development

No branches or pull requests

1 participant