Skip to content

Changing spool Mid-print while paused leads to undefined behavior #183

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

Closed
codingcatgirl opened this issue Jun 6, 2021 · 5 comments
Closed

Comments

@codingcatgirl
Copy link
Contributor

When changing a spool mid-print, SpoolManager asks whether you want the usage so far to be commited to the previously selected spool.

It works well – unless you do it while the print is paused. For some reason, then, it changes the spool just like that, without asking. This causes a pretty big problem when you notice that it didn't come up and klick the new spool again. When you confirm "commit to the previously selected spool" then, since the new spool was already selected, it will commit it to that spool.

So, two suggestions from me:

  • When selecting a spool mid-print, the dialog should pop up too, currently it doesn't.
  • When changing a spool mid-print without having selected the desired behavior (in the dialog), it should be rejected – right now it doesn't seem to do that if the print is paused.
  • Lastly: When selecting a "new" spool that is actually the one that is currently selected, the dialog probably shouldn't pop up.
@github-actions
Copy link

github-actions bot commented Feb 7, 2022

This issue has been automatically marked for closing, because it has not had activity in 30 days. It will be closed if no further activity occurs in 10 days.

@github-actions github-actions bot added the status: markedForAutoClose Issue will be closed automatically label Feb 7, 2022
@dimo414
Copy link

dimo414 commented Feb 7, 2022

I've observed this issue as well. Maybe it's since been fixed, but I don't think this issue should be auto-closed.

@github-actions github-actions bot removed the status: markedForAutoClose Issue will be closed automatically label Feb 8, 2022
@OllisGit
Copy link
Owner

There was a bugfix (#217) in version https://github.com/OllisGit/OctoPrint-SpoolManager/releases/tag/1.6.0
No dialog, if the printer is in pause-mode the currently extruded amount is subtracted from the spool.

SO, from my option this issue is fixed. If not, please report with more details (e.g. which version do you use)

@github-actions
Copy link

This issue has been automatically marked for closing, because it has not had activity in 30 days. It will be closed if no further activity occurs in 10 days.

@github-actions github-actions bot added the status: markedForAutoClose Issue will be closed automatically label Mar 19, 2022
@github-actions
Copy link

This issue was closed, because it has been already marked for 10 days with no activity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants