thorrak / fermentrack

A replacement web interface for BrewPi
MIT License
138 stars 61 forks source link

use gravity as setpoint in ferm profile #287

Open brendongough opened 6 years ago

brendongough commented 6 years ago

In additional to the current time based setpoint, allow for a gravity based setpoint. There's a lot of factors which could change expected fermentation throughout a time frame however the gravity measurement is a true indication. This would be particularly useful for stepped fermentation profiles; fast ferment lager or using it to do a diacetyl rest close to terminal gravity. A check could be added prior to assigning a ferm profile to a controller to ensure it has a gravity measurement before allowing a user to apply a specific profile.

thorrak commented 6 years ago

This is coming. The goal is to enable three types of gravity-based profiles:

Gravity targets (Hold temp until 1.030) Attenuation targets (Hold temp until 30% attenuated) Phase targets (Hold temp until end of lag phase)

Gravity targets are the simplest to implement, phase targets may turn out to be more difficulty than its worth - but the goal is to have all 3.

These will be implemented after #93 and will probably require some refactoring of the fermentation profile code.

gizzygizmo commented 3 years ago

one note... the profiles should be able to have an "AND/OR" ability. There is a possibility of range issues, battery issues, network issues, etc. If your profile only progresses to next step when "gravity" breaks a set-point it may be stuck forever if we've lost communications. Or worse, if a sudden spike causes your fermenter to d-rest during turbulent fermentation.

"AND" would be good for "3days" and "Gravity below 1.020" to cause diacetyl rest "OR" would be good for similar logic if you trust tilt better but want to ensure that at least day7 we have rest -> "7 days or below 1.020"

thorrak commented 3 years ago

Agreed

uSlackr commented 3 years ago

Agreed. Perhaps the gravity reading should be for some period of time – gravity below 1.010 for 4 hours or some such.

\Gregfile://Greg

From: Johnmailto:notifications@github.com Sent: Tuesday, December 8, 2020 11:25 AM To: thorrak/fermentrackmailto:fermentrack@noreply.github.com Cc: Subscribedmailto:subscribed@noreply.github.com Subject: Re: [thorrak/fermentrack] use gravity as setpoint in ferm profile (#287)

Agreed

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fthorrak%2Ffermentrack%2Fissues%2F287%23issuecomment-740737612&data=04%7C01%7C%7Cc731280c25444d178e4f08d89b95edd8%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637430415519742192%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=kpjmpVygmcpg6C3AbTFUNuF0O08tYtyAoR2KMZZoU%2FY%3D&reserved=0, or unsubscribehttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAGEO6Z67YWCKX7PLPG3VTLTSTZHQ3ANCNFSM4FJF3VFA&data=04%7C01%7C%7Cc731280c25444d178e4f08d89b95edd8%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637430415519752188%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=%2FFC1G9sXlRkKNQ7UgfStCjbZDGqKHMAhyPyDwFG2hNg%3D&reserved=0.

thorrak commented 3 years ago

For what it's worth, I'm leaning towards targeting this next after I get Docker support released. I've got a lot of Tilt-related changes that I'm working on behind the scenes - this feels like a good time to get this in.

gizzygizmo commented 2 years ago

Another thought... while setting up a lager today. There is no "free rise" capability. Would be nice to disable the heater function in the beer profile so I could set a max temp and let the beer free rise.

For example, i would normally pitch at 50degF and let it free rise to 55degF. If I try this with a profile i have to set a 1d or 2d at "55" and let it ramp... but fermentrack will actually use the heater to get it there... rather than a real free rise.

Maybe an indicator that you're setting "free rise" and that means the cooling will only be activated and the heater will be disabled until the next setpoint.

Frits1980 commented 2 years ago

Plus one for this feature request, from my part. Really like the ideas presented in this thread. Makes total sence.