-
-
Notifications
You must be signed in to change notification settings - Fork 731
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
Sequre T55 Support #1928
Sequre T55 Support #1928
Conversation
As already commented in #1885: LGTM |
Thanks for testing, can you list some examples of what is cut off ? |
Perhaps this is because the full OLED resolution is not yet supported |
Just got one of these and tested the firmware. It's possible this issue is something broken on my end when building (the CI artifacts have expired so I can't test an official build for this), but the localization strings for the profile support seem to be missing, resulting in a blank menu item for all of those settings. When I deleted all the |
@ReimuHakurei @StevenSeifried I believe the cut off should be fixed 😓 |
I compiled the firmware with I'll test the latest CI artifact this evening and see if it works properly there. |
@Ralim the cut off seems to be fixed. I tested with the current version from GitHub actions (German and English). I didn't click through all the settings and all the descriptions in both languages, just checked a few, but none of them was cut off |
@ReimuHakurei yeah that would make sense. I advise to do all commands inside the docker container to ensure we are all using the same tooling. Can you check it's working fine in the CI builds to compare against ? |
Can confirm it works correctly in the CI build. |
Another observation: I assume this is an issue in the dev branch and is just showing up here from the rebase earlier, but while it still seems to enter profile mode when you hold the button, it is no longer displaying details (ie: time elapsed / remaining / current phase / target temperature). This is with detailed idle/soldering view set; not sure how it's supposed to look otherwise as I always use the detailed views. |
This is most likely something missing in the drawing modes for the higher res oled screen. I'll try and have a look tonight |
Phase printing should work now I think |
I just got T55 today and found this, hows the firmware coming along? |
Only @Ralim can tell it excactly, but I'm optimistic that the T55 will be officially supported with 2.23 |
Cool |
Hardware should work, just the UI is limited |
Adding support for the T55 hotplate.
Closes #1885.
What is the new behavior (if this is a feature change)?
Other information:
Like all Sequre devices, the unit is designed to minimum cost. So there is no inductor or enough caps to filter out noise.
We get some PWM hum when running it as fast as we can to keep current regulation smooth.