facebook/prophet

Clarify Valid Unit Types if Unit Type is Unrecognized

Open

#1 989 ouverte le 8 août 2021

Voir sur GitHub
 (2 commentaires) (2 réactions) (0 assignés)Python (17 686 stars) (4 451 forks)batch import
enhancementgood first issue

Description

I'm trying to use prophet cross_validation but I received a value error for unrecognized type when I try:

df_cv = cross_validation(m, initial='120 months', period='12 months', horizon = '12 months')

I think the error should return a list of valid types so that I don't have to go look up why months aren't recognized as a valid unit.

Guide contributeur

Stack technique
python
Domaine
developer experience
Type d'issue
feature
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.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
PythonProphet library
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80
Direction de recherche
Locate the cross validation function in the Prophet codebase, likely in prophet/diagnostics.py. Find where unit types are validated and the error is raised. Modify the error message to include a list of valid unit types (e.g., 's', 'sec', 'min', 'hour', 'day', etc.). Ensure consistency with other similar validation error messages in the code.