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

I think new pause @ layer isn't working properly? #1283

Closed
PhilBaz opened this issue Feb 22, 2023 · 17 comments · Fixed by #1291
Closed

I think new pause @ layer isn't working properly? #1283

PhilBaz opened this issue Feb 22, 2023 · 17 comments · Fixed by #1291
Labels
⚡ Type: Bug Something isn't working ❔ User Input Further information is requested

Comments

@PhilBaz
Copy link

PhilBaz commented Feb 22, 2023

Mainsail Version:

2.5 beta

Browser:

Chrome

Device:

Desktop PC

Operating System:

Windows

What happened?

I started with a comment here. Sorry if that's the wrong place to comment.
#1230

When I click the top input field (that's where I expect to enter the layer?) I get a box that says no data. I cant enter anything into it.

first
fdgdgfd
then
fdgdgf

What did you expect to happen instead?

Have the ability to set a layer number there.

How to reproduce this bug?

Click on field.

Additional information:

2 thoughts.

  1. I have custom pause macros for planed mid print pause, lots of beeping etc. I would love to be able to use those with this function.

  2. It would be amazing to have the option to pause the print at infill rather than the beginning of a new layer. To minimize the very likely possibility of a surface artifact. I use Cura, I know it identifies infill in the Gcode. That would make this amazing.

@PhilBaz PhilBaz added the ⚡ Type: Bug Something isn't working label Feb 22, 2023
@meteyou
Copy link
Member

meteyou commented Feb 23, 2023

Could you let me know if you updated your mainsail-config? The button for this feature should not be displayed if you don't have the new gcodes for it.

Just check the gcode. There is no possibility to have fancy special pause places or not because the moment of the layer change comes from your slicer. it just interrupts the layer change gcode. nothing more.

@zellneralex
Copy link
Member

zellneralex commented Feb 23, 2023

First of all did you follow the instructions mentioned here https://github.com/mainsail-crew/mainsail-config#new-feature-pause-at-next-layer-and-pause-at-layer-.

For your 1st question: you should be able to use your own Pause but you need the 3 macros of the mainsail.cfg
For your 2nd question: we will be happy to integrate it, if you find a way that Klipper sends a message when infill starts

And as @meteyou ask do you have the latest mainsail.cfg (could be found in the linked repo) and also did you include it?

@zellneralex zellneralex added the ❔ User Input Further information is requested label Feb 23, 2023
@PhilBaz
Copy link
Author

PhilBaz commented Feb 23, 2023

@meteyou

No, I didn't update my mainsail.cfg. and I can see the button. Sorry I didn't realize changes were needed. I clicked on the feature link in the release message but didn't see that I needed to update.

I understand. I though maybe the gcode would also indicate the printing of infill and the pause could be delayed until there. I know Cura creates comments in the gcode when a wall vs skin vs infill is printing. But, of course don't know how to code or the limits of what can be implemented.

@zellneralex
Copy link
Member

@PhilBaz that you see the button is a bug. Follow the link I posted there you see the needed slicer changes.

The problem is that cura does comment and PrusaSlicer does other comments and … does other comment and so one.
What we did was enhancing a feature that we added to get a more accurate layer numbering.

@PhilBaz
Copy link
Author

PhilBaz commented Feb 23, 2023

@zellneralex

No I didn't. How was I supposed to know about those instructions?

If they were mentioned here
fghfhf

Or here
gfhgfhgf

I would have.

For your 1st question: you should be able to use your own Pause but you need the 3 macros of the mainsail.cfg

Awesome, thanks.

For your 2nd question: we will be happy to integrate it, if you find a way that Klipper sends a message when infill starts

I thought this might be a possibility as Cura creates comments in the gcode when a wall vs skin vs infill is printing. I don't know how to code or the limits of what can be implemented. But this would be amazing. If its not possible I wouldn't know that. I just like 3d printers, and its a good idea.

@zellneralex
Copy link
Member

For cura you could also use https://github.com/pedrolamas/klipper-preprocessor to get the macros in. As said cura is one of many …

@zellneralex
Copy link
Member

That is a beta release we need to get a better documentation for the release.

@PhilBaz
Copy link
Author

PhilBaz commented Feb 23, 2023

@zellneralex

@PhilBaz that you see the button is a bug. Follow the link I posted there you see the needed slicer changes.

The problem is that cura does comment and PrusaSlicer does other comments and … does other comment and so one. What we did was enhancing a feature that we added to get a more accurate layer numbering.

Thank you, ill do that.

Yes, I get what you are saying. Its to bad that is not standardized in slicer gcode. Its would be very useful. That being said with Cura and Prusa slice they are the two most popular. Maybe there is a solution that works for those two?

Yes, pause @ layer is awesome. And don't mean to sound ungrateful. I am! I wait for new releases and am always excited for the next one. And the problem is not with your feature of course but the nature of pause, and its creating artifacts at the point it resumes. The feature is fantastic.

@PhilBaz
Copy link
Author

PhilBaz commented Feb 23, 2023

@zellneralex

That is a beta release we need to get a better documentation for the release.

No problem. I just had no clue to do it.

Hey, at least a genuine bug was discovered in a round about way.

@PhilBaz
Copy link
Author

PhilBaz commented Feb 24, 2023

@zellneralex @meteyou

For your 1st question: you should be able to use your own Pause but you need the 3 macros of the mainsail.cfg For your 2nd question: we will be happy to integrate it, if you find a way that Klipper sends a message when infill starts

Hi Alex,

I mentioned this idea to Pedro the developer of the klipper preprocessor for Cura. He suggested how it might work. I think he proposes what you are requesting above.

pedrolamas/klipper-preprocessor#1

I'm not a programmer, just a keen user of 3d software and 3d printers. Is there anything I can do to advance this idea? I guess someone on the klipper side would need to create the new parameter that Pedro purposes.

@zellneralex
Copy link
Member

zellneralex commented Feb 24, 2023

Yes he suggested a change that someone needs to do. Any programmer can do that, the original implementation on the Klipper side is from me. The hard part will be to convince Kevin (The main Klipper developer) to merge that

But honestly why you want the pause at infill, I see that pause for color changes or the insertion of nuts or magnets and all that will work at a layer start. So what is the reason to pause at infill?

@PhilBaz
Copy link
Author

PhilBaz commented Feb 24, 2023

@zellneralex

To prevent surface artifacts. Pause always has some small imperfection with flow, due to the retract, de-retract, and some ooze.

If pause comes back to an outer wall you will see a flaw in that spot. If it comes back to infill there will be no flaw.

I will mainly use pause to swap out rolls that are running out. I print a lot so this is a regular need. For me I think there is real value in this idea as it will eliminate surface flaws that often result from a pause for a filament swap.

@zellneralex
Copy link
Member

I see, as said the hard part will be to convince Kevin. And we would also need to enhance the moonraker post processor to get that feature available for more slicer .

@PhilBaz
Copy link
Author

PhilBaz commented Feb 24, 2023

@zellneralex

How can I help?

@PhilBaz
Copy link
Author

PhilBaz commented Mar 1, 2023

@zellneralex

Hi Alex,

I made a post on the Klipper forum in hopes that Kevin O'Conner will consider implementing the parameter Pedro proposed that would allow for pause at infill.

Here is the post.

klipper.discourse.group/t/fr-new-parameter-on-set-print-stats-for-section-type-infill-wall-skin-etc/6943

Ill send kevin a message through the form asking him to look at the post.

I do think having Klipper be aware of what type of feature it is printing may have additional benefits beyond delaying a Pause until infill.

Anyhow, I hope I did not overstate your willingness to entertain the idea of implementing this into Mainsail.

@zellneralex
Copy link
Member

@PhilBaz we will see, as said cura is only a small part of it. To call it a feature it need to be solvable at the common slicer. I would say the same that support exclude object (SuperSlicer, PrusaSlicer, Slic3r, Cura, IdeaMaker) would be a good start.
And that’s the hard part here, analyzing there code output see if the add there Type as comment and than do a preprocessor for them or better integrate that all in the prepprocessor used by moonraker for exclude object.
So you see what I mean you may be only interested at a solution insight cura but to get the thing „flying“ we need to look at the bigger picture.
I personally have no interest in investing my spare time in that right know. There is more important stuff to solve on my plate atm.
Using that then in the macros of the 2 Pause functions should not be a big deal at all if we get it in.

But we need to do that after V2.5 as an FR as the real bug is that you should not have seen the feature at all as your Goode does not uses the needed macros yet.

@PhilBaz
Copy link
Author

PhilBaz commented Mar 1, 2023

@zellneralex

Hi Alex,

I hear exactly what you are saying. Its a slicer issue too.

I'm happy to help with the research and analyze/characterize the outputs from the different slicers. If and when there is a possibility of having development of this functionality go forward.

I'm a researcher and I'm good with a spreadsheet.

Also, Pedrolamas said he would integrate the functionality into his preprocessor. I don't know if that saves any work. Maybe his work will also be suitable for the moonraker preprocessor?

I guess we will see if Kevin is receptive first.

Anyhow, I hope I'm not misreading your Intention. I'm hearing.... you might be willing to entertain the development of this feature in the future but, a) You don't want to be responsible for all the footwork on slicers and moonraker b) The parameter must be implemented in Klipper for this to work. Is that right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚡ Type: Bug Something isn't working ❔ User Input Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants