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

Update gauge data #192

Closed
lexvand opened this issue Jan 18, 2019 · 3 comments
Closed

Update gauge data #192

lexvand opened this issue Jan 18, 2019 · 3 comments

Comments

@lexvand
Copy link

lexvand commented Jan 18, 2019

Offerte: Reduce update time for gauge data: Update both the dashboard and the iFrame more
frequently with real time gauge data, preferred 15mins.

Tijdinschatting:
40 uur

Proposed solution:
De rasters updaten nu niet, of ze updaten veel te vaak.
We missen controle over deze feature.
Het lijkt of ergens impliciet een update vandaan komt.
We moeten deze impliciete manier opsporen en verwijderen.
Daarna een expliciete update toevoegen.

Nice to have Configuratie:
We willen per tile vastleggen hoevaak deze wordt geupdate en ook nog voor gridlayout versus fulllayout.

#127

Todo:

  • Uitzoeken hoe we we voorkomen dat plotly bij iedere refresh terugschiet naar oorspronkelijke zoomlevel
    --> Done by How can I preserve the view state of the plot? plotly/react-plotly.js#90 and saving the xaxis in state from plotly's figure.layout.xaxis
  • Uitzoeken of en waarom timeseries niet opnieuw worden aangeroepen
    --> Done dit gebeurd nu wel.
    Reden was dat timeseries in component did mount werden aangeroepen. Verbetering is dus om ze ook in componentdidupdate op te halen. Dit heb ik ook voor alarmen gedaan.
    Beide fetches voor alarmen en timeseries moeten wel achter een conditie dat de current time is verandert.
    Alarmen geven verder de huidige tijd niet mee in de api call. Alarmen staan namelijk statefull in de database. Zij gaan uit van de huidige tijd en hebben een status of ze zijn gebreached.
  • Tijd configureren in redux store
    WIP
  • Uitzoeken hoe spinners niet altijd te tonen bij refresh
  • Uitzoeken hoe de refreshrate te configureren en hoe fijnkorrelig. Willen we dit voor het hele dashboard of per tile?
    Eigelijk willen we per tile, maar wellicht is dit voor Parramatta niet nodig.

[PROJ-995] created by remco.gerlich

@lexvand
Copy link
Author

lexvand commented Mar 6, 2019

Reviewed and merged to master by Tom

#129

by 5b3b278169812b2ef3f78d91

@lexvand
Copy link
Author

lexvand commented Mar 6, 2019

Lisanne Wapstra

Wolf Schouten

Hi Lisanne en Wolf,

Deze ticket zorgt voor het verversen van de data op de tile dashboards.

Dit is lasstig functioneel te testen, maar je kan bijvoorbeeld zien dat:

  • in de timeseries de “now” lijn iedere minuut verschuift
  • De rasterdata opnieuw wordt opgehaald
  • De timeseries data opnieuw wordt opgehaald
  • De alarmen data opnieuw wordt opgehaald
  • Dit zou ook in de network tab van de browser devtools zichtbaar moeten zijn

Ook deze willen we graag op 14 maart al naar productie.

Indien mogelijk graag zo snel mogelijk testen

Alvast bedankt !

groet Tom

by 5b3b278169812b2ef3f78d91

@lexvand
Copy link
Author

lexvand commented Mar 8, 2019

Met de developer tools van de browser ernaast getest.

Tiles op het dashboard en de data in een open tile worden elke minuut ververst.

by wolf.schouten

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

No branches or pull requests

1 participant