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

resources: add Eurolite LED PLL-480 CW/WW #1496

Merged
merged 1 commit into from
Dec 24, 2023

Conversation

bdrung
Copy link
Contributor

@bdrung bdrung commented Dec 23, 2023

Add Eurolite LED PLL-480 CW/WW [1]. The names and description were taken from 40001895-Manual-119588-1.000-de_en.pdf [2]. This fixture definition has some shortcomings:

  1. There is no type for floodlights, so ColorChanger is used.
  2. There are no separate presets for warm and cold white. So both warm and cold white have to use the same preset.
  3. The bulb colour temperature field does not allow specifying the color range 3200K - 5600K. So the lower warm white colour is specified.

[1] https://www.steinigke.de/en/mpn40001895-eurolite-led-pll-480-cw-ww-panel.html
[2] https://www.steinigke.de/download/40001895-Manual-119588-1.000-eurolite-led-pll-480-cw-ww-panel-de_en.pdf

Add Eurolite LED PLL-480 CW/WW [1]. The names and description were taken
from 40001895-Manual-119588-1.000-de_en.pdf [2]. This fixture definition
has some shortcomings:

1. There is no type for floodlights, so ColorChanger is used.
2. There are no separate presets for warm and cold white. So both warm
   and cold white have to use the same preset.
3. The bulb colour temperature field does not allow specifying the color
   range 3200K - 5600K. So the lower warm white colour is specified.

[1] https://www.steinigke.de/en/mpn40001895-eurolite-led-pll-480-cw-ww-panel.html
[2] https://www.steinigke.de/download/40001895-Manual-119588-1.000-eurolite-led-pll-480-cw-ww-panel-de_en.pdf

Signed-off-by: Benjamin Drung <bdrung@posteo.de>
@bdrung
Copy link
Contributor Author

bdrung commented Dec 24, 2023

Thanks. Addressed the comments. Where should I report those three points mentioned in commit message as feature request?

@mcallegari
Copy link
Owner

You can report here:
https://www.qlcplus.org/forum/viewforum.php?f=18

@mcallegari mcallegari merged commit 518628c into mcallegari:master Dec 24, 2023
6 checks passed
@mcallegari
Copy link
Owner

Thanks 👍

@coveralls
Copy link

Coverage Status

coverage: 32.067%. remained the same
when pulling 1022f93 on bdrung:Eurolite-LED-PLL-480
into 4556d3c on mcallegari:master.

@bdrung bdrung deleted the Eurolite-LED-PLL-480 branch February 7, 2024 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants