relm923 / MMM-Roomba

Roomba980 module for the Magic Mirror.
MIT License
10 stars 5 forks source link

question for supporting multiple roomba's #1

Closed DoctorOgg closed 3 years ago

DoctorOgg commented 6 years ago

how would one support more than one rooma?

relm923 commented 6 years ago

I only have one Roomba but I should be able to add multi support fairly easily. I'll ping you once I'm done but for now you can add multiple copies of the same widget as a stop gap

DoctorOgg commented 6 years ago

i tried adding multiple instances, however, they seem to only read one of the Roomba settings

relm923 commented 5 years ago

Looks like a limitation of MagicMirror unfortunately. You'd need to give each instance a unique name ad install multiple copies of MMM-Roomba under the unique names.

I can add multi-roomba support but it will require a breaking config change. I'll try to get to soon

relm923 commented 5 years ago

Added multiple Roomba support on mutli_robot_support branch. I haven't fully tested it yet but feel free to give it a shot.

TATUMTOT commented 4 years ago

ok I just tested and if the module is enabled the magic mirror wont load. Hopfully we can get this working.

TATUMTOT commented 4 years ago

@relm923 I have it working with Multiple Instances of MMM-Roomba. I have 3 Roomba's Displaying their status 2X S9+ and 1X 985

image

relm923 commented 3 years ago

Is there still interest in support multiple devices? @TATUMTOT @DoctorOgg

TATUMTOT commented 3 years ago

@relm923 I had to copy the mmm-roomba 3 times for mine. Then go edit the code to make each of them unique. So I have mmm-roomba and then mmm-roomba1 and so on.

relm923 commented 3 years ago

Thanks @TATUMTOT I'll take another look at it. Unfortunately it's difficult for me to test with only one device

relm923 commented 3 years ago

@TATUMTOT did you get errors when attempting to use the mutli_robot_support branch?

TATUMTOT commented 3 years ago

@relm923 i remember you talking about is but i never got it to work. Or the directions were not clear. But i think it just did not show the second one.

relm923 commented 3 years ago

@TATUMTOT mind pulling the latest version of that branch and trying again? You'll need to update your config to pass the robot metadata as a list:

var config = {
    modules: [
        {
            module: 'MMM-Roomba',
            position: 'top_right',
            header: 'Roomba', // Optional
            config: {
                // See below for configurable options
                robots: [
                    {
                        username: 'xxxxxxxxxxxxx',
                        password: ':1:1486937829:gktkDoYpWaDxCfGh',
                        ipAddress: '192.168.1.44'
                    }
                    {
                        username: 'xxxxxxxxxxxxx',
                        password: ':1:1486937829:gktkDoYpWaDxCfGh',
                        ipAddress: '192.168.1.45'
                    }
                ]
            }
        }
    ]
}
TATUMTOT commented 3 years ago

@relm923 I have not had time to redo this. I have to fully redo it. Roombas have been swapped out for replacements and still have not gotten arround to re-registering the API's.

Kanostic commented 3 years ago

Having issues with getting this module to work, I am getting "Config.username is required for module:MMM-Roomba.
MMM-

Below is what I have in my config, replaced the username and password for sharing. { module: 'MMM-Roomba', position: 'middle_center', header: 'Bender', // Optional config: { // See below for configurable options robots: [ { username: 'xxxxxxxxxxxxxx', password: 'xxxxxxxxxxxxxx', ipAddress: '192.168.86.114', } ] }

I do have multiple roombas, just trying to get one to work.

Thanks

relm923 commented 3 years ago

@Kanostic what branch are you using?

relm923 commented 3 years ago

The latest version (2.0.0) now support multiple robots. You'll need to update your config.js to match the new format. See the README for an example