-
-
Notifications
You must be signed in to change notification settings - Fork 25
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
Object of type SettingFunction is not JSON serializable #55
Comments
I can not reproduce your error with cura 5.2-beta - please attach your cura.log |
This error is triggered by a profile with a setting reset to its computed value: change a setting vlue then reset it with the I've pinged the authors on the PR that introduced it, but not response so far. |
Indeed, when I upload as gcode, the error is gone. Feel free to close this issue! For reference, the relevant log is as follows:
|
same here! |
There was a change/commit within UFPWriter which seems to fix this error -> Ultimaker/Cura@4e785c3 |
Have you tried with a computed setting? For example try setting an arbitrary value for "Minimum Wall Line Width", then reset by clicking the icon. The |
I've tried setting all values to their computed value in one of my profiles, but it makes no difference. Error remains. |
You'll have to use a stock "machine profile", not a user created profile. And then click "Discard current changes" in the profile menu. |
Yup, that works. |
Sorry, Im not following. Is this a bug? Is there a fix? |
I replaced this .py with the linked version. problem persists.
|
Ok... can reproduce it now. It seems to be a bug within Cura (UFP Writer) itself. I think there is something wrong with the handling of "older" custom/inherited profiles - not all these profiles are causing this error. |
I just copied the UFPwriter folder from Cura 5.1 to 5.2 beta and it seems to be working fine, for now. Is this a bad idea? C:\Program Files\Ultimaker Cura 5.2.0-beta\share\cura\plugins |
I tested this but I don't think it is using your settings, my speed and temps were not correct in the g code.
|
@PhilBaz In general yes, but it might work in this case since there is only a few changes in this file between 5.1.1 and 5.2.0-beta. |
@Piezoid Thanks for this patch. It seems to fix the issue. You can also rebuild the custom profiles and that seems to fix it as well.
|
@Piezoid @emtrax-ltd Looks good! Thanks guys. |
Getting this error when using your plugin version 1.6.3 on Cura 5.2.0 Beta to upload to printer. Save to gcode works fine.
The text was updated successfully, but these errors were encountered: