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.

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

Add support for Cecotec Conga #689

Closed
wants to merge 2 commits into from
Closed

Add support for Cecotec Conga #689

wants to merge 2 commits into from

Conversation

adrigzr
Copy link

@adrigzr adrigzr commented Feb 15, 2021 •
edited
Loading

Type of change

  • [ x] New robot implementation

Description

Add library Agnoc for working with Conga Robots.

Currently tested against models 3090, 3490 and 4090.

@Snakesms

This comment has been minimized.

Sign in to view
@adrigzr adrigzr force-pushed the feature-conga branch 4 times, most recently from 06e2956 to 7cc2659 Compare April 2, 2021 12:54
@adrigzr adrigzr changed the title Add support for Cecotec Conga 3490 Add support for Cecotec Conga Apr 2, 2021
@adrigzr adrigzr force-pushed the feature-conga branch 7 times, most recently from b1f6756 to 9ba22b0 Compare April 7, 2021 11:48
@adrigzr adrigzr force-pushed the feature-conga branch 2 times, most recently from e7e705e to a477f66 Compare April 22, 2021 12:55
@adrigzr adrigzr marked this pull request as ready for review April 22, 2021 13:37
@adrigzr adrigzr force-pushed the feature-conga branch 3 times, most recently from 0acbdc8 to 9adfb46 Compare April 25, 2021 11:04
@adrigzr adrigzr mentioned this pull request Apr 26, 2021
Open
@adrigzr adrigzr force-pushed the feature-conga branch 3 times, most recently from 3b04271 to 842bc69 Compare April 27, 2021 14:35
@witokondoria
Copy link

I have a pair of doubts regarding this PR and my setup (Conga 3090, built from 32b01e1):

· segments are not displayed on the web UI, even there are some classes implementing the capabilities. Is it a know bug or a WIP action?
· on previous builds, the segments tab was displayed, showing the map. The issue there is that the background was blue and no segment could be spliy/merged/renamed. Could this be a 3090 limitation?

On top of that: awesome contribution 👍

@adrigzr
Copy link
Author

adrigzr commented Jun 17, 2021 •
edited
Loading

I have a pair of doubts regarding this PR and my setup (Conga 3090, built from 32b01e1):

· segments are not displayed on the web UI, even there are some classes implementing the capabilities. Is it a know bug or a WIP action?
· on previous builds, the segments tab was displayed, showing the map. The issue there is that the background was blue and no segment could be spliy/merged/renamed. Could this be a 3090 limitation?

On top of that: awesome contribution 👍

Hi. Model 3090 has no support for segments as segments are rooms identified by the robot itself. You can achieve almost the same using zones instead of segments.

@adrigzr adrigzr force-pushed the feature-conga branch 2 times, most recently from bec2d97 to e229898 Compare July 7, 2021 08:25
@adrigzr adrigzr force-pushed the feature-conga branch 2 times, most recently from 1cd61ae to 661ce56 Compare July 30, 2021 09:58
adrigzr and others added 2 commits September 20, 2021 09:24
@adrigzr
feat(vendor.cecotec): add support for cecotec conga
6797971
@txitxo0 @adrigzr
feat(ui): re-enable zones button with new api
3eaf42b 
Co-authored-by: Jesús de Luis <jesuslui@youforce.net>
Co-authored-by: Jesus de Luis <jdeluiserrano@gmail.com>
@Hypfer Hypfer closed this Oct 10, 2021
Repository owner locked as resolved and limited conversation to collaborators Oct 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@adrigzr @Snakesms @witokondoria @Hypfer @txitxo0