search

Roastero / Openroast

An open source, cross-platform application for home coffee roasting
GNU General Public License v3.0
128 stars 47 forks source link

Roast Temp always on high #73

Open jarrettelliott opened 6 years ago

jarrettelliott commented 6 years ago

No matter what I set the target temperature to. Using custom or pre-installed recipes, it just sits all of the time pegged on high. It never cycles through Cool, Low, Med, and High like it's supposed to.

Win 7 Lenovo T450s Plugged into a USB 2.0 port

jolarockarolla commented 6 years ago

We are having this same issue. No matter what the target temperature is set at the current temp is either above 400F and climbing or on cool reading 150. The only way to manipulate temperature is through fan speed, but we can't get it below 400F. Does anyone know how to more gradually raise the temperature?

trankin commented 6 years ago

I’m having exactly the same problem. I’ve tried on 5 different machines, same results. 2 macs and 3 pcs. Same results regardless, Target temp is 200, and the fresh roast continues to climb well past 400.

This is the first time I’ve used the hardware, or software. All of my installs are from the downloadable installers.

trankin commented 6 years ago

FYI, I don't think it has anything to do with USB compatibility. I ended up pulling down the python library and running some tests interfacing with it directly.

I can confirm that even with this simple program, the temperature continues to rise, and my expectation would be that it would not at all. Please let me know if I misunderstand what this should do.

roaster = freshroastsr700.freshroastsr700()

roaster.connect()

roaster.heat_setting = 0 roaster.fan_speed = 9 roaster.time_remaining = 20

roaster.roast() time.sleep(30)

trankin commented 6 years ago

Well gents, good news and bad news. I think I have a clear understanding of the problem.

The Roastero sr700 driver is designed to modulate between heat setting 0 and heat setting 3 in order to regulate temperature to the desired temperature level. I think what we have found is that some SR700 models basically ignore a heat setting 0.

I wrote a test application in c# to verify this, and I found that I was able to modulate the temperature by juggling heatlevel 3 and roaster state cooldown. So, for example .. instead if setting heatlevel to 0 in my example instance, I changed the control packet value to 0x04, 0x04.. to put the roaster into the cooldown state.

This effectively is equivalent physically to setting heat setting to 0.. with the exception that the display on the SR700 shows that it's in cooldown.. only to cycle back to roasting when the heat is turned back on.

I plan on finishing out my C# execution over the next few weeks and add in a proper PID controller.. and a parser for the OpenRoast protocol....

Just wanted to share my findings in case any Python guru wants to dig in and change the way that works in the python driver... after reviewing the code.. it looks like it will take a little refactoring.. because .. the code has logic that is performed when the roaster is in the cooldown state.. and effectively disables the PID.... this would have to be rewritten to know that the roaster was intentionally set to cooldown in order to simulate heatlevel 0..

trankin commented 6 years ago

My proof of concept is here: https://github.com/trankin/OpenRoast-DotNet

trankin commented 6 years ago

Well, it was a bit more straight forward than I thought. Building my version of the code helped me read through the python code better an I've implemented my change in the freshroast driver repository. I've submitted a pull request, no idea what the process is or how long it may take to get that in.

Tagging @int3ll3ct, and @lodge93 here, because while the project isn't terribly active, you two do seem to have the most recent history.

Thanks,

Thomas

msilvey commented 5 years ago

@trankin: I'm playing catchup to this issue. Did you ever get a PR into the freshroastsr700 repo?

pgolay commented 5 years ago

I just now ran my brand new 700 for the first time with one of the supplied (openroast 1.2.1) recipes and indeed the temperature was 440-490 the entire time regardless of the target temp - I cut the roast short by quite a bit, needless to say. Is there any solution to this?

" I think what we have found is that some SR700 models basically ignore a heat setting 0." Did I get a bum one?

daibew commented 5 years ago

Any update on @trankin solution getting integrated in?

trankin commented 5 years ago

No, it turned out to be more effort than I had anticipated. I'm still just running my patch manually.

On Fri, Nov 1, 2019, 6:04 PM daibew notifications@github.com wrote:

Any update on @trankin https://github.com/trankin solution getting integrated in?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Roastero/Openroast/issues/73?email_source=notifications&email_token=AABH5SABRA4AOWNU5JSJ5FLQRSYXRA5CNFSM4FMKFJV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEC4MBEI#issuecomment-548978833, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABH5SHJDZGNORMM3CH7G7LQRSYXRANCNFSM4FMKFJVQ .

daibew commented 5 years ago

@trankin would you be willing to point me in the direction of installing the patch myself? I'm running it on OS X.

trankin commented 5 years ago

Yes. There are a few posts on here somewhere where I put some directions. I'll check later to see if I can track them down.

On Fri, Nov 1, 2019, 6:08 PM daibew notifications@github.com wrote:

@trankin https://github.com/trankin would you be willing to point me in the direction of installing the patch myself? I'm running it on OS X.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Roastero/Openroast/issues/73?email_source=notifications&email_token=AABH5SGHO5D7Z67F3EJHLIDQRSZHNA5CNFSM4FMKFJV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEC4MHFA#issuecomment-548979604, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABH5SFFAUGAI7A7HZA7DVTQRSZHNANCNFSM4FMKFJVQ .

trankin commented 5 years ago

Sorry, I though this was from the Facebook group. That's where I posted.

On Fri, Nov 1, 2019, 6:09 PM Thomas Rankin tom@tomrankin.net wrote:

Yes. There are a few posts on here somewhere where I put some directions. I'll check later to see if I can track them down.

On Fri, Nov 1, 2019, 6:08 PM daibew notifications@github.com wrote:

@trankin https://github.com/trankin would you be willing to point me in the direction of installing the patch myself? I'm running it on OS X.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Roastero/Openroast/issues/73?email_source=notifications&email_token=AABH5SGHO5D7Z67F3EJHLIDQRSZHNA5CNFSM4FMKFJV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEC4MHFA#issuecomment-548979604, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABH5SFFAUGAI7A7HZA7DVTQRSZHNANCNFSM4FMKFJVQ .

trankin commented 5 years ago

I'm going to pick this back up soon to give a full build another try.

Here's the link to the FB discussion and some minor documentation for those that are inclined to try to get the manual version running.

https://www.facebook.com/groups/1471311819774625/permalink/1994758334096635/

On Fri, Nov 1, 2019 at 6:10 PM Thomas Rankin tom@tomrankin.net wrote:

Sorry, I though this was from the Facebook group. That's where I posted.

On Fri, Nov 1, 2019, 6:09 PM Thomas Rankin tom@tomrankin.net wrote:

Yes. There are a few posts on here somewhere where I put some directions. I'll check later to see if I can track them down.

On Fri, Nov 1, 2019, 6:08 PM daibew notifications@github.com wrote:

@trankin https://github.com/trankin would you be willing to point me in the direction of installing the patch myself? I'm running it on OS X.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Roastero/Openroast/issues/73?email_source=notifications&email_token=AABH5SGHO5D7Z67F3EJHLIDQRSZHNA5CNFSM4FMKFJV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEC4MHFA#issuecomment-548979604, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABH5SFFAUGAI7A7HZA7DVTQRSZHNANCNFSM4FMKFJVQ .

-- Thomas Rankin

trankin commented 4 years ago

I've got a windows build successfully created. 1.2.1.1 here https://github.com/trankin/Openroast/releases

tgdnt commented 4 years ago

Hi all, I forked openroast and freshroastsr700 to apply trankin's patch and provide installation instructions for Linux. I'm using it successfully on Archlinux. @trankin thank you for providing the fix, I hope you'll find I have credited you appropriately. Here it is https://github.com/tgdnt/openroast

mattmartinez2006 commented 3 years ago

@trankin @tgdnt I wasn't able to use the default driver from the pull for my Mac 10.13. I had to download a newer version on the chipset driver for the laptop to connect. Do you think the newer versions of the sr700 models have a different proxy protocol for the heat that isn't allowing the roaster to cycle through low, medium, high? I'm not that familiar with the code nor the packet transfer back and forth.