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] Crash after removing SD card #894

Closed
qwazwsx opened this issue May 1, 2023 · 7 comments
Closed

[BUG] Crash after removing SD card #894

qwazwsx opened this issue May 1, 2023 · 7 comments
Labels
bug: Confirmed Something isn't working

Comments

@qwazwsx
Copy link

qwazwsx commented May 1, 2023

Did you test with a precompiled firmware?

Yes, and the problem still exists.

Bug Description

Version: Latest latest Release #20230312. Also confirmed on: #20221222

Firmware crashes (freezes and shows version popup) when selecting the "media" tab after printing and removing the SD card.

Bug Timeline

No response

Expected behavior

Expected Behavior
When media is removed, the selected file should be reset to the top position ("Back").

Actual behavior

The selection indicator remains in the same position as the last selected file, even though there are no files present. Moving the scroll wheel selects the "Back" entry, and promptly freezes. After a few seconds, the firmware will restart, showing the version info.

Steps to Reproduce

Video showing the bug in action

Reproduction:

  1. Select an item in the Media menu. Either successfully print or cancel. You should be brought back to the main menu and your selection will be saved when you return to the media menu.
  2. Remove the SD card. Observe "Media Removed" M117 message
  3. Enter the Media menu. Observe that the selection from before is still 'selected' even though the menu is blank. See image above.
  4. Move the selector wheel, the selection should jump to the top ("Back") and the firmware will hang, before showing the version info popup and restarting.

Version of Professional Firmware

Professional Firmware v2.1.3 - Build Time 20230312 (Lastest GitHub release as of posting)
Ender3V2-422-BLTUBL-MPC-20230312.bin

Printer model

Ender 3v2

Electronics

No response

Add-ons

BL Touch

Bed Leveling

UBL

Your Slicer

None

Host Software

None

Additional information & file uploads

No response

@Majboor
Copy link

Majboor commented May 1, 2023

xpected behavior should be that when the media is removed, the selection should be set to the top item "Back" in the media menu, and the application should not crash?

@qwazwsx
Copy link
Author

qwazwsx commented May 1, 2023

Just tested on the latest Release #20230312 and can confirm that I am still experiencing this bug. I'll update the issue to reflect that.

@Majboor Yes. Opening the media menu after printing and removing the SD card causes a crash.

I've recorded a quick (and quite shaky, my apologies) demonstration video of the bug in effect on my Ender 3 v2. You can view that here: https://streamable.com/aafdwb

Here's what the bug looks like. Moving the scroll wheel at this point causes the printer to restart.
image

I've added this information, and clarified a few things, in the original post.

@mriscoc mriscoc added the bug: Confirmed Something isn't working label May 1, 2023
mriscoc added a commit that referenced this issue May 22, 2023
- Runout icon indicator over the Extruder flow dashboard icon.
- Dashboard Speed shows the current axes speed in mm/s intermittently with the speed percentage
- Enhanced preview with arbitrary thumbnail size
- Preview QR code removed for improve support with some display units
- Improved 3D/BL/CRTouch auto leveling
- Enable BLTouch HS mode for bed tramming wizard
- Set Multiple probing to 0 as default
- Better limits for mesh insets editing
- New C35 custom g-code for launch the bed tramming wizard.
- Added toolbar option for park head.
- Force axes homing to allow parking at pause
- Raise Z when printing is aborted
- Fix a possible bug with filename drawing in printing page
- Fix Crash after removing SD card #894
- Several minor bug fixes and Misc. optimizations
- and many more fixes from latest Marlin's bugfix 2.1.x

In source:
- TJC support to enable preview and grid mesh viewer
- Added Creality CV laser module support with automatic file type detection
- Added Creality Cloud Support
- Menu items value faster/smooth change rate
- More #define flags to disable features to help to reduce code size
- Many improvements from the latest Marlin's bugfix sources
mriscoc added a commit to mriscoc/Special_Configurations that referenced this issue May 22, 2023
- Runout icon indicator over the Extruder flow dashboard icon.
- Dashboard Speed shows the current axes speed in mm/s intermittently with the speed percentage
- Enhanced preview with arbitrary thumbnail size
- Preview QR code removed for improve support with some display units
- Improved 3D/BL/CRTouch auto leveling
- Enable BLTouch HS mode for bed tramming wizard
- Set Multiple probing to 0 as default
- Better limits for mesh insets editing
- New C35 custom g-code for launch the bed tramming wizard.
- Added toolbar option for park head.
- Force axes homing to allow parking at pause
- Raise Z when printing is aborted
- Fix a possible bug with filename drawing in printing page
- Fix Crash after removing SD card mriscoc/Ender3V2S1#894
- Several minor bug fixes and Misc. optimizations
- and many more fixes from latest Marlin's bugfix 2.1.x

In source:
- TJC support to enable preview and grid mesh viewer
- Added Creality CV laser module support with automatic file type detection
- Added Creality Cloud Support
- Menu items value faster/smooth change rate
- More #define flags to disable features to help to reduce code size
- Lot of improvements from the latest Marlin's bugfix sources
@classicrocker883
Copy link

do you have SD extender installed? try enable that setting in the configuration.h file.

@mriscoc
Copy link
Owner

mriscoc commented May 25, 2023

This bug was fixed in the latest release.

@mriscoc mriscoc closed this as completed May 25, 2023
@qwazwsx
Copy link
Author

qwazwsx commented May 25, 2023

Cheers! Thanks for continuing to support this firmware

@shayenss

This comment was marked as off-topic.

@mriscoc

This comment was marked as off-topic.

Repository owner locked as resolved and limited conversation to collaborators May 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug: Confirmed Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants