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

πŸƒ Add buffer texture #72

Merged
merged 3 commits into from
Feb 4, 2024
Merged

πŸƒ Add buffer texture #72

merged 3 commits into from
Feb 4, 2024

Conversation

paulcscharf
Copy link
Member

✨ What's this?

  • Refactors texture uniforms with shared base class
  • Adds buffer texture
  • Fixes texture targets being mutable, which I believe to be invalid (breaking change)

πŸ” Why do we want this?

Buffer textures allow for easily pushing large amounts of unstructured binary data for efficient use in shaders.

πŸ— How is it done?

See by commits if you like.

πŸ’₯ Breaking changes

Making the texture targets immutable is strictly speaking a breaking change.
However, the argument was already optional, and only used in a single example I'm aware of, where it passed the default value anyway.
Because of that, I consider this largely breaking on a binary level, but not code level.

If you want me to split out a separate PR, I'll do so, but I believe the change to be small enough to include here.

πŸ”¬ Why not another way?

Shader Storage Buffer Objects are even more flexible, but only supported from OpenGL 4.3 onwards, and not available on MacOS.

@paulcscharf paulcscharf merged commit dc357b7 into master Feb 4, 2024
1 check passed
@paulcscharf paulcscharf deleted the buffer-texture branch February 4, 2024 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants