Add brightness range to display model

[joostlek]

Add brightness range to display model, because when the device is connected to a computer, it won't be able to be put on full brightness. Hence we need to know what the min and max are

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.68%. Comparing base (53134cb) to head (99f8c76).
Report is 137 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #788      +/-   ##
==========================================
+ Coverage   93.73%   94.68%   +0.95%     
==========================================
  Files           6        6              
  Lines         383      414      +31     
  Branches       35       24      -11     
==========================================
+ Hits          359      392      +33     
+ Misses         22       17       -5     
- Partials        2        5       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[frenck]

Should we leave in brightness range as well?

[joostlek]

But I'm not sure what that would represent

[frenck]

That would represent the min/max (represented as 0-100% in Home Assistant), while the limit limits what can be set at that point in time?

[joostlek]

The limits are already represented by the limits. I am not sure why they return 2 ranges, but the limit seems to be smaller

[frenck]

I am not following? What is your goal? To redefine the range of the slider in Home Assistant?

If that is the case, I think that would be incorrect.

Looking at the newer LaMetric model, it will still show the full audio volume range on display, but doesn't allow one to go beyond a certain point.

To mimic that behavior, you'd need both?

[joostlek]

Let's have both and have this discussion in the home assistant repo

[frenck]

Thanks, @joostlek 👍

../Frenck