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

feat: add some new roborock codes and add custom command #234

Merged
merged 3 commits into from
Nov 12, 2024

Conversation

Lash-L
Copy link
Collaborator

@Lash-L Lash-L commented Nov 10, 2024

No description provided.

@humbertogontijo
Copy link
Owner

Wouldn't be better to return all parsed data? Something like get_parsed_map

@Lash-L
Copy link
Collaborator Author

Lash-L commented Nov 11, 2024

Wouldn't be better to return all parsed data? Something like get_parsed_map

@humbertogontijo

Eh - maybe? But after talking with Piotr for the map card, he really just needs the calibration points.

Calibration points are already in a dictionary and serializable. The entire parsed map is not, which I'm afraid could cause some problems with Vacuum: Send Command

I think I would prefer to leave it like this and if a use case pops up, I will add a second one for get parsed map and think through a solution

@humbertogontijo
Copy link
Owner

Alright

@Lash-L Lash-L merged commit c8507ef into main Nov 12, 2024
7 checks passed
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