CapnBry
commented
8 years ago Closing pull request, mostly redundant servo max ceil and fan min floor options. Create a new pull request if additional functionality is desired.
Closed R-Burk closed 8 years ago
CapnBry
commented
8 years ago Closing pull request, mostly redundant servo max ceil and fan min floor options. Create a new pull request if additional functionality is desired.
I'll start off with I don't expect this to be the final form.
What I have tried to accomplish is allowing the servo to do natural convection but the fan can be brought in a linear fashion if more is needed. This has allowed me to walk away and not be as worried if I have the top damper setting correct. It really saved me over Thanksgiving as I got involved in a football game. Half time I noticed the fan was running at 20% output. Opened the top damper some more over half-time and turkey came out great
Considered making the fan and servo go end to end. So it would be 100 points for servo, then 100 for fan. Would be easy to just divide by 2 where readouts are needed. Found myself wanting some overlap and could see others wanting it as well. Especially the guy with a damper on the output and fan on the input.
I tried to address what happens when the firmware gets flashed and first use would pull a 255 from a new eeprom position. Tried to make the _pidOutput split out in a logical fashion when status is sent to linkmeter. I still like having the fan as a separate item on the graph. I like data but I know that can be overwhelming to some. Probably would create questions.
The print out on the LCD doesn't bother me but I'm worried it could confuse some when it's readying more than 100. Pondered if it could do something like 1-100 as normal. and then just output the fanspeed with an * ( such as 90* or 100*). Really didn't want to go into the menu system as that's an a huge amount of callbacks going on there.
I really think there needs to be a lockout to prevent a user from using all the alternate modes. I added it to the setFanoffset then turned it off with a compile directive. Perhaps someone has a reason for needing to activate several at once. From reading the forums this project ends up in places that are unexpected.
This uses 146 bytes of additional flash compared to current line
Tried to attach a screenshot of last cook. Time 14:20 I noticed the fan was having to pump. Went out and opened the top damper ( it was raining, blowing, and I had unexpected company if you are wondering how I it took me hours to noticed). Bright blue line is the fan