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

[BUG] Status text not shown on Ender 3 V2 DWIN MarlinUI #23988

Closed
The-EG opened this issue Apr 2, 2022 · 13 comments
Closed

[BUG] Status text not shown on Ender 3 V2 DWIN MarlinUI #23988

The-EG opened this issue Apr 2, 2022 · 13 comments
Labels

Comments

@The-EG
Copy link
Contributor

The-EG commented Apr 2, 2022

Did you test the latest bugfix-2.0.x code?

Yes, and the problem still exists.

Bug Description

The status text is no longer shown on the status screen when using a genuine 'DWIN' screen shipped with the Ender 3 v2 using MarlinUI. When the status changes, it flashes briefly and then disappears. The status screen also flickers a bit.

Bug Timeline

4953946 from #23949

Expected behavior

The status text to be shown properly

Actual behavior

The status text is no longer shown on the status screen when using a genuine 'DWIN' screen shipped with the Ender 3 v2 using MarlinUI. When the status changes, it flashes briefly and then disappears. The status screen also flickers a bit.

Reverting 4953946 results in expected behavior and no flickering.

Steps to Reproduce

  1. Use appropriate MarlinUI config on an ender 3 v2 with a genuine DWIN LCD with encoder.
  2. Flash
  3. Boot up
  4. Observe status screen
  5. (Optional) Send additional status messages, note not shown on LCD

Version of Marlin Firmware

Bugfix-2.0.x as of 2022-04-02 (d98c61c)

Printer model

Ender 3 v2

Electronics

4.2.2 board, DWIN LCD

Add-ons

No response

Bed Leveling

UBL Bilinear mesh

Your Slicer

No response

Host Software

No response

Additional information & file uploads

ender3v2.zip

@tome9111991
Copy link
Contributor

tome9111991 commented Apr 2, 2022

I have same issue.

And also
Always refreshing/blinking text if i'm scrolling.
https://youtube.com/shorts/iNm_I5J0zCc?feature=share

I use skr e3 turbo board
With latest bugfix verson from now.

EDIT:
I can confirm issue is gone after revert 4953946

@thisiskeithb
Copy link
Member

@InsanityAutomation: Do you have an (original) DWIN screen and can you take a look at this? Your Ender-3 S1 PR seems have broken some things #23949

@thisiskeithb
Copy link
Member

Always refreshing/blinking text if i'm scrolling.

I pinged InsanityAutomation on Discord and he confirmed the UI/screen flashing bug, but couldn't reproduce the status missing text screen bug.

Do these screens have revisions printed on them by chance? Perhaps it's a difference of hardware/DWIN features.

@thisiskeithb
Copy link
Member

@The-EG: Can you upload a video showing the status screen bug?

Do these screens have revisions printed on them by chance?

To answer my own question, I found my E3V2 DWIN screen and couldn’t find any real markings to indicate board revisions. I’ve been busy IRL, but still plan to test this on my end.

@thisiskeithb thisiskeithb added Needs: More Data We need more data in order to proceed and removed Bug: Confirmed ! labels Apr 6, 2022
@tome9111991
Copy link
Contributor

If i insert a SD card i see the message "SD insert" for 1 sec then is status text clean.

Before the bug
I insert a SD card and i can the the message "SD insert" all the time.

@The-EG
Copy link
Contributor Author

The-EG commented Apr 7, 2022

2 videos, current bugfix and with 4953946 reverted:

  • turned printer on
  • waited for boot up
  • inserted SD card
  • removed SD card

Current bugfix behavior:
https://youtu.be/mOol4y_RkEc
image

It's worth pointing out: either I didn't notice how bad this was before or it's gotten worse in the past few days. Not only is the status text missing, some other parts of the status screen are too. Even the images shown during bootup are different.

After reverting 4953946:
https://youtu.be/O7h0h0sIxSE
image

@thisiskeithb thisiskeithb added Bug: Confirmed ! Fix Included A fix is included in the description and removed Needs: More Data We need more data in order to proceed labels Apr 7, 2022
@thisiskeithb
Copy link
Member

To fix in 4953946 Remove all DWIN_UpdateLCD(); in PR.

I assume that'll break non-DWIN screens. @InsanityAutomation can you test this fix on your end?

@InsanityAutomation
Copy link
Contributor

I'll probably have some more tweaks this weekend. They're not all required, some were insurance. Looks like the above may be triggered if status scrolling is disabled, so enabling that is a workaround for the moment.

@thisiskeithb
Copy link
Member

This should be fixed in #24013

@The-EG
Copy link
Contributor Author

The-EG commented Apr 23, 2022

Sorry for the delay, #24013 does fix this issue.

@tome9111991
Copy link
Contributor

tome9111991 commented Apr 29, 2022

i can confirm fix works good.

only have in some menus kind of color overlay.

IMG_20220429_140357_1

IMG_20220429_141507_1_1

@github-actions
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 Jun 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

5 participants
@The-EG @thisiskeithb @InsanityAutomation @tome9111991 and others