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

Alarms not properly moving sliders in Version 1.3.1 #213

Closed
infinigrove opened this issue May 25, 2018 · 1 comment
Closed

Alarms not properly moving sliders in Version 1.3.1 #213

infinigrove opened this issue May 25, 2018 · 1 comment
Labels
Milestone

Comments

@infinigrove
Copy link

infinigrove commented May 25, 2018

Expected Behavior

1 second after Start move Slider Timer to 4
2 seconds after Start move Slider Fan to 2
3 seconds after Start move Slider Temp to 200

Actual Behavior

On Start Slider Fan is moved to 2
1 second after Start Slider Temp is moved to 100 (there is no 100)
2 seconds after Start Slider Timer is moved to 4

Steps to Reproduce the Problem

  1. Load settings and alarms, Click "ON", Click "START"
  2. See attached file for settings, alarms, alog and screenshot
  3. I'm attempting to improve https://github.com/infinigrove/SR700-Artisan-PDServer now that sliders are no longer locked into a range of 0-100.

Thank you for any help!

Specifications

  • Artisan Version: Artisan 1.3.1 (1)
  • Artisan Build (number in brackets shown in the about box): (1)
  • Platform (Mac/Windows/Linux + OS version): Linux
  • Connected devices or roasting machine: SR700

Please attach your current Artisan settings file (as exported via menu Help >> Save Setings as *.aset) file.
Please attach any relevant Artisan *.alog profiles.

Note that you need either add a .txt extension or zip the files before uploading. Otherwise you will receive a "Not a supported file type" error on uploading.
alarm-debug.zip

MAKOMO added a commit that referenced this issue May 31, 2018
…ing Issue #213)

- use zero-based port numbering in Phidgets tab
@MAKOMO MAKOMO added this to the v1.4 milestone May 31, 2018
@MAKOMO
Copy link
Member

MAKOMO commented May 31, 2018

Alarm actions triggering slider changes were still limited to the range 0-100. My commit just fixed that. The strange order of alarm processing in this case is caused by Artisan processing alarms only once per sampling interval, which seems to be 2sec here, firing only one of the "enabled" alarm in no specific order. This "lazy" behaviour might not change in the near future.

@MAKOMO MAKOMO closed this as completed Jul 7, 2018
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

3 participants