-
-
Notifications
You must be signed in to change notification settings - Fork 346
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
Terrain brush precision controls #661
Conversation
Very cool, thank you! The PR seems to be quite big and will take a day or so to review it, I'll try to do this quickly. |
There is no rush. The automated CI test has revealed a potentially serious problem with the brush rasterization that was not apparent when I was playing with the brushes in an open editor. I am going to need time to track down the source of the problem. Edit: It turned out to just be a small bug in the test, not a problem in brush rasterization at all. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is hard to grasp such amount of code at review, so I basically checked for obvious mistakes and there are two tiny comments.
I have overhauled the terrain to give the user more precise control over terrain editing. This includes:
Brush modes now include:
Each brush stroke now keeps track of which chunks it touches, and copies are made of the relevant texture of each chunk before the texture is modified, allowing the undo command to now keep copies of only the modified textures rather than textures for every chunk.