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

BasicUI: Images and Charts not updating according to Configuration #3308

Closed
Auto-Mark opened this issue Apr 25, 2017 · 10 comments

Comments

@Auto-Mark
Copy link

commented Apr 25, 2017

There's been a discussion in the community about images and charts not updating as expected:
https://community.openhab.org/t/basic-ui-image-refresh-rate-in-seconds-instead-miliseconds/27152

Some tests suggest that the refresh parameter is interpreted differently:

  • ClassicUI: milli sec
  • BasicUI: sec

Fix: Use same time base for all UIs

Also, in my own setup the update works only on images not on charts.
The chart initially loads in BasicUI but then instead of updating it shows a broken image symbol.

@kaikreuzer

This comment has been minimized.

Copy link
Member

commented Apr 25, 2017

It should be a value in milli secs, see here.

So I would say that it is a bug in the Basic UI. @resetnow, Would you agree?

@resetnow

This comment has been minimized.

Copy link
Contributor

commented Apr 25, 2017

Yes, it's a bug, value is multiplied by 1000 in BasicUI. Obvious fix but I think we need to set a lower limit (50 ms?) to avoid accidental request spam.

@kaikreuzer

This comment has been minimized.

Copy link
Member

commented Apr 25, 2017

Yes, a lower limit makes sense. 50ms sounds sensible to me, maybe even 100ms.

@Auto-Mark

This comment has been minimized.

Copy link
Author

commented Apr 28, 2017

Using basicUI CalDAV also is not updating. Maybe this is related to the issue described here:
openhab/openhab1-addons#5170
And maybe the weather binding not updating is also connected to this:
openhab/openhab-distro#447

lolodomo added a commit to lolodomo/smarthome that referenced this issue May 13, 2017
Basic UI: fix refresh interval for image and chart
Fix eclipse#3308

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
@lolodomo

This comment has been minimized.

Copy link
Contributor

commented May 13, 2017

I proposed a fix, setting the minimum value to 100 ms.

kaikreuzer added a commit that referenced this issue May 14, 2017
Basic UI: fix refresh interval for image and chart (#3444)
Fix #3308

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
@Auto-Mark

This comment has been minimized.

Copy link
Author

commented May 15, 2017

According to a user the behaviour hasn't changed:
community.openhab.org

@lolodomo

This comment has been minimized.

Copy link
Contributor

commented May 15, 2017

Normal, it is not yet included in OH snapshots. You will have to wait few days, the time for kaikreuzer to include a new ESH version in OH. The last is from the 12th of May.

@kaikreuzer

This comment has been minimized.

Copy link
Member

commented May 15, 2017

You can always check here, which changes are already included in openHAB: https://hudson.eclipse.org/smarthome/job/SmartHomeDistribution-Stable/changes

@Auto-Mark

This comment has been minimized.

Copy link
Author

commented May 16, 2017

Thanks for this clarification. I will do so in the future.

@lolodomo

This comment has been minimized.

Copy link
Contributor

commented Jun 2, 2017

Please note that refresh intervals look like inappropriate for charts in the demo sitemap:

				Switch item=Weather_Chart_Period label="Chart Period" icon="chart" mappings=[0="Hour", 1="Day", 2="Week"]
				Chart item=Weather_Chart period=h refresh=600 visibility=[Weather_Chart_Period==0, Weather_Chart_Period=="NULL"]
				Chart item=Weather_Chart period=D refresh=3600 visibility=[Weather_Chart_Period==1]
				Chart item=Weather_Chart period=W refresh=3600 visibility=[Weather_Chart_Period==2]

I am not sure but my first feeling is that these values are in seconds rather than in milliseconds ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.