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

Replace to_dataframe() by new stats export methods #233

Closed
3 tasks
gplssm opened this issue Sep 1, 2017 · 1 comment
Closed
3 tasks

Replace to_dataframe() by new stats export methods #233

gplssm opened this issue Sep 1, 2017 · 1 comment
Assignees
Milestone

Comments

@gplssm
Copy link
Contributor

gplssm commented Sep 1, 2017

  • Find all usages of to_dataframe()
  • Replace by according new stats export function and adjust dependent code
  • Make sure to_dataframe() does not contain any figures that are not included in the new functions and delete to_dataframe()
@miguelparadacontzen
Copy link

Like we spoke, better not delete to_dataframe() because it gives information (a list of nodes and edges with some attributes) that is different than that of the stats functions and might be useful. in particular, we use it for the regression test function, in order to compare if two networks are equal (if they have the same nodes and edges with the same attributes)

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

No branches or pull requests

2 participants