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

Add text more inclusive for Bioconductor #11

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mjones01
Copy link

@mjones01 mjones01 commented Jan 8, 2018

Several packages on Bioconductor are useful beyond genomics. Currently the text indicates that the packages would not be useful to those outside genomics. Added text to indicate greater use for some packages as the current text rather limits the scope which may mislead some readers.

Several packages on Bioconductor are useful beyond genomics. Added text to indicate this as the current text rather limits the scope which may mislead some readers.
@sellorm
Copy link
Owner

sellorm commented Jan 9, 2018

Thanks Megan,

I must admit I'm a little confused by this one. The Bioconductor website itself says that it's a project that "provides tools for the analysis and comprehension of high-throughput genomic data".

Are you able to provide any more context around why the current text is misleading?

Thanks,
Mark

@mjones01
Copy link
Author

mjones01 commented Jan 9, 2018

@sellorm Feel free to ignore my edit if you prefer. It is honestly a minor comment. While Bioconductor is self described as for genomics, several of the packages are also useful for other fields. For example, the rhdf5 package. For a good explanation of the overlap of CRAN & Bioconductor packages dealing with these file types that are widely used outside of genomics, see the table and short description here: https://cran.r-project.org/web/packages/h5/vignettes/h5-Intro.html

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

Successfully merging this pull request may close these issues.

2 participants