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

Version upgrade 4.5 to 4.6 for Maximum Deviation changes #7247

Closed
wants to merge 3 commits into from

Conversation

BagelOrb
Copy link
Contributor

@BagelOrb BagelOrb commented Mar 9, 2020

Includes value changes for meshfix_maximum_deviation, so that its behavior remains the same.

Required for Ultimaker/CuraEngine#1214

Some profiles used to set the meshfix_maximum_deviation to an odd number of microns. I rounded them down after halving their values.

BagelOrb added 3 commits March 9, 2020 08:55
Because Cura 4.6 fixes a bug in simplify, which means the max deviation setting is interpreted 4x as high

Increments version number in profiles and fdmprinter.def.json ans fdmextruder.def.json and CuraApplication.py
Due to a bug the setting used to be interpreted half as high as they were.
Now the bug is fixed and the values are interpreted as twice what they used to be.
In order to keep the behavior the same, we therefore halve all the values of that setting
required to update values of meshfix_maximum_deviation such that the behavior is the same
@BagelOrb
Copy link
Contributor Author

BagelOrb commented Mar 9, 2020

This doesn't include any changes to the version "4.5" to "4.6". Should it?
I only need the version upgrade system to update to a new settings_version, which is from 11 to 12.

I don't think doing the whole version 4.6 should be part of this PR.

@fieldOfView
Copy link
Collaborator

Wow, 1400+ files need to change for such a small update. That puts startup performance into perspective...

@BagelOrb
Copy link
Contributor Author

BagelOrb commented Mar 9, 2020

Haha yeah. I always have the biggest PRs :D

@fieldOfView
Copy link
Collaborator

It is sort of indicative though of a larger issue that the current approach of having all those quality profiles for all those printers in the main download is not tenable. Wasn't there once a plan to host them externally (Marketplace)?

@ChrisTerBeke
Copy link

@fieldOfView there was, but it requires some refactoring regarding XML material profiles (they can contain printer-specific settings) so we never came around to it so far.

@Ghostkeeper Ghostkeeper changed the title Version upgrade 4.5 to 4.6 Version upgrade 4.5 to 4.6 for Maximum Deviation changes Mar 11, 2020
@Ghostkeeper
Copy link
Collaborator

This PR is superseded by #7385.

@Ghostkeeper Ghostkeeper closed this Apr 1, 2020
@Ghostkeeper Ghostkeeper deleted the simplify_test branch April 1, 2020 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants