Feature request: permit location of 0/negative_infinity() in neg_binomial_2(_log)_lpmf

Abierto
#3,302 5 comentarios 1 reacción 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
55/100
Tipo de issue
Nueva funcionalidad
Claridad
Bastante claro
Estado de actividad
Tranquilo
Stack tecnológico
cpp
Área
tooling

Línea de trabajo

Comience con los puntos de entrada neg_binomial_2_lpmf y neg_binomial_2_log_lpmf, y compare su validación de ubicación con poisson_lpmf y poisson_log_lpmf. Se considera terminado cuando las funciones binomial negativa acepten los casos solicitados de zero y negative_infinity() de forma coherente con las funciones de Poisson, y el comportamiento relevante esté cubierto por pruebas.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

distributions good first issue numerics

Good day,

It is currently possible to have location parameter of 0 in poisson_lpmf and negative_infinity() in poisson_log_lpmf; I've also verified this works with the _glm functions:

model {
  real works = poisson_lpmf(1 | 0),
       works2 = poisson_log_lpmf(1 | negative_infinity());
}

However, this is not possible with neg_binomial_2 functions:

model {
  real fails = neg_binomial_2_lpmf(1 | 0, 1),
       fails2 = neg_binomial_2_log_lpmf(1 | negative_infinity(), 1);
}

which errors out with:

Exception: neg_binomial_2_lpmf: Location parameter is 0, but must be positive finite!

I think it would be reasonable to permit similar behaviour as for the Poisson functions?

Lenguaje dominante
C++
Estrellas
845
Forks
225
Merge medio
3 d 9 h
PR fusionados (30 d)
12

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de stan-dev/math

Todos los issues de stan-dev/math

Issues similares

Más issues de C++

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.