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

Split netdiffuseR into multiple packages #22

Open
gvegayon opened this issue Oct 23, 2017 · 7 comments
Open

Split netdiffuseR into multiple packages #22

gvegayon opened this issue Oct 23, 2017 · 7 comments

Comments

@gvegayon
Copy link
Member

gvegayon commented Oct 23, 2017

I like the idea of following what statnet has done. Having a baseline "common" package with the basic structure and methods, and from that deploying a set of offspring packages with specific goals. It makes it easier to maintain and easier to the user to figure out what they need.

At first, I can think of the following structure:

  • diffnet: Core structures and methods
  • diffnet.rand: Random graph generation
  • diffnet.permute: Permutation based tests
  • diffnet.plot: Plot Infect suscept,
  • netmatch: Aral's network matching method
  • diffnet.data: med innov, br farmers, etc.
  • diffnet.stats: threshold, exposure, struct_equiv, Infect suscept, etc.

And netdiffuseR as an R package that installs all of it.

What do you think @twvalente @srpitts?

@srpitts
Copy link
Collaborator

srpitts commented Oct 24, 2017

@gvegayon That sounds good to me! If you want to discuss breaking it apart, I'm happy to. Anything that makes it easier to maintain is a plus. Something I always want to do on this is contribute to vignettes & help files!

@gvegayon
Copy link
Member Author

gvegayon commented Oct 24, 2017 via email

@srpitts
Copy link
Collaborator

srpitts commented Oct 31, 2017

Good point--- we should look more into EpiModel. I have a goal of after next Monday being able to contribute again to things like this. There's also this R package: https://cran.r-project.org/web/packages/spatialnbda/index.html that we should look into as well, that we were looking at a long while back.

@gvegayon
Copy link
Member Author

gvegayon commented Nov 1, 2017 via email

gvegayon added a commit that referenced this issue Nov 2, 2017
@gvegayon
Copy link
Member Author

gvegayon commented Nov 2, 2017

By the way, here I have a work-in-progress document going though EpiModel https://github.com/USCCANA/netdiffuseR/blob/master/playground/epimodel/epimodel_notes.md#notes-on-the-epimodel-r-package

@srpitts
Copy link
Collaborator

srpitts commented Nov 2, 2017

I'm very interested in where this is going--- and sorry I didn't send the papers on NBDA--- I actually have those from long ago. Next week I hope to begin looking into this further. I have a deadline on Monday, but will redirect my free time that I spend on work to this after then. I'm interested in the ability to specify how the diffusion process works with the EpiModel package.

@gvegayon
Copy link
Member Author

Hey, so I just made a last commit on this side project that should be the starting point for splitting netdiffuseR, readnet. The idea is that the survey-reading functions (and other data import-like functions) to be hold here, and use netdiffuseR as a wrapper around this. Currently, the only functions that I have are survey_to_edgelist (which is similar to what is survey_to_diffnet) and a couple of other functions to coerce the objects into igraph objects. I'm kind of stuck here as I'm not fully sure of what are the needs of the potential users... any clues?

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