rate_values default time

Abierto
  • ¿Algo no está bien?
#243 2 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
45/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
python
Área
api

Línea de trabajo

Comienza en ratings.py, en rate_values, y compara el comportamiento documentado cuando se omite times con el comportamiento cuando se proporciona un tiempo explícito. Reproduce el caso de la valoración más reciente usando ambas llamadas y, a continuación, verifica que la implementación y la documentación coincidan sobre el tiempo utilizado para seleccionar las valoraciones.

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

Descripción

As shown below, the documentation in ratings.py states that if times are not specified, the current time is used. However, in my experience, the latest rating curve is not being used when I leave the time out. I had to specify a time to get it to use the latest rating.

Not a big deal to add the time (probably best to always do so anyway). I just wanted to make note of this.

Parameters
----------
rating_id: string
    The rating-id (rating specification) of the the rating set to use
office_id: string
    The owning office of the rating set.
units: string
    The unit of each independent parameter separated by commas concatenated with a semicolon followed by the
    desired dependent_parameter unit (e.g., 'ft;ac-ft' or 'unit,%,ft;cfs')
values: list[list[float]]
    The independent parameter values to rate (one list for each independent parameter, in parameter order).
    For multiple independent parameters, each list must be of the same length.
times: list[integer] Optional Default = None
    A list of times (in milliseconds since 1970-01-01T00:00:00Z) for the independent parameter values.
    If None, the current time is used.
    If specified but the constains fewer times than the number of independent parameter values, the last
    time is used for all independent parameter values from that position to the end of the list.
rating_time: integer Optional Default = None
    A specific date/time to use as the "current time" of the rating. No ratings with a create date later than
    this will be used. Useful for performing historical ratings. If not specified or NULL, the current time is used.
round: boolean Optional Default = False
    Whether to round the rated values according to the rounding spec contained in the rating specification in the database
Lenguaje dominante
Python
Estrellas
16
Forks
11
Merge medio
7 h 35 min
PR fusionados (30 d)
3

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 HydrologicEngineeringCenter/cwms-python

Todos los issues de HydrologicEngineeringCenter/cwms-python

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.