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

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

Open
nilsbenson opened this issue Jun 18, 2020 · 10 comments
Open

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

nilsbenson opened this issue Jun 18, 2020 · 10 comments
Milestone

Comments

@nilsbenson
Copy link
Collaborator

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.

@nilsbenson nilsbenson added this to the low-n-slow milestone Jun 18, 2020
@arquicanedo
Copy link
Owner

This is a great feature.

@nilsbenson
Copy link
Collaborator Author

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
Copy link
Owner

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

@arquicanedo
Copy link
Owner

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
Copy link
Collaborator Author

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:

  • you can estimate the smoking time and temperature based on meat type and weight
  • enter the time to smoke
  • setup the smoke/water/temp check
  • get notifications during the smoking to check on it so you don't forget
  • when notified you can stick a thermometer in it when you think it might be ready

with thermomter:

  • things work the same
  • when the probe reaches the desired internal temp it notifies and you know that it's done, regardless of how much time you set on it

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
Copy link
Owner

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
Copy link
Collaborator Author

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
Copy link
Owner

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 added a commit that referenced this issue Jul 29, 2020
…them if they occur at the same time: #62. I cleaned up the mess with the custom strings for smoke checks.
@arquicanedo
Copy link
Owner

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

  • Smoke timer is reset (e.g., set for another 2 hours)
  • App wakes up - the warning flags are not persisted now so there is no way of knowing whether the warning has been cleared by the user

@arquicanedo
Copy link
Owner

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.

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

No branches or pull requests

2 participants