censo2017: Base de Datos de Facil Acceso del Censo 2017 de Chile (2017 Chilean Census Easy Access Database)

[pachadotdev]

Persona Encargada: Pachá (@pachamaltese)
Repositorio: https://github.com/pachamaltese/censo2017
Versión Enviada: 0.2
Editor: @melvidoni
Reviewers: @mpaulacaldas, @FvD

Due date for @mpaulacaldas: 2021-02-25

Due date for @FvD: 2021-02-25
Archivo: TBD 2021-2-25
Versión Aceptada: TBD

---

• Pega el archivo DESCRIPTION completo dentro del siguiente bloque de código.

Package: censo2017
Title: Base de Datos de Facil Acceso del Censo 2017 de Chile
    (2017 Chilean Census Easy Access Database)
Version: 0.4
Authors@R: c(
        person(given = "Mauricio",
               family = "Vargas",
               role = c("aut", "cre"),
               email = "mvargas@dcc.uchile.cl",
               comment = c(ORCID = "0000-0003-1017-7574")),
        person(given = "Juan",
               family = "Correa",
               role = c("ctb")),
        person(family = "Instituto Nacional de Estadisticas (INE)",
               role = c("dtc")
        )
    )
Description: Provee un acceso conveniente a mas de 17 millones de registros
    de la base de datos del Censo 2017. Los datos fueron importados desde
    el DVD oficial del INE usando el Convertidor REDATAM creado por Pablo De
    Grande. Esta paquete esta documentado intencionalmente en castellano
    asciificado para que funcione sin problema en diferentes plataformas.
    (Provides convenient access to more than 17 million records from the
    Chilean Census 2017 database. The datasets were imported from the official
    DVD provided by the Chilean National Bureau of Statistics by using the
    REDATAM converter created by Pablo De Grande and in addition it includes the
    maps accompanying these datasets.)
URL: https://pacha.dev/censo2017/
BugReports: https://github.com/pachamaltese/censo2017/issues/
License: CC0
Encoding: UTF-8
LazyData: true
Roxygen: list(markdown = TRUE)
RoxygenNote: 7.1.1
Imports:
    DBI,
    duckdb,
    httr,
    tibble,
    purrr,
    cli,
    crayon,
    rstudioapi,
    tools
Suggests:
    testthat,
    covr,
    knitr,
    dplyr,
    dbplyr
Depends:
    R (>= 4.0)

Alcance

• Por favor, indica qué categoría(s) aplican a este paquete. Las puedes encontrar en nuestras políticas de inclusión de paquetes (Inglés). Por favor, tilda todas las apropiadas. Si no estás seguro, te sugerimos que comiences un pre-envío.

  • recuperación de datos (data retrieval)
  • extracción de datos (data extraction)
  • munging de datos (data munging)
  • disposición o declaración de datos (data deposition)
  • automatización de flujos de trabajo (workflow automation)
  • control de versiones (version control)
  • manejo de citas y bibliometría (citation management and bibliometrics)
  • envoltorios de software científico (scientific software wrappers)
  • herramientas para trabajo de campo y reproducibilidad (field and lab reproducibility tools)
  • ligamientos con software de base de datos (database software bindings)
  • datos geoespaciales (geospatial data)
  • análisis de texto (text analysis)

• Explica cómo y por qué el paquete encaja dentro de estas categorías (1 a 3 oraciones):

Este trabajo es una adaptación de citesdb, también parte de rOpenSci, a un contexto diferente.

Para ayudar a hacer investigación reproducible, converti la informacion desde el DVD en formato REDATAM usando el Convertidor REDATAM creado por Pablo De Grande. La modificacion a estos archivos, que incluye geometrias detalladas, consistio en unir todos los archivos shp regionales en una unica tabla por nivel (e.g en lugar de proveer R01_mapa_comunas, ..., R15_mapa_comunas combine las 15 regiones en una unica tabla mapa_comunas).

• ¿Cuál es la audiencia esperada y las aplicaciones científicas de este paquete?

Economistas, sociologos y otros profesionales que trabajan con datos demograficos.

• ¿Hay otros paquetes de R que logren el mismo objetivo? Si los hay, ¿cómo se diferencian del tuyo, o alcanzan nuestro criterio del mejor de su categoría (documento en Inglés)?

No existen otros

• (Si aplica) ¿Tu paquete cumple con nuestras guías de Ética, Privacidad de Datos e Investigación de Sujetos Humanos (documento en Inglés)?

Si, de hecho no individualiza a personas, sino que el nivel detalle es a nivel de zonas (divisiones comunales). Existe una tabla personas, pero es para permitir obtener el % de adultos mayores por zona, etc.

• Si ya has hecho una consulta de pre-envío, por favor pega el enlace al issue correspondiente, una publicación del foro, u otra discusión. Alternativamente, etiqueta al editor (con @tag) con el que te contactaste.

#410

Revisiones Técnicas

Tilda los siguientes items para confirmar que los has completado:

• He leído la guía para autores (documento en Inglés) y la guía de paquetes de rOpenSci (documento en Inglés).

Este paquete:

• No viola los Términos de Servicio de ningún servicio con los que interactúa.
• Tiene una licencia aceptada por CRAN y OSI.
• Contiene un archivo README con instrucciones para instalar la versión de desarrollo.
• Incluye documentación con ejemplos para todas las funciones, creada con roxygen2.
• Contiene una viñeta (vignette) con ejemplos de sus funciones esenciales y su uso.
• Tiene una suite de tests (documento en Inglés).
• Tiene integración contínua (documento en Inglés), incluyendo reporte de cobertura de tests, usando servicios como Travis CI, Coveralls y/o CodeCov.

