Add Axion Lighting integration

[Vrncanac]

Proposed change

This PR introduces a new integration for controlling Axion Lights via the Home Assistant UI. The integration allows users to connect their Axion DMX controllers with Home Assistant, offering support for various light types, including White, Tunable White, RGB, RGBW, and RGBWW. It provides a seamless way to manage lighting through Home Assistant, enhancing the automation capabilities for users with Axion Lighting systems.

Type of change

• [ ] Dependency upgrade
• [ ] Bugfix (non-breaking change which fixes an issue)
• [x] New integration (thank you!)
• [ ] New feature (which adds functionality to an existing integration)
• [ ] Deprecation (breaking change to happen in the future)
• [ ] Breaking change (fix/feature causing existing functionality to break)
• [ ] Code quality improvements to existing code or addition of tests

Additional information

• This PR fixes or closes issue: fixes #
• This PR is related to issue:
• Link to documentation pull request: https://github.com/home-assistant/home-assistant.io/pull/34532

Checklist

• [x] The code change is tested and works locally.
• [x] Local tests pass. Your PR cannot be merged unless tests pass
• [x] There is no commented out code in this PR.
• [x] I have followed the development checklist
• [x] I have followed the perfect PR recommendations
• [x] The code has been formatted using Ruff (ruff format homeassistant tests)
• [x] Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

• [x] Documentation added/updated for www.home-assistant.io

If the code communicates with devices, web services, or third-party tools:

• [x] The manifest file has all fields filled out correctly.
  Updated and included derived files by running: python3 -m script.hassfest.
• [ ] New or updated dependencies have been added to requirements_all.txt.
  Updated by running python3 -m script.gen_requirements_all.
• [ ] For the updated dependencies - a link to the changelog, or at minimum a diff between library versions is added to the PR description.

To help with the load of incoming pull requests:

• [ ] I have reviewed two other open pull requests in this repository.

[home-assistant[bot]]

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks :+1:

Learn more about our pull request process.

[joostlek]

Also, if you think you did the right thing in the comment, feel free to resolve it. If you are unsure or if it was a question, or you have a question back, please leave it open

This one didn't got an answer https://github.com/home-assistant/core/pull/125039#discussion_r1758711009

[Vrncanac]

Hi joostlek!

I've been thinking about this comment that you left, and there are few flaws with it.

What if we create a subclass for every light type? So we now have the AxionDMXLight, and then you could have the AxionDMXRGBLight which contains only the RGB logic for turning on and off (we can have the shared logic of the turning off and on in the base class, but the specifics in the sub class).

But this idea is with the assumption that you can't do RGB with a RGBW or RGBWW light.

You can do RGB with an RGBW or RGBWW light. Since each channel is controllable you could ignore the white on the RGBW and just do RGB if you wanted.

I see no gain in breaking each light into a sub-class, this will increase the complexity in terms of the number of classes (of course on the other hand it will reduce the complexity within each class). Also, since I tested this code on real hardware, and it works just as expected I think we're okay to leave it as-is.

[Vrncanac]

Hi @joostlek, just wanted to check if there’s anything else needed from my side to help move this PR forward. Looking forward to your feedback. Thanks!

[joostlek]

I did see your mail yesterday, it ended up in the spam. But last week was beta week so we were busy, I'll try to take a look at it again soon

[Vrncanac]

Hi @joostlek, just wanted to follow up and see if there’s any chance you’ll be able to review the PR soon? I understand things can get busy, but it would be great to get some movement on this PR. Is there anything I can do to help move things along? Thanks!

[ChrisAllenIsAwesome]

Hello @joostlek and @Vrncanac , we have a couple customers who have installed Home Assistant in their house and were looking to integrate these lights/system. May I ask when you think it would be available for them to download?

Thanks so much!

[Vrncanac]

Hello @joostlek and @autinerd!

Could we get some movement on this PR? You can pass it down to someone else on your team, if you guys are busy? As @ChrisAllenIsAwesome mentioned, we have some customers that are really excited about getting the Axion Lighting into the Home Assistant project, so that they can use the best from both worlds.

Thanks in advanced!

[ChrisAllenIsAwesome]

Sorry to be a pain on this, I have another person asking me about this integration. I have downloaded it and manually tried it with my system and it seems to be working properly, is there a bug or some other change needed before it can be published?

[frenck]

Converted back to draft, left a review comment above.

[Vrncanac]

Hi @frenck, @joostlek and @NoRi2909, hope you guys are all good! I was hoping to see if we can get some traction on this PR? We have some of our customers waiting for this integration in order to connect our DMX controllers with your platform.

[github-actions[bot]]

There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. If you are the author of this PR, please leave a comment if you want to keep it open. Also, please rebase your PR onto the latest dev branch to ensure that it's up to date with the latest changes. Thank you for your contribution!

[emontnemery]

@Vrncanac what's the status of this PR? If you consider the PR ready for review, click the "Ready for review" button.

[Vrncanac]

@emontnemery let me just push some of the latest changes, and I'll mark it as ready for review! Thanks for checking thou!

[github-actions[bot]]

There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. If you are the author of this PR, please leave a comment if you want to keep it open. Also, please rebase your PR onto the latest dev branch to ensure that it's up to date with the latest changes. Thank you for your contribution!