-
-
Notifications
You must be signed in to change notification settings - Fork 31
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
New property gets a min/max value of infinite or some really big number when added to MustardUI #186
Comments
This is because most of the properties in Blender has a max/min to infinity themselves, so it's transferred to the UI properties. Or is there a property that has different max/min and it's transferred uncorrectly in the UI? If so, that might be a bug |
I noticed that custom properties imported to the UI will have the min max value set to I think 2 million. Any other node that uses floats seem to import fine. |
Yeah I guess it's taking the value set on the property, I checked. Unfortunately I prefer this option rather than setting a random -1/1 or 0/1 without caring about the existent value. Did you find any specific part where this evidently an error? |
Alright that's fine. It just would be nice to not have it set to some absurd number every time I add a float-point property to the UI. |
Dunno if it is of any help, but a work around this issue is to assign the old "MustardUI - Float/Bool" on the name of the setting that is gonna be used on the UI and let the UI detection insert that into the UI on it's own, for example, for a shader node: (MustardUI Float - Subsurface). |
@J4ke6599 I understand the issue, maybe I can add a sort of limiter when -inf + inf is available. But again, this is how it was intended in Blender when you add some king of values to the UI. :( |
I was thinking about adding an option (False by default) to make people set a "default" max or min when adding custom properties, if the value is -inf/+inf. What do you think? |
Pretty much what the title says
The text was updated successfully, but these errors were encountered: