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

updated device names to match dyson names and models #28

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nberardi
Copy link

@nberardi nberardi commented Sep 7, 2022

I found the naming very confusing when trying to pick the correct device, and decided to provide a pull request to match the names against the Dyson names and models to help future people who consume the library.

Copy link

@dotvezz dotvezz left a comment

Choose a reason for hiding this comment

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

Great intent! However, some of these updates are explicitly incorrect, I think they would lead to more serious confusion.

DEVICE_TYPE_PURE_COOL_LINK: "Pure Cool Link (TP02)",
DEVICE_TYPE_PURE_COOL_LINK_DESK: "Pure Cool Desk (DP01)",
DEVICE_TYPE_PURE_HOT_COOL: "Pure Hot+Cool (HP04)",
DEVICE_TYPE_PURE_HOT_COOL_NEW: "Purifier Hot+Cool Formaldehyde (HP07/HP09)",
Copy link

Choose a reason for hiding this comment

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

I'm pretty sure this is incorrect.

DEVICE_TYPE_PURE_HOT_COOL_NEW is "527E" which is Formaldehyde HP09. However, the code for HP07 is "527K" and it is not a Formaldehyde model.

DEVICE_TYPE_PURE_HOT_COOL_NEW: "Purifier Hot+Cool Formaldehyde (HP07/HP09)",
DEVICE_TYPE_PURE_HOT_COOL_LINK: "Pure Hot+Cool Link (HP02)",
DEVICE_TYPE_PURE_HUMIDIFY_COOL: "Pure Humidify+Cool (PH01)",
DEVICE_TYPE_PURIFIER_HUMIDIFY_COOL_FORMALDEHYDE: "Pure Humidify+Cool Formaldehyde (PH03/PH04)",
Copy link

Choose a reason for hiding this comment

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

I believe this change is also incorrect. PH03 and PH04 are both "Dyson Purifier" and not "Dyson Pure" models.

Reference: https://www.dyson.com/air-treatment/air-purifier-humidifiers/purifier-humidify-cool-ph03/white-silver.

Additionally, PH04 is Formaldehyde, and PH03 is not.

DEVICE_TYPE_PURE_HOT_COOL_LINK: "Pure Hot+Cool Link",
DEVICE_TYPE_PURE_HUMIDIFY_COOL: "Pure Humidify+Cool",
DEVICE_TYPE_PURIFIER_HUMIDIFY_COOL_FORMALDEHYDE: "Purifier Humidify+Cool Formaldehyde",
DEVICE_TYPE_PURE_COOL: "Pure Cool (TP04/TP07)",
Copy link

Choose a reason for hiding this comment

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

I believe this is incorrect. DEVICE_TYPE_PURE_COOL is the code "475", which is the model TP04.

TP07 is a "Purifier Cool" model, and has a different code (Likely 475 with a letter suffix, similar to how Pure Hot+Cool HP04 is "527" and Purifier Hot+Cool HP07 is "527K")

@nberardi
Copy link
Author

I am going to be abandoning this change. The constants don't match up with the actual list of Dyson products, and it is getting too hard to desirer what is what.

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