Consolidate vignettes: ~49-line metadata setup duplicated across 9 files
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 48/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- r
- Área
- documentation
Línea de trabajo
Start with the eight customize-* vignette files and forestly.Rmd; compare their metadata setup blocks and identify the topic-specific sections to preserve. Read the vignette build configuration and the guidance for knitr child documents before choosing how to share the setup. Done means the setup has one source of truth, the customization topics are consolidated as proposed, and the vignettes still build.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
The metadata-construction section is duplicated almost verbatim across the vignettes. Every customize-* vignette opens with an identical "Step 1: build your metadata" section — the full meta_adam() → define_plan() → define_population() → define_observation() → define_parameter() (×4) → meta_build() pipeline — before a short topic-specific "Step 2". forestly.Rmd builds the same metadata as well.
Evidence
- 8
customize-*vignettes +forestly.Rmd= 9 copies of the same ~49-line setup. - The "Step 1" block is byte-identical across 6 of them; the other two (
customize-ae-specific-columns,customize-listing-columns) differ only by offset / one stray blank line. - Each
customize-*page contributes only ~40 genuinely unique lines (its "Step 2").
Any change to the example metadata API or data must currently be edited in 9 places, and the near-empty pages add navigation noise.
Proposal
Two complementary moves (either alone helps; together is best):
-
Deduplicate the setup. Factor the metadata build into a single shared source — e.g. a
_setup.Rmdpulled into each vignette via a knitrchild=chunk, or a tiny exportedexample_meta()helper — so there is one source of truth for the example metadata. -
Merge the 8
customize-*pages into one "Customize the display" vignette with a section per topic (color, digits, x-limits, diff label, widths, SOC-only, toggle buttons, listing columns). They are all justae_forestly()/format_ae_forestly()argument variations. Keepforestly.Rmdas the main end-to-end tutorial.
Option 1 removes all 9 duplicated copies; option 2 additionally collapses 8 thin pages into one cohesive, navigable article.
Notes
- Coordinate with #171 (vignette
self_containedfix) — both touch the same vignette YAML/structure, so it is natural to do them together.
- Lenguaje dominante
- R
- Estrellas
- 23
- Forks
- 6
- Merge medio
- 2 d 9 h
- PR fusionados (30 d)
- 12
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de Merck/forestly
-
Installed vignettes show blank tables/plots: self_contained: no drops lt assets on R CMD buildAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 2 días
-
Allow numerical filtering for percentagesPosiblemente ocupada @yihui la tomó hace 7 días. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
Merck/forestly#166 · 1 asignado ·
Los mantenedores suelen responder en 2 días
-
Add `ae_listing_label = ...` to `prepare_ae_forestly()`Posiblemente ocupada @fukuhiro2023 la tomó hace 19 días. Abierto
Merck/forestly#152 · 1 asignado ·
Los mantenedores suelen responder en 2 días
-
A lot of if-else in `format_ae_listing()`Posiblemente ocupada @fukuhiro2023 la tomó hace 20 días. Abierto
Merck/forestly#148 · 1 comentario · 1 asignado ·
Los mantenedores suelen responder en 2 días
-
Dificultad 3/5 1-2 días Aptitud para principiantes 65/100
Los mantenedores suelen responder en 2 días
Todos los issues de Merck/forestly
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
datacarpentry/semester-biology#1272 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 75/100
DOI-USGS/dataRetrieval#934 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
pkgdown build failureAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100