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

Use modern Lisp features and software ADC #4

Draft
wants to merge 40 commits into
base: main
Choose a base branch
from
Draft

Use modern Lisp features and software ADC #4

wants to merge 40 commits into from

Conversation

1zun4
Copy link
Contributor

@1zun4 1zun4 commented Jun 28, 2023

This pull request contains the following changes for VESC 6.05:

  • Utilizes let functions and set, resulting in significantly reduced code.
  • Reformatted the code for improved readability.
  • Modifies the script to pass throttle and brake as ADC input, thanks to the timeout function in VESC 6.05 that addresses previous safety concerns.
  • Detaches hardware ADC, and it is now possible to disable software ADC by changing the software-adc variable from 1 to 0.
  • Disables ADC app when the display is turned off.
  • Awaits resolution of GitHub issue #638 (Feature request: Expose store_backup_data to lisp vedderb/bldc#638) to enable storing odometer data in the future.
  • Enhances ADC functionality to better support features such as traction control, dual or multi-motor setups, and more.

Please review and merge the changes.

1zun4 added 30 commits April 2, 2023 15:58
VESC 6.05 introduced new LispBM features including the curly brackets
Changelog states "Removed array types other than byte arrays."
which made the array-create argument obsolete.
- Lowered default throttle high to 150 (before it was 167) to avoid people running on the edge of the throttle range and now getting the full power of the scooter. This should be changed to your throttle values, but it's a good starting point.
- Redefined the default secret mode values to be more useful for high power scooters.
- High motor temp will now show up on the display
- Locking the scooter will also disable the secret mode now
This script is not tested. DO NOT USE. It might not even run.
g30_dash.lisp confirmed to be working!
added missing { } (progn)
This issue was discussed in vedderb/bldc#655
> If the ADC app is disabled it will not run any set-current, set-rpm etc functions and therefore it stays at this value (and I am not sure if there is any timeout applied, I do not think so...)
@jifferon
Copy link

Any updates on ADC flavor?

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.

None yet

4 participants