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

Fortius Antifier v4.1.b for #121 #147 #164 #170

Closed
wants to merge 4 commits into from

Conversation

WouterJD
Copy link
Owner

Implemented:

Improve run-off procedure (see issue #147); runoff procedure structured and parameterized.
Ignore head unit buttons (see issue #164); main window can only be closed from GUI.
ANT Remote Control Support (see issue #121); UP / SELECT / DOWN button can be used to replace head unit buttons Up / OK / Down.

Thanks to @cyclingflow and @switchabl for input

@WouterJD WouterJD changed the title #147 improve run off procedure Fortius Antifier v4.1.b for #121 #147 #164 Dec 16, 2020
@WouterJD WouterJD requested a review from IlTulipano December 17, 2020 07:55
rolldown = False
rolldown_time = 0
ShortMessage = TacxTrainer.Message + " | Runoff - "
ShortMessage = "Tacx Trainer Runoff - "
Copy link
Contributor

Choose a reason for hiding this comment

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

Which one of the messages is meant to be displayed? Currently the latter will take precedence.

Copy link
Owner Author

Choose a reason for hiding this comment

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

Agree; first message uses the "standard" but is quite long.
Second message is as suggested by CyclingFlow and indeed replaces the first (could be commented for clarity)

@WouterJD
Copy link
Owner Author

Obsolete

@WouterJD WouterJD closed this Dec 22, 2020
@WouterJD WouterJD deleted the #147-Improve-run-off-procedure branch December 24, 2020 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants