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

statR version 2.4.0 #60

Merged
merged 75 commits into from
Feb 9, 2024
Merged

statR version 2.4.0 #60

merged 75 commits into from
Feb 9, 2024

Conversation

Thomas-Caspar-Fischer
Copy link
Contributor

  • redesigned user configurations
  • further modularized the functions used to generate worksheets
  • revised vignette on creating excel files & regenerated pkgdown

tknecht and others added 30 commits June 10, 2022 13:32
Merge branch 'dev' of https://github.com/statistikZH/statR into dev

# Conflicts:
#	.Rbuildignore
#	DESCRIPTION
#	docs/articles/Visualisierungen.html
#	docs/articles/Visualisierungen_files/figure-html/unnamed-chunk-10-1.png
#	docs/articles/Visualisierungen_files/figure-html/unnamed-chunk-11-1.png
#	docs/articles/Visualisierungen_files/figure-html/unnamed-chunk-12-1.png
#	docs/articles/Visualisierungen_files/figure-html/unnamed-chunk-13-1.png
#	docs/articles/Visualisierungen_files/figure-html/unnamed-chunk-5-1.png
#	docs/articles/Visualisierungen_files/figure-html/unnamed-chunk-6-1.png
#	docs/articles/Visualisierungen_files/figure-html/unnamed-chunk-7-1.png
#	docs/articles/Visualisierungen_files/figure-html/unnamed-chunk-8-1.png
#	docs/articles/Visualisierungen_files/figure-html/unnamed-chunk-9-1.png
#	docs/news/index.html
#	docs/pkgdown.yml
#	docs/reference/datasetsXLSX.html
#	docs/reference/display.statcol.all-1.png
#	docs/reference/display.statcol.all.html
#	docs/reference/motor_trend_car_road_tests.xlsx
#	vignettes/Visualisierungen.Rmd
Thomas Fischer and others added 4 commits July 31, 2023 12:29
Version 2.4.0
* redesigned user configurations
* further modularized the functions used to generate worksheets
* revised vignette on creating excel files & regenerated pkgdown

Co-authored-by: tknecht <[email protected]>
@Thomas-Caspar-Fischer Thomas-Caspar-Fischer marked this pull request as draft January 29, 2024 10:43
…rkaround for temporary installation (forgoes write to configstore during temp install); (0e, 0w, 0n). (#61)
@Thomas-Caspar-Fischer
Copy link
Contributor Author

The issue related to staged installation has been solved on RStudio Server and Windows systems. The config store file is allocated, and the default configuration is being loaded correctly. The checks are still failing, though.

Apparently the default config file is not being found during final installation in the GitHub action checks, and I'm not sure why.

tknecht and others added 10 commits February 6, 2024 10:41
@Thomas-Caspar-Fischer Thomas-Caspar-Fischer marked this pull request as ready for review February 7, 2024 12:39
Copy link
Contributor

@philbosch philbosch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Absolut starke Arbeit @Thomas-Caspar-Fischer!
Finde die Doku sehr, sehr gut aufgebaut und zugänglich. Nur kleine Typos/Grammatik-Hickups gefunden.

Falls es für dich funktioniert, gerne ignorieren:
Ich kann leider weder die Vignette noch die Pages builden.
> devtools::build_site() Error: ! in callr subprocess. Caused by error in purrr::map(topics, build_reference_topic, pkg = pkg, lazy = lazy, …: ! In index: 1. ℹ See $stdout for standard output. Type .Last.error to see the more details.

vignettes/Excel-Tabellen.Rmd Outdated Show resolved Hide resolved
vignettes/Excel-Tabellen.Rmd Outdated Show resolved Hide resolved
vignettes/Excel-Tabellen.Rmd Outdated Show resolved Hide resolved
vignettes/Excel-Tabellen.Rmd Outdated Show resolved Hide resolved
vignettes/Excel-Tabellen.Rmd Outdated Show resolved Hide resolved
@Thomas-Caspar-Fischer
Copy link
Contributor Author

Thomas-Caspar-Fischer commented Feb 8, 2024

Absolut starke Arbeit @Thomas-Caspar-Fischer! Finde die Doku sehr, sehr gut aufgebaut und zugänglich. Nur kleine Typos/Grammatik-Hickups gefunden.

Falls es für dich funktioniert, gerne ignorieren: Ich kann leider weder die Vignette noch die Pages builden. > devtools::build_site() Error: ! in callr subprocess. Caused by error in purrr::map(topics, build_reference_topic, pkg = pkg, lazy = lazy, …: ! In index: 1. ℹ See $stdout for standard output. Type .Last.error to see the more details.

Das Problem besteht wohl schon eine Weile. Problematisch, weil die pkgdown-Seite den Stand von Version 2.2 reflektiert.

pkgdown::build_site() # scheitert
pkgdown::build_site(examples = FALSE) # läuft

Es besteht noch ein weiteres Problem. Das pkgdown template wird vermutlich nicht verwendet (Fehlermeldung im parenttrace). Ich schau mir das noch an.
Update: devtools::build_site() läuft jetzt durch, devtools::build_vignettes() auch. Lösung war, den PATH-Eintrag im _pkgdown.yml zu löschen. Die Files im pkgdown/templates Verzeichnis werden auch so verwendet. Die Templates haben aber nicht richtig funktioniert (leere Einträge bei Autoren und Versions-Nr. von pkgdown). Deshalb habe ich die templates temporär in ein Subdirectory gelegt.

Danke für das Review. Die sprachlichen Änderungen habe ich übernommen und das Problem mit pkgdown sollte kurzfristig gelöst sein. Längerfristig müsste man allenfalls die Dateien im pkgdown/templates Verzeichnis überarbeiten.

@Thomas-Caspar-Fischer Thomas-Caspar-Fischer merged commit ad5d24b into master Feb 9, 2024
5 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.

6 participants