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

Add proof of concept support for HSL lighting #1

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Minims
Copy link
Owner

@Minims Minims commented Aug 6, 2023

For testing & review

Add ack and unack set_hsl functions
Add mode state (ctl or hsl)
Add correct return of state to HA
@Minims
Copy link
Owner Author

Minims commented Aug 11, 2023

@louisjennings I don't know if dominikberse will answer, i have tried to make some PR some month ago without any answer.
If you want we can use my fork to centralized all the work on BLE Mesh.

How are you running the code ? A Docker on a PI 4 ?

@@ -28,8 +28,8 @@ RUN sh ./install-bluez.sh

# install bridge
WORKDIR /opt/hass-ble-mesh
RUN git clone https://github.com/minims/homeassistant-bluetooth-mesh.git .
RUN git checkout master
RUN git clone https://github.com/louisjennings/homeassistant-bluetooth-mesh.git .
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
RUN git clone https://github.com/louisjennings/homeassistant-bluetooth-mesh.git .
RUN git clone https://github.com/minims/homeassistant-bluetooth-mesh.git .

After merge it will use minims/master

RUN git clone https://github.com/minims/homeassistant-bluetooth-mesh.git .
RUN git checkout master
RUN git clone https://github.com/louisjennings/homeassistant-bluetooth-mesh.git .
RUN git checkout light-hsl
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
RUN git checkout light-hsl
RUN git checkout master

@@ -1,7 +1,7 @@
asyncio-mqtt==0.12.1
bitstring==3.1.9
black==22.10.0
git+https://github.com/minims/python-bluetooth-mesh@43f3b368add0e0a40782478369a30454b95a9034
git+https://github.com/louisjennings/python-bluetooth-mesh@dc4132274b0883c9296eab9dd0998a345280e051
Copy link
Owner Author

@Minims Minims Aug 11, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
git+https://github.com/louisjennings/python-bluetooth-mesh@dc4132274b0883c9296eab9dd0998a345280e051
git+https://github.com/minims/python-bluetooth-mesh@d813d4860025bf28a7338681229e728feeeed851

I have rebase my fork to latest version of the official repo

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The HSL functionality depends upon changes I made in the required commit. I'll eventually try getting the HSL models included in the official repo. Until then, I can create a pull request from the feature branch to your repo.

@louisjennings
Copy link

Yes, I agree that it would be good to have a centralised repository for all the changes. If the original is inactive, then your repo would be the next best (my fork is a fork of your repo).

I have two instances (dev and production) both are run as docker containers but the production instance's docker host is running inside a VM with a BT adapter passed through.

@Minims
Copy link
Owner Author

Minims commented Aug 12, 2023

Ok so we can start by a PR in https://github.com/Minims/python-bluetooth-mesh
Because i have some update too on python-bluetooth-mesh

@Minims
Copy link
Owner Author

Minims commented Aug 12, 2023

An other thing, have you check the branch called imprive-stability ? I wonder if we need to integrate this in the master branch. https://github.com/dominikberse/homeassistant-bluetooth-mesh/tree/feature/improve-stability

@Minims
Copy link
Owner Author

Minims commented Aug 15, 2023

@louisjennings so python-bluetooth-mesh is up to date and rebase on SilvairGit/master

latest commit to use in requirements is : d813d4860025bf28a7338681229e728feeeed851

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.

2 participants