Allow structured format for unconstrained parameters
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 35/100
Línea de trabajo
Comienza localizando el punto de entrada actual de la API para vectores de parámetros no restringidos y el manejo correspondiente de los parámetros restringidos. Revisa cómo se determinan el orden y las dimensiones de los parámetros y, a continuación, define pruebas para entradas no restringidas con nombre, incluidos valores scalar y simplex, y para nombres o longitudes no válidos. Se considera terminado cuando las entradas con nombre se validan y se asignan de forma determinista sin afectar a las entradas vectoriales.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The current approach to providing unconstrained parameters as arguments (as inherited from rstan) requires that they are passed as a single numeric vector with all unconstrained parameter values concatenated.
After some API discussions with @n-kall, it was pointed out that this approach can easily lead to errors if any transformations or custom constructions of unconstrained parameters are performed - as users might accidentally provide parameter values in the wrong order.
Instead, we could allow unconstrained parameters to be passed as a named list like we do for constrained parameters, but with handling/checking for the different number of unconstrained values (e.g., for simplexes).
As a concrete example, for a model with parameters:
parameters {
real x;
simplex[4] y;
}
The unconstrained parameters would need to be passed as a four-element numeric vector:
unconstrained_vector <- c(1.2, 0.6, 1.6, -1.2)
This clearly allows for the easy mistake of accidentally providing the value of x as the last, rather than the first element (especially if the model is being iterated/revised).
I'm proposing we could accept the unconstrained parameters in the format:
unconstrained_list <- list(x = 1.2, y = c(0.6, 1.6, -1.2))
And then handle the ordering internally when mapping to a single vector of values.
Thoughts?
- Lenguaje dominante
- R
- Estrellas
- 160
- Forks
- 69
- Merge medio
- 1 d 8 h
- PR fusionados (30 d)
- 20
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
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 stan-dev/cmdstanr
-
Compiled model methods and `expose_functions` fail to load when RcppParallel's TBB is already in the sessionPosiblemente ocupada @jgabry la tomó hace 3 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
stan-dev/cmdstanr#1270 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
stan-dev/cmdstanr#506 · 6 comentarios · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
feature
Dificultad 5/5 Más de una semana Aptitud para principiantes 42/100
stan-dev/cmdstanr#1276 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
`cmdstan_version_compare()` conflates no version with old versionPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día
Todos los issues de stan-dev/cmdstanr
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 95/100
tidymodels/textrecipes#332 ·