GregValiant

Results 664 comments of GregValiant

[SkewTest.zip](https://github.com/user-attachments/files/20427594/SkewTest.zip) "Exception = name 'InstanceContainer' is not defined" I just finished replacing the battery in my 2017 GMC Acadia. What a friggin' job. 3 hours and $275 and used about...

Ghostkeeper, I'll think on it some more and see if I can address some of the programming concerns. From a mechanical perspective I can see such a feature as being...

I would find that acceptable. In the case of cylinders, there could be a cutoff once a minimum distance was reached between the "spokes" of infill. This is not an...

The ellipses are closer to what I had in mind. In the case of circles and ellipses any point on the exterior is "in view" of the center point of...

Most of these changes were requested by #18766 while other are because I'm unable to keep my fingers out of it. The changes have missed 5.7.0 so I will move...

I see no reason why Remco and Casper have to be the only ones horrified by my coding style.

Thanks for looking anyway. I'll make changes locally until a full review. Maybe I can get by with just a single additional commit.

Per a suggestion in #20509 I added the profile name to the other settings being inserted at the beginning of the gcode.

After working on PrintSkew and seeing that different printer names can be involved I changed this script to alter the Cura inserted "TARGET_MACHINE.NAME" line. Commit 17 example of the new...

@HellAholic I remembered seeing a PR for "Electricity Cost" (#14952 ). DisplayInfoOnLCD was already adding some stats so I threw in "Electricity Cost" as well. If you approve this then...