koalazak / dorita980

Unofficial iRobot Roomba and Braava (i7/i7+, 980, 960, 900, e5, 690, 675, m6, etc) node.js library (SDK) to control your robot
MIT License
942 stars 150 forks source link

690 & 890 compatible? #48

Closed iosdeveloper closed 6 years ago

iosdeveloper commented 7 years ago

Doesn't seem to work with the new 690. What about the 890?

koalazak commented 7 years ago

Hi, I dont know. I dont own a 690/980. Anybody try?

iosdeveloper commented 7 years ago

I checked with a 690 owner and it doesn't send the pose values.

pschmitt commented 7 years ago

But the rest works?

iosdeveloper commented 7 years ago

AFAIK, yes. No pose means no map though.

pschmitt commented 7 years ago

That actually makes sense since only the 980 and the 960 have a camera and have the map feature on their spec sheet. The 890 should behave similarly to the 690.

Imgur

koalazak commented 7 years ago

the only diference between 980 and 960 is the carpet bust and the battery. here is the comparison chart: http://www.irobot.com/For-the-Home/Vacuuming/Roomba.aspx#Comparison-Chart

pschmitt commented 7 years ago

@iosdeveloper What does the cap field contain?

For my Roomba 980 that'd be the following:

[('pose', 1),
  ('ota', 2),
  ('multiPass', 2),
  ('carpetBoost', 1),
  ('pp', 1),
  ('binFullDetect', 1),
  ('langOta', 1),
  ('maps', 1),
  ('edge', 1),
  ('eco', 1),
  ('svcConf', 1)]

pose and maps should be set to 0, right?

iosdeveloper commented 7 years ago
"cap": {
    "eco": 1,
    "ota": 1,
    "svcConf": 1
}
pschmitt commented 7 years ago

Oh so they are actually NOT set. Thanks for the info.