-
Notifications
You must be signed in to change notification settings - Fork 1
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
Model the alarm limits better #55
Comments
I know we discussed this issue a bit in the last status meeting. Someone (was it @beauremus ?) asked if this should be handled by DPM. Seems to me that's the best spot for it. Thoughts? |
If the param page switches between NOM/TOL and MIN/MAX, does it resend the request to DPM? The modeling I proposed above would do the switch in the app and DPM would simply provide the data returned by the device. I guess the question is: is it useful for DPM to provide the switch? I can't think of scripts or apps (besides param page) that actually care about the alarm limits. They may care that a device is in alarm, but not so much the limits. The param page cares because users may want to better understand the limits by switching between views. But, then, I've never written an in-depth Python or ACL script. Maybe they do commonly access the limits and like to switch between the types. |
Actually, what I've implemented so far displays all four values. I figured that the existing parameter page is asking you to toggle between the two because of limited screen space. But we can easily pack four values onto the page. And so, there's no switch to be done really. What's stored in the database - nom and tolerance? And so min/max are calculated? |
Something like:
|
I'm going to tuck this thought at the end of this issue: A parameter row could simply show the most general, useful stuff (name, description, setting, reading, status) and a details icon. Clicking the icon could animate the row to expand vertically to show details like basic status bits and alarm limits. |
Originally posted by @rneswold in #54 (comment)
The text was updated successfully, but these errors were encountered: