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

Feature/apply pretension #255

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

Conversation

EigenMania
Copy link
Contributor

Pretension feature, since many users keep asking for it. When enabled, a constant current of 1% is applied when above a configured road-speed. Speed sensor necessary to use the feature. Can be enabled and disabled from the GUI.

@danielnilsson9
Copy link
Owner

Thank you! Looks nice on a first check, I want to get this merged. But this PR has a lot of commits and merge commits in history. Any chance you could rebase your branch on my master and the submit your changes as a single commit instead?

@EigenMania
Copy link
Contributor Author

I also realized (after submitting the PR) that I should also update some of the GUI figures to show the new fields. So I'll make these updates and then re-submit the PR and squash the commits.

@EigenMania EigenMania force-pushed the feature/apply_pretension branch 2 times, most recently from cbd3b5c to e4efd94 Compare December 10, 2024 16:46
@EigenMania
Copy link
Contributor Author

Rebased and squashed :)

@DrMouse84
Copy link

Thanks for doing this gentlemen! 😊 Can't wait to update! 😁

@Larswagner1987
Copy link

Is there already a finished version available for download?

pretension speed cutoff configurable via GUI with units

small fixes and cleanup

double default speed for pretension feature for safety

fix compile error and pre-compute pretension cutoff speed in init

require use of speed sensor for pretension feature

update readme with new screenshot of config tool

fix line endings in app.c

fix line endings in Configuration.cs

fixed line endings in SystemView.xaml
@EigenMania EigenMania force-pushed the feature/apply_pretension branch from 93c0242 to 1839259 Compare January 15, 2025 20:48
@EigenMania
Copy link
Contributor Author

Fixed some issues with line endings, now the diff should be easier to review. Some files (like SystemView.xaml) use the Unix standard with LF, while most others follow the DOS standard with CRLF, and this caused a bit of a mess at some point.

@DrMouse84
Copy link

Hello gentlemen, any updates yet?

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.

4 participants