Closed MartinFXP closed 5 years ago
Received a valid push; starting a build. Commits are:
6e78c38 full multi knockdown support + vignette update
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
On one or more platforms, the build results were: "WARNINGS". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.
Please see the build report for more details.
Received a valid push; starting a build. Commits are:
91e83ae bump
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
On one or more platforms, the build results were: "skipped, ERROR". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.
Please see the build report for more details.
Received a valid push; starting a build. Commits are:
a1e49fb ok. Parallel is not supported? Let's hope the wind...
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
On one or more platforms, the build results were: "WARNINGS". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.
Please see the build report for more details.
Received a valid push; starting a build. Commits are:
349bc67 minor corrections in the vignette and trying to ge...
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
On one or more platforms, the build results were: "WARNINGS". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.
Please see the build report for more details.
Received a valid push; starting a build. Commits are:
9307cac added wrapper for testing different component size...
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
On one or more platforms, the build results were: "WARNINGS". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.
Please see the build report for more details.
Received a valid push; starting a build. Commits are:
916132d improved some help files with return value lists
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
On one or more platforms, the build results were: "WARNINGS". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.
Please see the build report for more details.
Dear @hpages
I think this is now somewhat the finalised version. Unless of course I or other users find any bugs...
I hope/think the vignette now gives a good overview of what to do with this package. The warning stems from the 7+ minutes runtime of the check on the windows machine, which seems to be (not always) quite a bit slower than its mac and linux colleagues (~3 minutes). I don't think it is trivial to account for that on my end.
Thanks again for your time and all the best, Martin
Received a valid push; starting a build. Commits are:
25da374 speed increased: vignette + examples
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
On one or more platforms, the build results were: "WARNINGS". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.
Please see the build report for more details.
Received a valid push; starting a build. Commits are:
035ee8e forgot to roxygenize examples; removed sim data ob...
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
On one or more platforms, the build results were: "WARNINGS". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.
Please see the build report for more details.
Received a valid push; starting a build. Commits are:
bc1216d resaved app.rda
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
Congratulations! The package built without errors or warnings on all platforms.
Please see the build report for more details.
Received a valid push; starting a build. Commits are:
48c48c9 fixed discrete and improved some help files
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
Congratulations! The package built without errors or warnings on all platforms.
Please see the build report for more details.
Hi @MartinFXP
Thanks for the much improved vignette!
2 things still need your attention:
Your man page titles are still the same as their description. For example opening the man page for getIC
(e.g. with ?getIC
) displays a 4-line title that repeats the entire Description section.
I still don't know how the app
data set was generated. The Data Generation section of the vignette says:
The R code for generating the data object ‘app’ is shown in the accompanying R file.
What is the "accompanying R file"? Where is it? Note that the primary documentation for any function or data set defined in an R package is its man page so any information about how the app
data set was generated should appear there.
Thanks, H.
Received a valid push; starting a build. Commits are:
603e535 separated title and description include=TRUE for d...
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
On one or more platforms, the build results were: "ERROR". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.
Please see the build report for more details.
Received a valid push; starting a build. Commits are:
c63805f changed returned 1:... to seq_len(...
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
On one or more platforms, the build results were: "ERROR". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.
Please see the build report for more details.
Received a valid push; starting a build. Commits are:
368eb62 corrected line indent
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
On one or more platforms, the build results were: "ERROR". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.
Please see the build report for more details.
Received a valid push; starting a build. Commits are:
7bc0f9f 3rd time's the cha...
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
On one or more platforms, the build results were: "ERROR". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.
Please see the build report for more details.
Dear @hpages
- Your man page titles are still the same as their description. For example opening the man page for
getIC
(e.g. with?getIC
) displays a 4-line title that repeats the entire Description section.
Fixed.
- I still don't know how the
app
data set was generated. The Data Generation section of the vignette says:The R code for generating the data object ‘app’ is shown in the accompanying R file.
What is the "accompanying R file"? Where is it? Note that the primary documentation for any function or data set defined in an R package is its man page so any information about how the
app
data set was generated should appear there.
I included the relevant R code into the vignette.
Thanks again for your time to help improve the quality of the package.
Best, Martin
Martin,
Throwing 400+ lines of non-evaluated, hard to read, and impossible to run, code in your vignette is a very bad idea. The proper way to do these things is to put the code in a script that you include in your package. The script should be organized/structured/commented and with most of the code put inside functions, typically 1 function per major step but defining some other smaller helper functions is also a good idea and encouraged. The input of the script should be explained. Right now the 1st lines of the code is:
data <- read.csv("GSE92872_CROP-seq_Jurkat_TCR.digital_expression.csv", stringsAsFactors = FALSE)
Then hundreds lines later:
load(paste0(dataset, "_kegg.rda"))
load(paste0(dataset, "_llr.rda"))
However since we don't know anything about those files, we're back to square one, that is, we still have no idea how the app
data set was generated. Bioconductor takes reproducibility very seriously (as it should be).
Thanks for understanding and sorry for the inconvenience,
H.
Received a valid push; starting a build. Commits are:
cffcdc4 added function to recreate app data; added hierarc...
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
On one or more platforms, the build results were: "skipped, ERROR". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.
Please see the build report for more details.
Received a valid push; starting a build. Commits are:
74226fc a lot of code cleaning; speed improvements
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
On one or more platforms, the build results were: "skipped, ERROR". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.
Please see the build report for more details.
Dear @hpages
I cleaned and thinned out the code and put it in a function. I also did a test run to make sure it works as I totally agree, that reproducibility is very important.
app <- createApp()
?createApp # can be run with different parameters for testing; even then still takes a very long time
I don't know why I am not registered as a subscriber to the bioc-devel mailing list. I tried to register again, but nothing happened.
The other two errors look like they are out of my control.
Best, Martin
Received a valid push; starting a build. Commits are:
f1cb5df Warshall's algorithm for transitive closure; adjus...
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
On one or more platforms, the build results were: "skipped, ERROR". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.
Please see the build report for more details.
Hi @MartinFXP ,
Thanks again for all the improvements you've made to mnem.
As indicated by the build report (see output of R CMD BiocCheckGitClone
) the package source tree contains the following compilation products: src/mm.o
, src/RcppExports.o
, src/mnem.so
This is likely to be related to the compilation error you get on Windows. Generally speaking, compilation products should never be committed to git.
The error we see on Mac looks like a problem on our side. Let's ignore it.
Best, H.
Received a valid push; starting a build. Commits are:
6ac916f cleaned src
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
On one or more platforms, the build results were: "skipped, ERROR". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.
Please see the build report for more details.
Received a valid push; starting a build. Commits are:
146c068 added precluster init; fixed null node for respons...
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
On one or more platforms, the build results were: "skipped, ERROR". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.
Please see the build report for more details.
Received a valid push; starting a build. Commits are:
65dbe2d general improvements; added accuracy function; fix...
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
On one or more platforms, the build results were: "skipped, ERROR". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.
Please see the build report for more details.
Received a valid push; starting a build. Commits are:
a5b1af7 added full cluster init
Dear Package contributor,
This is the automated single package builder at bioconductor.org.
Your package has been built on Linux, Mac, and Windows.
On one or more platforms, the build results were: "skipped, ERROR". This may mean there is a problem with the package that you need to fix. Or it may mean that there is a problem with the build system itself.
Please see the build report for more details.
Dear @hpages
regarding the last remaining error (on my side). I was subscribed to the mailing list before and for some reason and without my doing was unsubscribed. Several attempts of me to subscribe again were unsuccessful. I just get the note in the browser, that my request to subscribe was successful and I should get a notification mail once the request has been dealt with. But nothing happened. How should I go on about this?
Best, Martin
Have you checked your spam or filtered emails folder? Sometimes the email gets marked as spam.
Update the following URL to point to the GitHub repository of the package you wish to submit to Bioconductor
Confirm the following by editing each check box to '[x]'
[x] I understand that by submitting my package to Bioconductor, the package source and all review commentary are visible to the general public.
[x] I have read the Bioconductor Package Submission instructions. My package is consistent with the Bioconductor Package Guidelines.
[x] I understand that a minimum requirement for package acceptance is to pass R CMD check and R CMD BiocCheck with no ERROR or WARNINGS. Passing these checks does not result in automatic acceptance. The package will then undergo a formal review and recommendations for acceptance regarding other Bioconductor standards will be addressed.
[x] My package addresses statistical or bioinformatic issues related to the analysis and comprehension of high throughput genomic data.
[x] I am committed to the long-term maintenance of my package. This includes monitoring the support site for issues that users may have, subscribing to the bioc-devel mailing list to stay aware of developments in the Bioconductor community, responding promptly to requests for updates from the Core team in response to changes in R or underlying software.
I am familiar with the essential aspects of Bioconductor software management, including:
For help with submitting your package, please subscribe and post questions to the bioc-devel mailing list.