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

Roast Temp always on high #73

Open
jarrettelliott opened this issue Jul 26, 2018 · 17 comments
Open

Roast Temp always on high #73

jarrettelliott opened this issue Jul 26, 2018 · 17 comments

Comments

@jarrettelliott
Copy link

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

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

trankin commented Aug 23, 2018

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

trankin commented Aug 24, 2018

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

trankin commented Aug 28, 2018

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

trankin commented Aug 28, 2018

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

@trankin
Copy link

trankin commented Aug 29, 2018

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

msilvey commented May 8, 2019

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

@pgolay
Copy link

pgolay commented Aug 17, 2019

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

daibew commented Nov 1, 2019

Any update on @trankin solution getting integrated in?

@trankin
Copy link

trankin commented Nov 1, 2019 via email

@daibew
Copy link

daibew commented Nov 1, 2019

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

@trankin
Copy link

trankin commented Nov 1, 2019 via email

@trankin
Copy link

trankin commented Nov 1, 2019 via email

@trankin
Copy link

trankin commented Nov 2, 2019 via email

@trankin
Copy link

trankin commented Nov 19, 2019

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

@tgdnt
Copy link

tgdnt commented Oct 14, 2020

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

mattmartinez2006 commented Jan 10, 2021

@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.

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

8 participants