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

Abort nozzle change if filament is loaded #4758

Merged
merged 1 commit into from
Aug 23, 2024

Conversation

sarusani
Copy link
Contributor

Abort Nozzle change if filament sensor is enabled and filament is loaded.

Resolves #4703

Copy link

github-actions bot commented Aug 20, 2024

All values in bytes. Δ Delta to base

Target ΔFlash ΔSRAM Used Flash Used SRAM Free Flash Free SRAM
MK3S_MULTILANG 30 0 247930 5653 6022 2539
MK3_MULTILANG 90 0 247288 5662 6664 2530

@sarusani
Copy link
Contributor Author

Currently I check for both original and revo nozzles. Is unloading required for a revo nozzle change or should I only check for original?

Firmware/ultralcd.cpp Outdated Show resolved Hide resolved
@sarusani sarusani force-pushed the CheckFSBeforeNozzleChange branch 2 times, most recently from e6e3731 to 6f28608 Compare August 23, 2024 08:10
Abort Nozzle change if fsensor is enabled and filament is present.
@3d-gussner 3d-gussner added this to the FW 3.14.1 milestone Aug 23, 2024
Copy link
Collaborator

@3d-gussner 3d-gussner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on MK404

  • MK3S without filament loaded, behaves the same as before
  • MK3S with filament loaded, shows Unload message and exits
  • MK3S + MMU3 without filament loaded, behaves the same as before
  • Mk3S + MMU3 with filament loaded, shows Unload message and exits

@3d-gussner 3d-gussner merged commit 7d66cd1 into prusa3d:MK3 Aug 23, 2024
4 checks passed
@sarusani sarusani deleted the CheckFSBeforeNozzleChange branch August 23, 2024 09:20
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

Successfully merging this pull request may close these issues.

FIrmware should check filament sensor before nozzle change
3 participants