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

klipper-preprocessor not working with Smart Avionics Cura Master Branch #2

Closed
PhilBaz opened this issue Feb 24, 2023 · 9 comments
Closed
Labels
bug Something isn't working

Comments

@PhilBaz
Copy link

PhilBaz commented Feb 24, 2023

What happened

The script does not appear to have any effect when used with smart avionics fork of Cura. I tend to prefer his fork for its rock solid reliability and extra features.

https://github.com/smartavionics/Cura

What did you expect to happen

For the script to work as in Cura 5.2/3.

How to reproduce

Configure script and slice.

Additional information

Ill make a corresponding report at his git.

Thanks Pedro.

@pedrolamas
Copy link
Owner

Can you please upload a sample gcode file sliced on this slicer so I can look and see what is going on?

I'm happy to fix this if it is a simple change that doesn't conflict with mainstream Cura, but I'm not very inclined to support unknown Cura forks as these can diverge a lot from what Cura provides.

@PhilBaz
Copy link
Author

PhilBaz commented Feb 24, 2023

@pedrolamas

Fair enough. Smart Avionics fork is a fairly significant fork. He is a very active developer and very responsive to community feedback with issues. I would imagine his fork is the most actively used of any. He has developed many features that are excellent, his version is also totally rock solid compared to the UM 5 version.

I've included an Image of the settings and a sliced file.

gjhgjhg
Untitled test.zip

@pedrolamas
Copy link
Owner

Well, Layer info comments are definitely on the file, so the problem is something else!

I recommend you take a look at cura.log (Help -> Show Configuration Folder, and search for cura.log) and see if there is any error related to klipper-preprocessor.

Feel free to upload the log if you want me to take a look to!

@PhilBaz
Copy link
Author

PhilBaz commented Feb 24, 2023

cura.log.1.zip

I searched for ' klipper-preprocessor', but didn't get a hit. I'm not sure what else to look for. I attached the log.

Thanks Pedro.

@pedrolamas
Copy link
Owner

This is the most recent entry on the log for scrips loading:

image

There is no indication there that KlipperPreprocessor was loaded (in fact, no reference anywhere on the log when I search for "KlipperPreprocessor")

cura.log.1 is rolled log, you should also have a cura.log which is the one we need here!

@PhilBaz
Copy link
Author

PhilBaz commented Feb 24, 2023

Ahh. here it is. I just sliced it again and saw the file date update immediately so hopefully something relevant should be in there.
cura.zip

@pedrolamas
Copy link
Owner

Here it is:

image

I have no idea where that %1 bit came from...

@pedrolamas
Copy link
Owner

What did you set for "Path to klipper_estimator"?

@PhilBaz
Copy link
Author

PhilBaz commented Feb 24, 2023

@pedrolamas

I put the .exe in the script folder and set the path to it there. Ill just double check, but i think its done properly. Same config in 5.3 worked. Ill check.

.......................

Ahhhh. The path had a typo!!! I'm so sorry Pedro. My fault entirely. I thought I double checked it.

Sorry! Thank you for your help.

@PhilBaz PhilBaz closed this as completed Feb 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants