luci-app-statistics: sensors - missing graphs
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 7.9k
- Forks
- 2.9k
- Avg merge
- 4d 17h
- Merged PRs (30d)
- 28
Description
@jow-
You did a great job adding sensors for luci https://github.com/openwrt/luci/commit/cff2b99b4f9af0957ff6de4fe1ce48d30337b41b but one small thing is missing. You scan for all sensors and gather all data. But there is no definition for graphs - only for temperature. Maybe you can definitions for Volatge and fan speed graphs.
applications/luci-app-statistics/luasrc/statistics/rrdtool/definitions/sensors.lua
Thanks in advance
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with applications/luci-app-statistics/luasrc/statistics/rrdtool/definitions/sensors.lua and compare the existing temperature graph definition with the sensor data gathered there. Add graph definitions for voltage and fan speed, then verify that the corresponding sensor graphs appear in luci-app-statistics.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- lua
- Domain
- data-visualization
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100