arquicanedo / barbecueboss

Barbecue ConnectIQ App for Garmin Devices
Apache License 2.0
5 stars 0 forks source link

Smoking reminders to check the temperature / water / smoke #62

Open nilsbenson opened 4 years ago

nilsbenson commented 4 years ago

It would be really useful for smoking if the app could wake up periodically and let you know to go check the temperature, water status (for smokers with water), and wood status to make sure everything is good.

Another good check would be to warn the user 30 minutes or so before the timer is done to check the internal temperature of the meat so you know if the time needs to be extended.

arquicanedo commented 4 years ago

This is a great feature.

nilsbenson commented 4 years ago

I think we should consider at some point to not only have the temp check timer - but also be able to configure each probe to a set target temp so the user can be notified when the meat hits temp because based on the outside temperature it could happen well before they think it should be done.

arquicanedo commented 4 years ago

Yes, I agree. Probe temperature alarms are the most useful. This is the basic interface for the 2 thermometers I have.

arquicanedo commented 4 years ago

After writing the previous comment I was thinking that probably the most natural interface for the low-n-slow is to have unbounded timers. Everything is temperature-driven. Instead of having the DurationPicker as the main action in the low-n-slow, we could have a TemperaturePicker for the probes and simply let the time go as it needs to.

The only problem with this is if we don't reliably get temperature values from the probe (e.g., battery runs out in the thermometer). We could burn or overcook some food.

nilsbenson commented 4 years ago

I agree. My smoker sits outside far enough away that if I'm anywhere in the house I can't get a connection. It picks it up if I'm within about 40-50 ft., but that's not far enough for me to pick it up inside the house. The timer is more of a general thing, and makes it useful even if you don't have a thermometer.

The way I see it no therm:

with thermomter:

One of the issues I have is that the smoker sits in full sun, and in the middle of the summer it can be difficult to control the smoking temperature so the probe is a nice way to keep on top of it. If you don't have a probe, or don't have bluetooth it would still be nice to be told to check on it (or I have a tendency to forget).

arquicanedo commented 4 years ago

The current implementation keeps the reminder for 2 minutes after the check time has been reached. Perhaps adding vibration would be beneficial.

Having the user clear the alarm would be a bit problematic and cumbersome.

Another option would be to create a custom screen with the alarm. The water/smoke/temp icon in the center and some vibration until you clear it.

nilsbenson commented 4 years ago

I was thinking we could have a view that gets pushed that has the alert information. Then the user just clicks the go button and it goes away.

This could also be used to notify users when running the background that they need to check something.

For the low-mem devices I think it's possible too, but we will have to see.

arquicanedo commented 4 years ago

I'll prioritize this between tomorrow and Friday. Then cleanup the mess with the watch-specific strings.xml. This is unecessary and created a mess.

arquicanedo commented 4 years ago

I think the basic implementation works. The only possible improvement would be to reset the warning flags when:

arquicanedo commented 4 years ago

I think the basic implementation is ok. We just need to persist the warning flags and add the logic to decide when to warn the user after the app has been re-opened during a smoking session.