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 support for Linptech Submersion Sensor, Pressure Sensor and gdds.light.wy0a01 #1176

Merged
merged 3 commits into from
Oct 25, 2023

Conversation

justbin95
Copy link
Contributor

@justbin95 justbin95 changed the title Add support for Linptech Submersion Sensor and Pressure Sensor Add support for Linptech Submersion Sensor, Pressure Sensor and gdds.light.wy0a01 Oct 24, 2023
@AlexxIT AlexxIT merged commit b9f17cb into AlexxIT:master Oct 25, 2023
1 of 2 checks passed
@AlexxIT
Copy link
Owner

AlexxIT commented Oct 25, 2023

Thanks!

@ABUAIJUTIE
Copy link
Contributor

This converter it's not working. Still get unknown after adding it.
I make some modifications like below and move it from the end of device.py (MESH DEVICES part) to BLE part (right below 16143: ["Linptech", "Submersion Sensor", "RS1BB"] ). And it works.

{
    16204: ["Linptech", "Pressure Sensor", "PS1BB"],
    "spec": [
        BoolConv("pressure_state", "binary_sensor", mi="2.p.1060"),  # bool
        Converter("battery", "sensor", mi="3.p.1003"),
    ],
}

I'm not sure about converter's location in the device.py file will affect or not, but when this converter was placed at the end of the file, it just failed.

@Necroneco
Copy link
Contributor

This converter it's not working. Still get unknown after adding it. I make some modifications like below and move it from the end of device.py (MESH DEVICES part) to BLE part (right below 16143: ["Linptech", "Submersion Sensor", "RS1BB"] ). And it works.

{
    16204: ["Linptech", "Pressure Sensor", "PS1BB"],
    "spec": [
        BoolConv("pressure_state", "binary_sensor", mi="2.p.1060"),  # bool
        Converter("battery", "sensor", mi="3.p.1003"),
    ],
}

I'm not sure about converter's location in the device.py file will affect or not, but when this converter was placed at the end of the file, it just failed.

Agree, this should move before the line "default": "ble"

@AlexxIT
Copy link
Owner

AlexxIT commented Oct 30, 2023

@latel
Copy link

latel commented Nov 24, 2023

Will next release fix this issue, I've the same problem.

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.

5 participants