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

BlockModelRenderer.BrightnessCache has poor mappings #3604

Open
Technici4n opened this issue Jul 12, 2023 · 1 comment
Open

BlockModelRenderer.BrightnessCache has poor mappings #3604

Technici4n opened this issue Jul 12, 2023 · 1 comment

Comments

@Technici4n
Copy link
Member

The general idea should be as follows. I let you figure out the exact name.
getInt -> getLightmap
getFloat -> getAo

@PepperCode1
Copy link
Contributor

The renames for the brightness cache are not so straightforward. All uses of light and AO should also be reconsidered and made consistent with whatever the new standard terms are. These uses include the Block and BlockState methods that return the AO values, at least. Indigo and Sodium should also adapt to the new names.

lightmap to describe "packed light texture/light map coordinates" is okay and it is what I am used to, but lightmap really describes the texture itself more than coordinates for it. light, packedLight, lightCoords, or lightUv may be better names.

ao to describe the ambient occlusion value is not enough, I think. Such a name might give a wrong impression of how the value works. 1 is no AO, while 0 is all AO. aoMultiplier or aoBrightness may be a better name.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants