Skip to content

Unable to get correct room ids #37487

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.

Sign up for GitHub

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

Jump to bottom
Closed
rosinscj opened this issue Feb 16, 2025 · 4 comments
Closed

Unable to get correct room ids #37487

rosinscj opened this issue Feb 16, 2025 · 4 comments
Assignees
@allenporter
Labels
integration: roborock Stale The PR had no activity for quite some time, and is marked as Stale

Comments

@rosinscj
Copy link

rosinscj commented Feb 16, 2025 •
edited
Loading

Feedback

Following directions at https://www.home-assistant.io/integrations/roborock

Performing the roborock: get maps action, with the following payload:
action: roborock.get_maps target: entity_id: vacuum.roborock_qrevo_curv

Yields the following room ids with the wrong room labels (duplicated across maps):

  maps:
    - flag: 0
      name: 2nd floor
      rooms:
        "1": Master bedroom
        "3": Study
        "6": Guest bedroom
        "7": Unknown
    - flag: 1
      name: First floor - full
      rooms:
        "1": Master bedroom
        "3": Study
        "6": Guest bedroom
        "7": Unknown

I've attempted to switch maps in both the app and HA UI + reloading the integration, no changes to the above
Tried the workaround specified in home-assistant/core#123791 with no luck

I also tried using the ids in the resposne from get_room_mapping without success:

 	[2, '9256440', 15], 
 	[3, '9256494', 14], 
 	[4, '9256630', 10], 
 	[5, '9288998', 13]]

	[3, '9293020', 9], 
	[6, '9293022', 3], 
	[7, '', 15]]

The 7 digit numbers correspond to:

rooms=[HomeDataRoom(id=9293024, name='Master bedroom'),
	HomeDataRoom(id=9293022, name='Guest bedroom'),
	HomeDataRoom(id=9293020, name='Study'),
	HomeDataRoom(id=9288998, name='Dining room'), 
	HomeDataRoom(id=9256825, name='Entryway'), 
	HomeDataRoom(id=9256630, name='Hall'), 
	HomeDataRoom(id=9256494, name='Kitchen'), 
	HomeDataRoom(id=9256440, name='Bathroom'), 
	HomeDataRoom(id=9256439, name='Living room')])

URL

https://www.home-assistant.io/integrations/roborock/

Version

2025.2.4

Additional information

No response
@home-assistant
Copy link

Hey there @Lash-L, @allenporter, mind taking a look at this feedback as it has been labeled with an integration (roborock) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of roborock can trigger bot actions by commenting:

  • @home-assistant close Closes the feedback.
  • @home-assistant rename Awesome new title Renames the feedback.
  • @home-assistant reopen Reopen the feedback.
  • @home-assistant unassign roborock Removes the current integration label and assignees on the feedback, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information) to the feedback.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information) on the feedback.

@github-actions GitHub Actions
Copy link

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved.
If this issue is still relevant, please let us know by leaving a comment 👍
This issue has now has been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the Stale The PR had no activity for quite some time, and is marked as Stale label Apr 17, 2025
@Lash-L
Copy link
Contributor

Lash-L commented Apr 18, 2025

Hi @rosinscj This issue was created on the wrong repository. You actually want to make one here: https://github.com/home-assistant/core

That being said, please give 2025.4.2 a try as a lot of improvements for maps were made in 2025.3 and 2025.4 and some bugfixes were made in 2025.4.2

@Lash-L
Copy link
Contributor

Lash-L commented Apr 18, 2025

@home-assistant close

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@allenporter allenporter

Labels
integration: roborock Stale The PR had no activity for quite some time, and is marked as Stale
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@allenporter @rosinscj @Lash-L