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 My Broken Implementation of G_QUAD for f3dex That Never Worked #623

Merged
merged 1 commit into from
Jun 2, 2024

Conversation

MegaMech
Copy link
Contributor

@MegaMech MegaMech commented Jun 1, 2024

The args for G_QUAD to render quadrangles are correct now.

@briaguya-ai
Copy link
Collaborator

this PR has the same title as #599

i assume this is fixing something, would you mind updating the title/description to reflect what this is fixing?

@Kenix3
Copy link
Owner

Kenix3 commented Jun 1, 2024

Looks like it's fixing (or just optimizing) how the quad is generated for the rendering api.

@MegaMech MegaMech changed the title Implement G_QUAD for f3dex Fix My Broken Implementation of G_QUAD for f3dex That Never Worked Jun 1, 2024
@MegaMech
Copy link
Contributor Author

MegaMech commented Jun 1, 2024

The arguments were not correct before and I was drawing too many triangles.

Sorry, it was difficult to see if it was drawing correctly because the lighting doesn't work in the scene. But it's good now.

@Kenix3 Kenix3 merged commit 7d8cd81 into Kenix3:main Jun 2, 2024
5 checks passed
@MegaMech MegaMech deleted the quad branch June 2, 2024 16:26
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