congatudo / congatudo-add-on

A home assistant addon to provide a Valetudo server for Cecotec Conga vacuums
MIT License
40 stars 6 forks source link

Cannot edit Segments #39

Closed EzeLamar closed 1 year ago

EzeLamar commented 1 year ago

Describe the bug

Hi, I cannot rename, split or merge Segments using the UI. When I try to do that I got the next error:

image

And in the logs I got the next warning (rename action):

2023-02-12T02:18:31.334Z
Warn
Received "PUT "to "/api/v2/robot/capabilities/MapSegmentRenameCapability" with invalid payload [
  {
    keyword: 'type',
    dataPath: '.segment_id',
    schemaPath: '#/properties/segment_id/type',
    params: { type: 'string' },
    message: 'should be string'
  }
]

To Reproduce

Steps to reproduce the behavior:

  1. Create a new map by zero
  2. Try to edit a Segment
  3. See error

Vacuum Model

Conga 3490

Congatudo-Add-On Version

0.2.5 - 2022-10-21 (latest version)

Thanks!

elraro commented 1 year ago

Hi @EzeLamar !

Please, try the new release of the Congatudo addon 2023.02.0

txitxo0 commented 1 year ago

Anyway, this has to be a bug related to congatudo, not congatudo addon. Just to be clean and organiced

EzeLamar commented 1 year ago

Thanks @elraro for the quick answer! I will try it as soon as it is available

Nismonx commented 1 year ago

@EzeLamar It's very finicky with the last 2 releases, you have to click in the green lines ( not all of them work ) at least it's how I get around on my 4090. Not very practical if you use the congatudo interface Screenshot_20230212_170754_Home Assistant

EzeLamar commented 1 year ago

Thanks @Nismonx! I was fighting with the UI for a while until I figured out that I should click on the green lines 😅 Everything is working again and my segments were successfully configured. @elraro, you rock!

elraro commented 1 year ago

Hi @EzeLamar .

Can you share a screenshot of your map? Do you see the segments?

EzeLamar commented 1 year ago

Sure @elraro, I can see and edit the segments without any problem (tested join, add cutting lines and rename it). I attach a screenshot showing the problem solved.

image

Segments renamed:

image
Nismonx commented 1 year ago

@EzeLamar if you want to send the conga to clean lets say the 'Living' are you able to do so just by touching the triange or do you have to touch in the green lines?

EzeLamar commented 1 year ago

@Nismonx unfortunately, the only way is touching the green line, the triangle doesn't work.

Nismonx commented 1 year ago

Not very practical therefore the issue still persist despite the little work around to get things done. Hopefully @elraro and the team will find a solution 🤞

elraro commented 1 year ago

Hi, please update your addons with the new version Congatudo 2023.03.0 in beta channel

elraro commented 1 year ago

Hello,

Can you try the beta addon @EzeLamar ? If you still have problems, open a issue in https://github.com/congatudo/Congatudo