NCBI-Hackathons / Scan2CNV

MIT License
1 stars 0 forks source link

work on README #13

Open ekarlins opened 7 years ago

ekarlins commented 7 years ago

Work on README for this github repo. We should have something there by noon.

ekarlins commented 7 years ago

https://guides.github.com/features/mastering-markdown/

mtbrown22 commented 7 years ago

Ready to go for now!

On Tue, Mar 21, 2017 at 10:35 AM Eric Karlins notifications@github.com wrote:

https://guides.github.com/features/mastering-markdown/

— You are receiving this because you were assigned.

Reply to this email directly, view it on GitHub https://github.com/NCBI-Hackathons/Global_Screening_Arrays/issues/13#issuecomment-288097817, or mute the thread https://github.com/notifications/unsubscribe-auth/AK6r1xOxelenH4bzQNxJQeg_mGDxyCfFks5rn-BCgaJpZM4Mj2gq .

ekarlins commented 7 years ago

Thanks @mtbrown22!

ekarlins commented 7 years ago

I think the synopsis should remove mention of Illumina specific files and software: idat, egt, bpm, Genome Studio. Currently our pipeline supports Illumina data, but we're not opposed to adding other platforms in the future. Maybe we can just say something along those lines.

ekarlins commented 7 years ago

Somewhere in the documentation we may need to address that we are using the BeadArray submodule. You need to clone the repo differently for this: http://stackoverflow.com/questions/3796927/how-to-git-clone-including-submodules

ekarlins commented 7 years ago

We may want to split dependencies into groups: R packages python modules software downloads (maybe there's a better name)

mtbrown22 commented 7 years ago

Sure, we can just make lists for these then.

On Wed, Mar 22, 2017 at 7:07 AM Eric Karlins notifications@github.com wrote:

We may want to split dependencies into groups: R packages python modules software downloads (maybe there's a better name)

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/NCBI-Hackathons/Global_Screening_Arrays/issues/13#issuecomment-288366464, or mute the thread https://github.com/notifications/unsubscribe-auth/AK6r19YPtb3MVhzvreFQg6GD-lLr-Urmks5roQELgaJpZM4Mj2gq .

DCGenomics commented 7 years ago

Did you guys see the ideogram note?

On Mar 22, 2017 7:07 AM, "Eric Karlins" notifications@github.com wrote:

We may want to split dependencies into groups: R packages python modules software downloads (maybe there's a better name)

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/NCBI-Hackathons/Global_Screening_Arrays/issues/13#issuecomment-288366464, or mute the thread https://github.com/notifications/unsubscribe-auth/AFePtSziE4M_WdwVX0LOulhKZyR0WQK9ks5roQEKgaJpZM4Mj2gq .

ngiangre commented 7 years ago

I'm gonna look more at the ideograms note when I get to the NIH in like 30 minutes

Sent from my iPhone

On Mar 22, 2017, at 7:52 AM, DCGenomics notifications@github.com wrote:

Did you guys see the ideogram note?

On Mar 22, 2017 7:07 AM, "Eric Karlins" notifications@github.com wrote:

We may want to split dependencies into groups: R packages python modules software downloads (maybe there's a better name)

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/NCBI-Hackathons/Global_Screening_Arrays/issues/13#issuecomment-288366464, or mute the thread https://github.com/notifications/unsubscribe-auth/AFePtSziE4M_WdwVX0LOulhKZyR0WQK9ks5roQEKgaJpZM4Mj2gq . — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

ekarlins commented 7 years ago

The dependencies section is not completely correct and is incomplete. Just making a note here to work on it. Also, the documentation of how to run needs to be clearer.