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

Update glow crystals to new coloured blocks palette #894

Merged
merged 7 commits into from
Jan 7, 2025

Conversation

careeoki
Copy link
Contributor

@careeoki careeoki commented Jan 7, 2025

"New" as of... (checks calendar) 9 months ago.
image
image
image
They now have reflectivity textures which enhances the crystal look a lot.

Reminder to how the colours have been renamed:
gray -> grey
dark_gray -> grey
light_blue -> aqua
dark_red -> crimson
dark_green -> viridian

@ikabod-kee
Copy link
Collaborator

BIG FAN OF THESE

@IntegratedQuantum
Copy link
Member

I'm not sure if reflectivity is actually a good fit here, it kind of makes things brighter on the sides, and I'm not so happy about this.
Screenshot at 2025-01-07 23-18-48
But maybe that would need a fix in the actual reflectivity calculation, like this:
Screenshot at 2025-01-07 23-25-39

@IntegratedQuantum
Copy link
Member

Alright I have decided that this should not block this issue should be resolved by changing the responsible logic and I made an issue for it: #895

Copy link
Member

@IntegratedQuantum IntegratedQuantum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think the glass changes are correct.

@@ -6,7 +6,7 @@
},
.transparent = true,
.hasBackFace = true,
.absorbedLight = 0xd0c16d,
.absorbedLight = 0x2f3e92,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this is correct, this makes it absorb more blue light, turning it yellow.

@@ -6,7 +6,7 @@
},
.transparent = true,
.hasBackFace = true,
.absorbedLight = 0x0d36f1,
.absorbedLight = 0xf2c90e,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I must have accidentally committed these.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, reverted the glass

@IntegratedQuantum IntegratedQuantum merged commit 7631be2 into PixelGuys:master Jan 7, 2025
1 check passed
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.

3 participants