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

Fix Coke Oven Hatch gaps #2442

Merged
merged 1 commit into from
Dec 10, 2024
Merged

Fix Coke Oven Hatch gaps #2442

merged 1 commit into from
Dec 10, 2024

Conversation

eve336
Copy link
Contributor

@eve336 eve336 commented Nov 26, 2024

What

it should be accepted so there isnt gaps between blocks bc the model is wrong

Implementation Details

i just changed some values, it doesnt even cause any z fighting or anything theres no issue

Outcome

i chaged some values so theres no gaps between blocks

Additional Information

image
image
no longer any gaps

@eve336 eve336 requested a review from a team as a code owner November 26, 2024 17:25
Copy link
Member

@screret screret left a comment

Choose a reason for hiding this comment

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

does it not z-fight now?
also,

Please fill in as much useful information as possible. Also, please remove all unused sections, including this and the other explanations.

@screret screret added the type: bug Something isn't working label Nov 29, 2024
@Spicierspace153 Spicierspace153 added the type: bugfix general bug fixes label Nov 30, 2024
@screret screret removed the type: bug Something isn't working label Dec 1, 2024
@jtuc
Copy link
Contributor

jtuc commented Dec 7, 2024

Tested on Intel Iris Xe Graphics; can confirm no Z-fighting or seams with both Vanilla and Embeddium rendering.
image
For comparison, GregTech 1.5.4 (seams visible):
2024-12-07_12 58 48

@MatthiasMann
Copy link

doesn't this now render 2 faces on the north face?

@screret
Copy link
Member

screret commented Dec 7, 2024

that's what it's always done

@MatthiasMann
Copy link

can't we just remove the north face from texture 1?

@krossgg krossgg merged commit ac947ed into GregTechCEu:1.20.1 Dec 10, 2024
2 checks passed
@krossgg krossgg changed the title fixed the coke oven hatch having gaps between other blocks Fix Coke Oven Hatch gaps Dec 10, 2024
screret pushed a commit that referenced this pull request Dec 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bugfix general bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants