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

oh-knob-card #1003

Closed
BiloxiGeek opened this issue Apr 17, 2021 · 2 comments · Fixed by #1012
Closed

oh-knob-card #1003

BiloxiGeek opened this issue Apr 17, 2021 · 2 comments · Fixed by #1012
Labels
bug Something isn't working main ui Main UI

Comments

@BiloxiGeek
Copy link

The problem

Knob card sends multiple commands when trying to set a new thermostat setpoint.

I configured an oh-knob-card to control Thermostat_Setpointcooling item on a CT100 thermostat. When changing the setpoint in the UI I see multiple commands being sent in the events.log and the setpoint ends up only changing by one degree.

Expected behavior

I expected to see a single command sent after the knob was released and the command would be wherever the knob was set at when released.

I was able to run a send command in the console to change the setpoint by multiple degrees and it worked as expected, setpoint immediately changed to the new value.

Your environment

Some discussion on the community: https://community.openhab.org/t/knob-only-incrementing-by-1/120924

OS: CentOS 8

runtimeInfo:
  version: 3.1.0.M3
  buildString: Milestone Build
locale: en-US
systemInfo:
  configFolder: /etc/openhab
  userdataFolder: /var/lib/openhab
  logFolder: /var/log/openhab
  javaVersion: 11.0.10
  javaVendor: Azul Systems, Inc.
  javaVendorVersion: Zulu11.45+27-CA
  osName: Linux
  osVersion: 4.18.0-301.1.el8.x86_64
  osArchitecture: amd64
  availableProcessors: 4
  freeMemory: 50923512
  totalMemory: 197132288
bindings:
  - astro
  - hue
  - icloud
  - zoneminder
  - zwave
clientInfo:
  device:
    ios: false
    android: false
    androidChrome: false
    desktop: true
    iphone: false
    ipod: false
    ipad: false
    edge: false
    ie: false
    firefox: true
    macos: false
    windows: false
    cordova: false
    phonegap: false
    electron: false
    nwjs: false
    webView: false
    webview: false
    standalone: false
    pixelRatio: 0.8955223880597015
    prefersColorScheme: light
  isSecureContext: false
  locationbarVisible: true
  menubarVisible: true
  navigator:
    cookieEnabled: true
    deviceMemory: N/A
    hardwareConcurrency: 8
    language: en-US
    languages:
      - en-US
      - en
    onLine: true
    platform: Linux x86_64
  screen:
    width: 2144
    height: 1206
    colorDepth: 24
  support:
    touch: false
    pointerEvents: true
    observer: true
    passiveListener: true
    gestures: false
    intersectionObserver: true
  themeOptions:
    dark: light
    filled: true
    pageTransitionAnimation: default
    bars: filled
    homeNavbar: default
    homeBackground: default
    expandableCardAnimation: default
  userAgent: Mozilla/5.0 (X11; Fedora; Linux x86_64; rv:87.0) Gecko/20100101 Firefox/87.0
timestamp: 2021-04-17T18:55:25.480Z
@BiloxiGeek BiloxiGeek added bug Something isn't working main ui Main UI labels Apr 17, 2021
@BiloxiGeek
Copy link
Author

Additional: I've seen this behavior using safari on iOS and firefox on Linux. It doesn't appear to be browser related.

@Rossko57
Copy link

Rossko57 commented Apr 18, 2021

See
https://community.openhab.org/t/knob-only-incrementing-by-1/120924

Sending multiple commands tracking movement isn't a bug, though a useful enhancement could be made to optionally manage that in more detail.

hubsif added a commit to hubsif/openhab-webui that referenced this issue Apr 20, 2021
hubsif added a commit to hubsif/openhab-webui that referenced this issue May 16, 2021
Fix oh-slider issue

Signed-off-by: Hubert Nusser <[email protected]>
ghys pushed a commit that referenced this issue May 16, 2021
Fix oh-slider issue

Signed-off-by: Hubert Nusser <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working main ui Main UI
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants