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.

Sign up for GitHub

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

Jump to bottom

Blender FloatProprety not WYSIWYG for places where perfect precision is required #474

Closed
tngreene opened this issue Sep 20, 2019 · 2 comments
Closed

Blender FloatProprety not WYSIWYG for places where perfect precision is required #474

tngreene opened this issue Sep 20, 2019 · 2 comments
Labels
Bug priority high
Milestone
v4.1.0-beta.2

Comments

@tngreene
Copy link
Contributor

Make an Axis Detent Range, set its start to .83. Because Axis Detent Ranges represent very specific ranges you want it to be exact - especially if it is going to be part of a script.

Instead you might get .829999999! Blender's UI actually lies. A FloatProperty always stores the whole float and only shows a rounded value in the UI.

We need to investigate places where float properties must perfectly match what the UI says.
@tngreene tngreene added good first issue Bugs in one or two easy plces that can be fixed quickly help wanted labels Oct 7, 2019
@tngreene
Copy link
Contributor Author

tngreene commented Jun 5, 2020

This is probably adjusting the precision of the prop and rounding to that precision

@tngreene tngreene removed the good first issue Bugs in one or two easy plces that can be fixed quickly label Oct 27, 2020
@tngreene tngreene added this to the v4.1.0-beta.2 milestone May 10, 2021
tngreene added a commit that referenced this issue May 13, 2021
@tngreene
Partially closes #474, we have the code, we have new unit test, but o…
e411953 
…ther unit tests are broken.
tngreene added a commit that referenced this issue May 13, 2021
@tngreene
Closes #474 by fixing old unit tests. The only thing that's changed i…
508796a 
…s the precision and rounding. The test remains the same. the .blend file's only changes are things like setting Root Collection and Print Debug Info.
@tngreene
Copy link
Contributor Author

Finished! Hopefully no one's manipulators get messed up

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug priority high
Projects
None yet
Milestone
v4.1.0-beta.2
Development

No branches or pull requests
1 participant
@tngreene