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 glob vignette #25

Merged
merged 2 commits into from
Jan 5, 2024
Merged

add glob vignette #25

merged 2 commits into from
Jan 5, 2024

Conversation

tdhock
Copy link
Owner

@tdhock tdhock commented Jan 5, 2024

vignette building fails on macbook with french locale, see below

(base) tdhock@tdhock-MacBook:~/R/nc(master*)$ bash build.sh 
Package from DESCRIPTION: nc

R Under development (unstable) (2023-12-22 r85721) -- "Unsuffered Consequences"
Copyright (C) 2023 The R Foundation for Statistical Computing
Platform: x86_64-pc-linux-gnu

R est un logiciel libre livré sans AUCUNE GARANTIE.
Vous pouvez le redistribuer sous certaines conditions.
Tapez 'license()' ou 'licence()' pour plus de détails.

R est un projet collaboratif avec de nombreux contributeurs.
Tapez 'contributors()' pour plus d'information et
'citation()' pour la façon de le citer dans les publications.

Tapez 'demo()' pour des démonstrations, 'help()' pour l'aide
en ligne ou 'help.start()' pour obtenir l'aide au format HTML.
Tapez 'q()' pour quitter R.

Le chargement a nécessité le package : grDevices
> if(require(inlinedocs))package.skeleton.dx('.')
Le chargement a nécessité le package : inlinedocs
Le chargement a nécessité le package : data.table
> 
> 
Copying nc to nc-release
Editing nc-release for CRAN submission
Building nc-release
* checking for file ‘nc-release/DESCRIPTION’ ... OK
* preparing ‘nc’:
* checking DESCRIPTION meta-information ... OK
* installing the package to build vignettes
* creating vignettes ... ERROR
--- reconstruction de ‘v0-overview.Rmd’ à l’aide de rmarkdown
--- reconstruction de ‘v0-overview.Rmd’ terminée

--- reconstruction de ‘v1-capture-first.Rmd’ à l’aide de rmarkdown
--- reconstruction de ‘v1-capture-first.Rmd’ terminée

--- reconstruction de ‘v2-capture-all.Rmd’ à l’aide de rmarkdown
--- reconstruction de ‘v2-capture-all.Rmd’ terminée

--- reconstruction de ‘v3-capture-melt.Rmd’ à l’aide de rmarkdown
--- reconstruction de ‘v3-capture-melt.Rmd’ terminée

--- reconstruction de ‘v4-comparisons.Rmd’ à l’aide de rmarkdown
--- reconstruction de ‘v4-comparisons.Rmd’ terminée

--- reconstruction de ‘v5-helpers.Rmd’ à l’aide de rmarkdown
--- reconstruction de 'v5-helpers.Rmd' termin?e

--- reconstruction de 'v6-engines.Rmd' ? l'aide de rmarkdown
--- reconstruction de 'v6-engines.Rmd' termin?e

--- reconstruction de 'v7-capture-glob.Rmd' ? l'aide de rmarkdown

Quitting from lines 123-138 [unnamed-chunk-10] (v7-capture-glob.Rmd)
Erreur : le traitement de la vignette 'v7-capture-glob.Rmd' a ?chou? avec le diagnostic :
la ligne 1 n'avait pas 4 ?l?ments
--- la reconstruction de 'v7-capture-glob.Rmd' a ?chou?

RESUME : le traitement du fichier suivant a ?chou? :
  'v7-capture-glob.Rmd'

Erreur : Vignette re-building failed.

@codecov-commenter
Copy link

Welcome to Codecov 🎉

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered ☂️

@tdhock
Copy link
Owner Author

tdhock commented Jan 5, 2024

same on non-french locale, other laptop

(base) tdhock@maude-MacBookPro:~/R/nc(glob-vignette)$ bash build.sh 
Package from DESCRIPTION: nc

R version 4.3.2 (2023-10-31) -- "Eye Holes"
Copyright (C) 2023 The R Foundation for Statistical Computing
Platform: x86_64-pc-linux-gnu (64-bit)

R is free software and comes with ABSOLUTELY NO WARRANTY.
You are welcome to redistribute it under certain conditions.
Type 'license()' or 'licence()' for distribution details.

  Natural language support but running in an English locale

R is a collaborative project with many contributors.
Type 'contributors()' for more information and
'citation()' on how to cite R or R packages in publications.

Type 'demo()' for some demos, 'help()' for on-line help, or
'help.start()' for an HTML browser interface to help.
Type 'q()' to quit R.

Loading required package: grDevices
> if(require(inlinedocs))package.skeleton.dx('.')
Loading required package: inlinedocs
Loading required package: data.table
> 
> 
Copying nc to nc-release
Editing nc-release for CRAN submission
Building nc-release
* checking for file ‘nc-release/DESCRIPTION’ ... OK
* preparing ‘nc’:
* checking DESCRIPTION meta-information ... OK
* installing the package to build vignettes
* creating vignettes ... ERROR
--- re-building ‘v0-overview.Rmd’ using rmarkdown
--- finished re-building ‘v0-overview.Rmd’

--- re-building ‘v1-capture-first.Rmd’ using rmarkdown
--- finished re-building ‘v1-capture-first.Rmd’

--- re-building ‘v2-capture-all.Rmd’ using rmarkdown
--- finished re-building ‘v2-capture-all.Rmd’

--- re-building ‘v3-capture-melt.Rmd’ using rmarkdown
--- finished re-building ‘v3-capture-melt.Rmd’

--- re-building ‘v4-comparisons.Rmd’ using rmarkdown
--- finished re-building ‘v4-comparisons.Rmd’

--- re-building ‘v5-helpers.Rmd’ using rmarkdown
--- finished re-building 'v5-helpers.Rmd'

--- re-building 'v6-engines.Rmd' using rmarkdown
--- finished re-building 'v6-engines.Rmd'

--- re-building 'v7-capture-glob.Rmd' using rmarkdown

Quitting from lines 123-138 [unnamed-chunk-10] (v7-capture-glob.Rmd)
Error: processing vignette 'v7-capture-glob.Rmd' failed with diagnostics:
line 1 did not have 4 elements
--- failed re-building 'v7-capture-glob.Rmd'

SUMMARY: processing the following file failed:
  'v7-capture-glob.Rmd'

Error: Vignette re-building failed.

@tdhock
Copy link
Owner Author

tdhock commented Jan 5, 2024

issue is that read.table does not know how to handle gz files, whereas fread does.

@tdhock tdhock merged commit c93c04f into master Jan 5, 2024
6 checks passed
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