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

[Q] Can we now officially remove the M109 / M190 redirections? #1974

Closed
Tannoo opened this issue Jun 5, 2021 · 7 comments · Fixed by #2156
Closed

[Q] Can we now officially remove the M109 / M190 redirections? #1974

Tannoo opened this issue Jun 5, 2021 · 7 comments · Fixed by #2156
Labels
question Further information is requested

Comments

@Tannoo
Copy link

Tannoo commented Jun 5, 2021

I have seen that the M109 / M190 have been redirected to M104 / M140 respectively.

The original issue of not waiting for that process to finish seems to have been corrected.

I, personally, have removed the redirection for now by commenting out these lines:
https://github.com/bigtreetech/BIGTREETECH-TouchScreenFirmware/blob/master/TFT/src/User/API/interfaceCmd.c#L638
and
https://github.com/bigtreetech/BIGTREETECH-TouchScreenFirmware/blob/master/TFT/src/User/API/interfaceCmd.c#L708

Testing it seems to be running just fine and temperature LED transitions work just fine.
Can we revert this redirection? Or is there another fix/angle being worked on still?

@Tannoo Tannoo added the question Further information is requested label Jun 5, 2021
@oldman4U
Copy link
Contributor

oldman4U commented Jun 7, 2021

As far as I know BTT would like to solve this problem using another approach. We will see soon.

@Tannoo
Copy link
Author

Tannoo commented Jun 10, 2021

I believe is was fixed with PR #402 but the "re-direction" was not removed.

@discip
Copy link
Contributor

discip commented Jul 27, 2021

I'll second the official removal of this redirection, since there is no way to use custom code including M109 / M190 at the moment.

regards

@Tannoo
Copy link
Author

Tannoo commented Aug 18, 2021

I have noticed that the transitions work just fine as long as the printer doesn't sit idle for too long.
After sitting for some unknown time, another print can be started just fine, but the LED transition does not work.

I have found that if:

  1. I start a print and the LEDs do not respond. I cancel the print while the bed is heating, then once it stops, the transitions and LED "automatic" controls work just fine when the print is restarted.
  2. If I reset the printer via the TFT E-Stop button first, then print is started, the transitions and LED "automatic" controls work just fine.
  3. After some time (Amount of time is random so far due to life, but greater than an hour.) Used TFT to set the LEDs to a color, then set them off. I start a print and the LEDs do not respond.
  4. After some time (same as # 3 above), I set the LEDs to WHITE for printing. I then start a print and the transitions does not work and the print continues without a hitch. Once the print is finished, the LEDs remain WHITE and do not turn GREEN. I use the TFT to set the LEDs off.

So far, I was off on a trip out of town over the last week. I left the printer on. When I got back today, I started a print and the LEDs did not respond as predicted. I did the same as # 1 above, and they were working just fine after that.

I have not found anything other than serial issues when searching for the reasoning of these "bypasses". There is still an issue and I am trying to describe them the best as I can as I try to get this figured out.

digant73 added a commit to digant73/BIGTREETECH-TouchScreenFirmware that referenced this issue Sep 3, 2021
@oldman4U
Copy link
Contributor

oldman4U commented Sep 7, 2021

Thank you

@discip
Copy link
Contributor

discip commented Sep 7, 2021

@digant73

Thank you Sir! 😃👍

Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
question Further information is requested
Projects
None yet
3 participants