-
Notifications
You must be signed in to change notification settings - Fork 10
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
Build fix #75
Build fix #75
Conversation
Right: we should just finalize that package, and submit to Bioc, when time will allow.. .. and meanwhile indeed just remove from Suggests.. perhaps examples can be moved to miaTime package if useful. |
Yes, sure I just disabled miaTime examples and test using data from it because it seems to cause an error miaTime is using package SEtools which is using biocGenerics::rowSums --> rowSums has moved to MatrixGenerics Bioconductor/BiocGenerics@5d95680 |
Ok good to open issues in miaTime but seems you just opened some already. |
This reverts commit f76468f.
miaTime cannot even be in "suggests" as it causes an error in build https://master.bioconductor.org/checkResults/3.17/bioc-LATEST/miaViz/nebbiolo1-checksrc.html