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

[Feature Request]: Option to only power off if a printer is connected to Octoprint #268

Open
jalabb opened this issue Apr 26, 2021 · 4 comments
Labels
enhancement New feature or request

Comments

@jalabb
Copy link

jalabb commented Apr 26, 2021

Is your feature request related to a problem? Please describe.
I was printing from the USB stick, not using octoprint. The plugin thought the printer was idle and turned off the plug, ruining my print (the Prusa Mini doesn't resume after a power loss).

Describe the solution you'd like
A checkbox with something like "Only power off if a printer is connected" -> see attached mockup

image

EDIT : USB stick not SD card

@jalabb jalabb added the enhancement New feature or request label Apr 26, 2021
@jneilliii
Copy link
Owner

I'm sorry this happened to you, I don't print from SD card and you're actually the first person to bring this up. I never thought about that scenario, but was curious if you started the print from the printer's controller directly or from OctoPrint? I'm fairly certain that OctoPrint knows when printing from SD.

@jalabb
Copy link
Author

jalabb commented Apr 26, 2021

No problem, it was just a Benchy ^^

I was doing some sanity checking and wanted to cut Octoprint out of the equation completely so I disconnected the printer from Octoprint and I started the print from the printer. Since the plugin had no input from the printer, after the idle timeout ran out it did its job and cut the power, 30 minutes into a 1h30 print.

@jneilliii
Copy link
Owner

ok, that makes sense then. that was one of the reasons I added the toggle switch in the sidebar for the plugin, to easily disable/enable the idle timeout for maintenance tasks, etc. but it definitely is something I can add. I think the settings UI needs a major overhaul though, it's starting to get to be too busy I think and need to split things up a little better.

@jalabb
Copy link
Author

jalabb commented Apr 26, 2021

Thanks for looking into it :)

toggle switch in the sidebar

Silly me, I'll think about it next time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants