Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problema de segmentación de datos por step en Urbo #64

Open
ignaciodomingo opened this issue Jan 25, 2018 · 0 comments
Open

Problema de segmentación de datos por step en Urbo #64

ignaciodomingo opened this issue Jan 25, 2018 · 0 comments
Labels

Comments

@ignaciodomingo
Copy link
Contributor

Se ha detectado el siguiente problema en Urbo, que afecta de forma general a todos los widgets que segmenten la información en periodos (1h, 4h, 1d), también conocido como step.
Actualmente, si se seleccionan las fechas de 7 días de información:
image.png
Se pinta información sólo de 6 días:
image.png

A pesar de que en llamada a servicio se piden 7 días:
POST https://urbo-aquagis-backend.geographica.gs/api/aljarafe/variables/timeserie

{  
   "agg":[  
      "SUM"
   ],
   "vars":[  
      "aq_cons.sector.forecast"
   ],
   "csv":false,
   "time":{  
      "start":"2018-01-28T23:00:00Z",
      "finish":"2018-02-04T22:59:59Z",
      "step":"1d"
   },
   "filters":{  
      "condition":{  

      },
      "group":"aq_cons.sector.usage"
   }
}

El problema radica en que no se pasan un periodo de 7 días completos, sino 6 días 23h 59m 59s. Esto, postgresql nos obtiene 6 datos, y no 7, al descartar el último por no ser completo. Esto pasa también con otros steps menores (dado que nunca damos el step completo).

Esto se ha discutido con @cayetanobv @josemazo @Josmorsot y @padawannn , llegando a la conclusión que es un problema general a Urbo y que hay que darle solución. Una posible solución es modificar la función que prepara las fechas para enviarlas a la API y hacer que finish siempre sume 1 segundo, de forma que las peticiones vayan desde el día 1 a las 00:00 al día n+1 a las 00:00 (siendo n la última fecha de la que se quieren datos).

En cualquier caso, la solución pasa por hacer modificaciones que tendrán que llegar a todo Urbo (aunque inicialmente se ha propuesto ir haciendo widgets que hereden de los existentes y adaptarles la función que prepara las fechas, e ir migrando poco a poco)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant