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

Make atlas.Pack reenterant. Add some utility functions to atlas.Textu… #109

Merged
merged 3 commits into from
Aug 20, 2024

Conversation

dusk125
Copy link
Contributor

@dusk125 dusk125 commented Aug 20, 2024

…reId

@dusk125 dusk125 requested a review from a team August 20, 2024 15:40
Copy link
Contributor

@bhperry bhperry left a comment

Choose a reason for hiding this comment

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

LGTM

return
}

// If we've already packed the textures, we need to convert them back to images to repack them
Copy link
Contributor

Choose a reason for hiding this comment

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

What's the reason for re-packing?

@dusk125 dusk125 merged commit dda5970 into gopxl:main Aug 20, 2024
1 check passed
@dusk125 dusk125 deleted the atlas-updates branch August 20, 2024 17:11
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.

2 participants