Add more sistalk monsterpub devices and sensors for them#651
Add more sistalk monsterpub devices and sensors for them#651anon1im wants to merge 2 commits intobuttplugio:devfrom
Conversation
|
Exactly which devices did you test? The config updates suggest pressure sensors on devices I'm fairly certain do not have them. |
|
There is "Monster Pub 2 Premium Version" lineup, also called "Monster Pub Kegel" (here). I have tested:
Dr. Whale Premium's pressure sensor works in official app and when using buttplug api |
|
So, I think there's a bit of a translation issue somewhere: the MonsterPub devices (with a few exceptions) come in Whale, Devil and "Godzilla" not Godkilla. The existing Whale 2 identifier was another model with pressure sensor. The new identifier should just be added to the existing configuration. |
|
I suspect that the new identifier should have been noted on buttplugio/docs.buttplug.io#19 first |
|
Naming is a mess: Godzilla is named as "Gokilla" and "Godkilla" on the same screen in the Monster Pub App. Ok, I'll comment new info in the issue |
|
Renamed Godkilla to Godzilla and merged "premium" Whale 2 with existing one. |
blackspherefollower
left a comment
There was a problem hiding this comment.
I think this is a good start.
When I get back, I'll see if the battery readings can be more generalized across the entire brand.
I think longer term, pressure levels via subscription will be more useful, but I'm waiting for the next iteration of the sensor support in the Buttplug protocol before I was going to start looking into that seriously.
ca203ea to
110b302
Compare
|
@blackspherefollower Where we at on this one? |
|
It's probably fine. But I'll have time to test on Monday. |
0f9713e to
e71d832
Compare
Added a few devices that I own and could reverse engineer
Tested with intiface-engine and a python script.
I wanted to add subscriptions to newly added sensors, but couldn't find any mention of SensorSubscribeCmd in
buttplug-device-config-v3.yml