Opciones de Publicación

• ¿Tienes intenciones de subir este paquete a CRAN? ya esta en CRAN

• ¿Tienes intenciones de enviar este paquete a Bioconductor?

• ¿Deseas enviar un Artículo de Aplicaciones sobre tu paquete a Methods in Ecology and Evolution (documento en Inglés)? Si es así:

Opciones para MEE

• Este paquete es novedoso y será de interés para la mayoría de lectores de la revista.
• El manuscrito que describe el paquete no tiene más de 3000 palabras y está escrito en Inglés.
• Tienes intenciones de archivar el código del paquete en un repositorio a largo plazo, que cumple los requerimientos de la revista (mira las Políticas de Publicación de MEE (documento en Inglés))
• (Alcance: Considera los Objetivos y Alcance de MEE (documento en Inglés) para tu manuscrito. No otorgamos garatías de que tu manuscrito esté en el ámbito de MEE.)
• (Aunque no es requerido, recomendamos tener un manuscrito completamente preparado y en Inglés, al momento de enviar.)
• (Por favor, no envíes tu paquete de forma separada a Methods in Ecology and Evolution)

Código de Conducta

• Estoy de acuerdo en cumplir el Código de Conducta de rOpenSci (documento en Inglés) durante el proceso de revisión, y en mantener mi paquete si éste es aceptado.

[pachadotdev]

@melvidoni hola, muchas gracias por crear este template
lo que no encuentro del original es el topico "database access"

[melvidoni]

Editor checks:

• Documentation: The package has sufficient documentation available online (README, pkgdown docs) to allow for an assessment of functionality and scope without installing the package.
• Fit: The package meets criteria for fit and overlap
• Automated tests: Package has a testing suite and is tested via Travis-CI or another CI service.
• License: The package has a CRAN or OSI accepted license
• Repository: The repository link resolves correctly

---

Editor comments

Hola @pachamaltese, muchas gracias por pilotear las revisions en español con rOpenSci. Para empezar, estos son los resultados de goodpractice:

It is good practice to

  ✖ write unit tests for all functions, and all package code in general. 0% of code lines are covered by test cases.

    R/connect.R:3:NA
    R/connect.R:4:NA
    R/connect.R:5:NA
    R/connect.R:6:NA
    R/connect.R:8:NA
    ... and 178 more lines

  ✖ omit "Date" in DESCRIPTION. It is not required and it gets invalid quite often. A build date will be added to the package when you perform
    `R CMD build` on it.
  ✖ avoid long code lines, it is bad for readability. Also, many people prefer editor windows that are about 80 characters wide. Try make your
    lines shorter than 80 characters

    R\connect.R:14:1
    R\connect.R:27:1
    R\connect.R:60:1
    R\connect.R:78:1
    R\connect.R:131:1
    ... and 7 more lines

  ✖ avoid sapply(), it is not type safe. It might return a vector, or a list, depending on the input data. Consider using vapply() instead.

    R\zzz.R:11:11

¿Podrías trabajar en esto y avisarme cuando este listo? Por favor, dejá una respuesta detallada con los cambios.

Por el momento, iré contactando candidatos a revisores.

---

Reviewers: @FvD and @mpaulacaldas
Due date: 2021-2-25

[pachadotdev]

Listo!! Implementé casi todo lo que dice goodpractice, salvo lo del sapply() pues si implemento el cambio se rompe todo. Con respecto a los tests, las funciones que goodpractice que dice que no están cubiertas, es porque los tests existentes indirectamente llaman a esas funciones, por ejemplo https://github.com/pachamaltese/censo2017/blob/main/tests/testthat/test-02-tables.R#L12 llama a la conexion del panel de todos modos y usa connect.R internamente.

[melvidoni]

Hola @pachamaltese, muchas gracias por los cambios.

Los revisores son @anadiedrichs y @FvD. Por favor, recuerden usar la Guía para Revisores y las plantillas correspondientes para la revisión. Sabemos que las plantillas están en inglés, pero bueno... por favor, completen todo en español. Esta revisión es un piloto y aún no tenemos todo traducidor.

Fecha Límite para la revisión: Dada la temporada del año, y que es 2020, sumado a una revisión piloto... la fecha límite para la revisión va a ser el 14 de Enero de 2021.

[pachadotdev]

Hola @melvidoni
¿Existirá un potencial conflicto de interés? con ambos revisores hemos colaborado en múltiples instancias (R4DS, plumber, shiny, etc). Yo no lo veo así, creo que tendrán la confianza para enviar feedback de lo bueno y lo malo, pero siempre es bueno avisar.

[melvidoni]

Te conoce demasiada gente @pachamaltese! En fin, me parece genial que avises, por ahora también lo dejo a criterio de los revisores. Tenemos nuestra Guía de Conflictos de Interés. Si ellos no se consideran libres de COI, pido que me avisen, y volvemos al estadío en busca de revisores.

[FvD]

@melvidoni y @pachamaltese siguiendo la Guía de Conflictos de Interés yo me siento libre de algún conflicto de interés.

[melvidoni]

@melvidoni y @pachamaltese siguiendo la Guía de Conflictos de Interés yo me siento libre de algún conflicto de interés.

Excellente. Por favor, procede con la revisión entonces.