stan-dev/stan

deprecate int_step

Ouverte

#2 433 ouverte le 9 nov. 2017

Voir sur GitHubGuide contributeur
 (5 commentaires) (0 réaction) (0 personne assignée)C++ (387 forks)batch import
bugdocumentationgood first issue

Métriques du dépôt

Stars
 (2 761 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Summary:

int_step(x) and step(x) return different values for 0.

The version of step(x) matches the BUGS definition and the standard definition of the Heaviside step function. Therefore, it should not be changed.

Changing int_step()'s behavior would be confusing, so instead it should be deprecated and replaced with a suggestion to replace existing uses of int_step(n) with the boolean expression (n >= 0), which evaluates to 1 if n >= 0 and 0 otherwise.

Current Version:

v2.17.0

Guide contributeur

Direction de recherche
Localisez la fonction int step dans le code source de Stan et ajoutez un avertissement de dépréciation. Mettez à jour la documentation pour suggérer l'utilisation de (n >= 0) à la place. Assurez la cohérence avec la définition de step.
Stack technique
r
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
StanR
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70