-
Notifications
You must be signed in to change notification settings - Fork 311
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
Question: Loop re-ordering. #43
Comments
@GilesBathgate it does not reorder the loop walls. It will print them in the normal order as the gcode dictates, only adjusting the Z on every other wall loop. EDIT: bricklayersReorderedGcode.py.txt remove the .txt |
@xboxhacker Thanks for your script. I get an error when I try to view in the prusaslicer gcode viewer:
|
@xboxhacker I was able to get it to display in the gcodeviewer:
Unfourtunatley the gcode isn't right. I will try tweaking some settings. |
@GilesBathgate Sorry, I totally forgot to tell you, this is also a standalone script. Just double-click on the file to open a GUI. then select the gcode you want to modify. It works outside of the slicer. Also, as I mentioned it was tested with Orca only. I will make a few changes, let's see. Try this: Added the config block for prusa. The first layer seems to be messed up, that will take some looking into. |
@xboxhacker That new version works great W.R.T prusa config block. I look forward to seeing the first layer fixes version. |
@TengerTechnologies Is it worth making this a feature request? It seems to me that when printing the perimeter numbered 6 in the CNC kitchen image above, under the current implementation this would be trying to print partially under the perimeter labeled 7. @xboxhacker Has provided the basic implementation to get this working in an arugably better order. It just needs some finalisation and converting back from a standalone script. |
@GilesBathgate So after some more testing, this mod is great for straight vertical walls. Not so great for complex curvy objects. What we really need is “slicer level” brick layering…. Here is my latest. bricklayersCMD.py.txt The command line should work in the slicer as a post processor. It will not output a log txt file unless you use the flag -log Only tested with Orca slicer, not sure if it will work with prusa yet. Full transparnecy, Copilot AI helped with this. |
@xboxhacker Thanks I will give these a try.
|
@xboxhacker It seems like CoPilot AI removed the perimeter order modifications! 👎 😆 |
@GilesBathgate I had to remove it. As I said, this reorder was great for straight vertical walls. Not so great for complex curvy objects, or variable wall thicknesses. It really messed things up for my slices. |
@xboxhacker I don't understand the point of your scripts then, in the context of my original question. |
@GilesBathgate Incremental implementation. I had to start over. These new versions fix gaps in the outer walls when printing inner/outer wall order. |
@xboxhacker Got it, lol. It was unexpected to me for you to go back to a version that works the same as the original and publish it for testing. However I am happy to test anything, and I am eagerly looking forward the re-ordering feature. |
Hi I use PrusaSlicer under linux. The script seems to work, but I am confused about the order of the loops. I expected the script to modify the loops such that it prints the odd perimeters first, and then raizes Z and prints the even perimeters. i.e from internal to external with the ascii-art below first printing 0, 2, 4, and 6, then raizing Z and printing 1, 3, and 5.
Reloading the gocde into Prusa gcodeviewer seems to show them print them in the order 0,1,2,3,4,5,6. Meaning that perimeter 2 has to undercut the already layed down perimeter 1?
Hope this makes sense. Probably I am just misunderstanding something.
Thanks, Giles
The text was updated successfully, but these errors were encountered: