Skip to content

Add initial support for Goat A1600 RTK (#852) #949

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.

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

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

comtel2000
Copy link

@comtel2000 comtel2000 commented May 6, 2025

I still got issues with pytest, is there hardware required to run it?

ERROR tests/test_device.py - NameError: name 'x5d34r' is not defined

For now, it's just a copy of the GOAT G1 without AdvancedMode and a new CustomCutMode..

from .common import GetEnableCommand, SetEnableCommand


class GetCustomCutMode(GetEnableCommand):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does this option do?
Can you post a screenshot from this setting in the app

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please answer this comment

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's what I tried to find out in my test-setup. I've tried almost everything and it's not changeable via app setting, but it probably determines which options are available in the app. It is activated for the A1600 RTK in any case.

It may enable the "Enhanced" mode..

image

@edenhaus edenhaus added the pr: new-feature PR, which adds a new feature label May 7, 2025
@comtel2000
Copy link
Author

I'm just trying to get to grips with the deebot and python. The MR is only intended as a very simple basic support for the A1600.
With my very limited python skills I also like to help out with captured mitmproxy data 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: new-feature PR, which adds a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants