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

Beep when temp reached #1561

Merged
merged 39 commits into from
Jan 30, 2021

Conversation

kisslorand
Copy link
Contributor

@kisslorand kisslorand commented Jan 24, 2021

Description

This PR adds an acoustic acknowledge that the desired temperature is reached, given it to be the nozzle, the bed or the chamber.
It is working only if the temperature is set on the TFT by hand or automatically (Load/Unload, Extrude. Preheat, Heat, etc). It's not active if a gCode sets the temperature. It's not active if you set the temperature on the terminal or by any external command.
It is falling into the rule of "alert_sound" in the Config.ini.
It makes distinctive sounds, depending on the temperature if it has fallen or risen to the desired one.

The detection algorithm has a hysteresis of 2 °C.

Fixes what?

Fixes #1166 and adds more.

Additional info

If desired, it can be made to be separately adjustable to be on or off, only for nozzle, only for bed, only for chamber, only for nozzle and bed, only for nozzle and chamber, only for bed and chamber, for all three, only for rising, only for falling and any of these combinations.
It's not my idea, I had it like this on a previous custom FW.

A few hours later...

It's been made configurable by user, it can be enabled or disabled in the "Config.ini" and Menu->Settings->Screen->Sound.
Default is ON.

kisslorand and others added 30 commits November 28, 2020 12:39
@sarvenn
Copy link

sarvenn commented Jan 25, 2021

Hi @kisslorand , nice idea.
+1

@traffic-light
Copy link
Contributor

traffic-light commented Jan 25, 2021

Thanks for duplicates....
#1209

@digant73
Copy link
Contributor

digant73 commented Jan 26, 2021

Minor cleanup. As done for all the parameters description, in config.ini append "." at the end of your new param:

# heater_sound: Enable/disable this to control acoustic feedback when temperature
#_________________has reached the desired value on heaters (nozzle, bed, chamber)

@kisslorand
Copy link
Contributor Author

kisslorand commented Jan 26, 2021

Minor cleanup. As done for all the parameters description, in config.ini append "." at the end of your new param:

#heater_sound: Enable/disable this to control acoustic feedback when temperature

#_________________has reached the desired value on heaters (nozzle, bed, chamber)

Done. Thanks for letting me know.

@bigtreetech bigtreetech merged commit 891ab66 into bigtreetech:master Jan 30, 2021
Uz45 added a commit to Uz45/BIGTREETECH-TouchScreenFirmware that referenced this pull request Jan 31, 2021
adds keyboard layout for quertz awerty bigtreetech#1550
Correction CZ translation bigtreetech#1546
FIX bigtreetech#1539 bigtreetech#1541
Fix get filesize and cancle print for RepRapFirmware bigtreetech#1533
This PR is fixing the bugs introduced by merging PR bigtreetech#1487 bigtreetech#1526
update german translation and add missing icons bigtreetech#1525
Add missing icons and Hungarian language bigtreetech#1520
Added Home Offset menu + bugfixes and cleanup bigtreetech#1518
Readme.md update and format fix bigtreetech#1557
Reduce global variables and cleanup bigtreetech#1558
Added the possibility to change to time the bootscreen logo is shown bigtreetech#1560
Beep when temp reached bigtreetech#1561
Added new param in FeatureSettings menu + toggle ACK in Terminal menu… bigtreetech#1563
jeffeb3 pushed a commit to V1EngineeringInc/BIGTREETECH-TouchScreenFirmware that referenced this pull request Nov 10, 2021
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.

[FR] Pre heat beep/tune/tone to acknoledge the set temperatures have been reached
6 participants