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

[SUGGESTION] Rename new radio type from znsp to either espznsp or espzb instead (so less easy to confuse/misstake for znp) #11

Open
Hedda opened this issue Mar 22, 2024 · 1 comment

Comments

@Hedda
Copy link
Contributor

Hedda commented Mar 22, 2024

Suggest renaming the new radio type for this from znsp to either espznsp or espzb instead, (for IMHO several very good reasons).

That way the radio type will have "esp" in the name which will better match the name of the library and products is compatible with.

It would also made it read less similar to the existing znp radio type that is already used by the zigpy-znp radio library, so more unique.

Cross-post from here -> zigpy/zigpy-cli#45 (comment)

@lhespress nitpicking and maybe trivial, however as suggested by puddly in zigpy/zigpy-cli#42 perhaps would it not be a a good idea before this is established if also the new radio type was renamed from znsp to either espznsp or espzb instead to it both has "esp" in the name and better match the name of the library so that it does not sound too similar to existing znp radio type that is used by the zigpy-znp radio library?

Perhaps just my dyslexia but reading znsp and znp as block words then they look similar to me at a glance. So at least I think that might make it a little less likely that the radio type name is accidentally confused with znp by users, or what do you think?

zigpy/zigpy-cli@39f6642#diff-bb72967ef9d1e64ff68ad3f1a4e13e19b4cc7f05359c100016f76560fbaea52c

https://github.com/zigpy/zigpy-cli/blob/39f6642aed66bea1ba5145e5c587655f8fae57b6/zigpy_cli/const.py#L17

https://github.com/zigpy/zigpy-cli/blob/39f6642aed66bea1ba5145e5c587655f8fae57b6/zigpy_cli/const.py#L62

@Hedda
Copy link
Contributor Author

Hedda commented Aug 4, 2024

@lhespress Do you have any more thoughts on these name change suggestions?

Again, please see all related comments in your pull request for zigpy-cli here -> zigpy/zigpy-cli#45

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

No branches or pull requests

1 participant