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.

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

Alphatec: workaround status 8 if disabled #7222

Merged
merged 3 commits into from Apr 6, 2023
Merged

Alphatec: workaround status 8 if disabled #7222

merged 3 commits into from Apr 6, 2023

Conversation

premultiply
Copy link
Member

No description provided.

@premultiply premultiply added the devices Specific device support label Apr 4, 2023
@premultiply premultiply marked this pull request as ready for review April 4, 2023 19:45
@premultiply premultiply requested a review from andig April 4, 2023 19:46
@premultiply premultiply self-assigned this Apr 4, 2023
@andig
Copy link
Member

andig commented Apr 5, 2023

@premultiply Könntest Du bitte noch kommentieren, was hier erreicht werden soll? Das ist nicht ganz offensichtlich.

Ich lese: bei Status 8 geben wir immer A/B zurück, je nachdem welchen Status wir vorher gesehen haben; aber nie C? Dann würde ich den gemerkten Status auch nur bei A/B setzen. Oder wir geben bei Status 8 einfach immer B zurück falls das gewünscht ist.

@premultiply
Copy link
Member Author

Genau. C kann es ja nicht sein, da disabled ja nicht gleichzeitig heissen kann dass weiter geladen wird. Also ist es B wenn es vorher C war.

Leider bekommt man aber im disabled-Zustand dann halt nicht mehr mit wenn ein Fahrzeug ab- oder angesteckt wurde...

@andig
Copy link
Member

andig commented Apr 5, 2023

Und wie kommt man in den Disabled Zustand? Genau mit disable? Wenn das so ist- sollten wir dann nicht lieber für disable Strom=0 schreiben und dieses Register schön in Ruhe lassen?

@premultiply
Copy link
Member Author

Strom muss natürlich mindestens 6 sein... 🤪

@andig
Copy link
Member

andig commented Apr 5, 2023

Alter... und 0 geht auch wirklich nicht zu schreiben? Auch wenn die Doku das nicht mag?

@premultiply
Copy link
Member Author

Ja, leider. Siehe Slack Thread.
Maximal dämlich.

@premultiply premultiply merged commit 413f184 into master Apr 6, 2023
5 checks passed
@premultiply premultiply deleted the fix/alphatec branch April 6, 2023 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devices Specific device support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants