-
Notifications
You must be signed in to change notification settings - Fork 17
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
[BUG] Vignette building on macOS fails without helpful logs #27
Comments
Hi @flying-sheep, I don't appreciate the tone of your message and it doesn't make me want to help you. You might benefit from checking these tweets https://gist.github.com/jimhester/9b5461108f93162e15c42684076de82f#file-tweets-md. Having said that, in order to run The GitHub action workflow provided by Going into the specifics of your error, from your log, you can see that there's an error at lines 119-221 from
You could also choose to ignore macOS by commenting out https://github.com/theislab/destiny/blob/91db4509ce9daaedbcb99d9bc8f5b5e5f63247f2/.github/workflows/check-bioc.yml#L58. Anyway, enjoy the ride down this rabbit hole. Best, |
Hi @lcolladotor, I did write that report while being frustrated by a intransparent decision being made by Bioconductor’s leadership forcing me to do churn work. My package and its vignettes worked fine for years until they basically told me “we’re not going to tell you why, but you must now use Rmd for your vignettes, no discussion”. This feels frustrating because I was given no chance to see their side and why my vignette building process caused them problems, despite me asking several times. Without any reasoning to go by, I feel like being ordered to do work. It seems that some of that frustration leaked into my tone, and for that I’m sorry, it’s not your fault.
Well, at least that’s the intention right? The command output I pasted above is from an invocation with that option set, and it still seems to build the vignettes, right?
Yes, that’s my assumption as well. I filed this issue to get help with debugging this and in turn to help you update the system dependencies in your GH action template, if our assumption turns out to be correct. I found the issue you linked (thanks!), but disregarded the Another possible fix might be tidyverse/ggplot2#2252 (comment) (a missing font). If we can get the GH action to emit warnings while building vignettes, we have a chance to fix things like this. But I have no idea how to do this.
Sure, but that would defeat the purpose of the unit tests. Since Bioconductor’s build bots only run every few days, it’s important to find possible errors early. |
Hi again, Regarding
Regarding the macOS-specific I see at https://bioconductor.org/checkResults/release/bioc-LATEST/destiny/ that on the Bioconductor release branch, your package As for switching vignette engine builders to Best, |
Thank you for the detailled explanation regarding vignette building in
Sure, but if if it breaks on the bioconductor build bots (which can still happen, see below) I need to fix this now. And to be able to debug this now, I think I need to be able to see warnings raised during
At the time of writing it says “Snapshot date 2022-01-20” which means that it ran on code from 3 days ago. 3 days ago, the code still used my old vignette building system. From all that I can see (e.g. those green checkmarks) it works fine. Still, the Bioconductor people told me I have to switch away from that one to Rmd. And in that process I encountered the errors mentioned above, which I’m not able to debug without help. |
See this for more details: - tidyverse/ggplot2#2252 (comment) - lcolladotor/biocthis#27
Hi @lcolladotor, since you marked this as completed: Could you please mention which action version (or if unreleased: PR) is this completed in? Thank you! |
Your If you have questions about this error, post them on the bioc-devel mailing list where other Bioconductor contributors or Core members might be able to help you. |
I’m very thankful that you brought this up on my behalf, but the problem I ran into is not the reason I filed this issue. If I understand correctly, I filed this issue assuming that
|
lcolladotor/biocthis#27 so changed to `error=TRUE` in RMarkdown doc for `ceef.Rmd` to see if this will just get it to pass. This seems to be something to do with fonts or trying to plot outside the plotting area. The log says its using UTF-8 so perhaps need to explicityl say this beforehand? The vignettes all build fine inside RStudio including using `devtools::check_mac_version()`
Context
I’m using the biocthis GH action basically unchanged, and if fails on macOS while trying to build the vignettes, with nothing but the unhelpful and extremely terse message
Since the workflow you generate is extremely complicated, and the error log isn’t helpful, I can’t debug this.
Workflow definition: https://github.com/theislab/destiny/blob/91db4509ce9daaedbcb99d9bc8f5b5e5f63247f2/.github/workflows/check-bioc.yml
Code
IDK what
--no-vignettes
in your workflow is supposed to mean, asrcmdcheck::rcmdcheck()
clearly still tries to build the vignettes.Unfortunately it does so without showing any warnings or tracebacks.
Where are the vignettes supposed to be built? How do I see the full logs when building them fails?
Small reproducible example
I’ll make one once you helped me get a proper log from the vignette building process.
R session information
Session info
The text was updated successfully, but these errors were encountered: