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

Combine all the network functions into a class #2

Closed
johnlees opened this issue Apr 16, 2021 · 2 comments · Fixed by #17
Closed

Combine all the network functions into a class #2

johnlees opened this issue Apr 16, 2021 · 2 comments · Fixed by #17
Assignees
Labels

Comments

@johnlees
Copy link
Member

No description provided.

@johnlees johnlees added the code label Apr 16, 2021
@johnlees johnlees self-assigned this Apr 16, 2021
@johnlees johnlees transferred this issue from bacpop/PopPUNK Aug 4, 2022
@johnlees
Copy link
Member Author

johnlees commented Aug 4, 2022

I think the desired interface for this package is eventually going to look something like this:

from pp_netlib import pp_graph
pp_graph.load()
pp_graph.save()
pp_graph.create()
pp_graph.convert()
pp_graph.assign()
pp_graph.prune()

We can talk about how to achieve this, but I think probably making a python class is best

@bruhad-dave
Copy link
Collaborator

Have this interface as a WIP

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants