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

Disable temp protection #95

Closed
stylesuxx opened this issue May 22, 2023 · 2 comments · Fixed by #100
Closed

Disable temp protection #95

stylesuxx opened this issue May 22, 2023 · 2 comments · Fixed by #100
Assignees
Labels
Milestone

Comments

@stylesuxx
Copy link
Contributor

Since temp protection has not been working till the last release a lot of people seem to have issues related to that. We should probably disable temp protection by default, especially now that the warmer season is starting.

@stylesuxx stylesuxx added this to the v0.20.0 milestone May 22, 2023
@damosvil
Copy link
Contributor

Agreed

@stylesuxx
Copy link
Contributor Author

This is simple enough - if it's OK for you, I would like to add this PR after cleanup ;-)

@stylesuxx stylesuxx self-assigned this May 31, 2023
stylesuxx added a commit that referenced this issue Jun 5, 2023
@stylesuxx stylesuxx linked a pull request Jun 5, 2023 that will close this issue
stylesuxx added a commit that referenced this issue Jul 14, 2023
* Codebase splitted in modules
* Updated makefile for splitting
* 'Cherry picked' 0.19.2 bugfix.
* Update editorconfig
* Removed dangling ENDIF
* Open bootloader file with banner
* Add autoformatter and linting tool
* Initial pass with the formatter
* Disable comment formatting by default (except inline comments)
* Make sure all files end with an empty line
* Unified comments in layout files according to #96
* Unified comments in accordance to #96
* Unified pinouts in main and layout files #13
* Do not remove spaces in inline comments
* Last pass with the formatter
* This commit reduces the scheduler timebase from 16ms to 128ms to interfere the minimum with commutation
* Refactored duplicate assignments #99 (#101)
* Refactored duplicate assignments
* Refactored CSEG definitions into separate file
* Refactored MCU types and checks
* Fixed check time bug and replaced schedule time literals by macros
* Replaced Stop_Adc and Start_Adc macro calls by Reastart_Adc macro call
* Updated names to more descriptive.
Added parameter to select timebase depending on the needs. Scheduler cycle set to 1024ms by default.
* Add 'enums' for PWM frequency and resolution #103 (#104)
* Bugfix/bring back bidirectional 3d mode (#105)
* Fixed temperature limiting affecting to 3D mode.
* Fixed Pwm_Limit_By_Rpm to match comments concept
* Implemented EDT safety arm feature
* Disable temp protection by default #95 (#100)
* Add ENUM for DSHOT commands (#117)
* Refactored MCU specific offsets to separate module (#116)
* Use a single macro for values. (#120)
* Renamed functions to improve readability
* Update melody size and tag count to be a single macro
* Implemented lock and unlock flash macros (#121)
* Refactored 16 bit division by two into seperate macro #112 (#119)
* Initial draft for supporter section (#89)
* Bump version number and mark as DEV version #122 (#124)
* Add TuneRC and T-Motor to the supporter section #85
* Add functionality to support two char layouts. Add OA layout. #113 (#114)
* Removed relative jumps (#126)
* Added/improved a couple of comments to better explain what is going on. (#107)
* Improved documentation, unified formatting, added explanations where necessary.
* Disable dithering by default (#130)
* Allow version string to be passed to Makefile via buildscript (for naming resulting files) (#128)
* RC7 tag (#132)
* Apply version also if building all files
* Fixed RC version
* Further reducing motor speed before changing direction in 3D bidirectional mode (#138)
* RC10
* Add happymodel and weebleed to the 'Supporters' section #139 (#140)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants