-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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 image requirement from 0.24 to 0.25 #12458
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think there are a few issues here:
read_image
needs a buffer sized tototal_bytes
(not just with that capacity)- The code that was adding alpha back into the image data is still needed if we are returning a
TextureFormat::Rgba32Float
. Thehdr
format never has an alpha channel.
There's an .hdr
file in the assets folder you can test with -- modify examples/2d/sprite.rs
to load textures/spiaggia_di_mondello_1k.hdr
.
Thanks a lot for the review, I needed some guidance here. I'll take a look at these issues and push some updates later. |
Updates the requirements on [image](https://github.com/image-rs/image) to permit the latest version. - [Changelog](https://github.com/image-rs/image/blob/main/CHANGES.md) - [Commits](image-rs/image@v0.24.0...v0.25.0) --- updated-dependencies: - dependency-name: image dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]>
@rparrett I managed to make changes so the example you've provided would load that HDR image properly. Please check if what I'm doing now is correct; I'm still unsure if I'm sizing the data buffer properly. Also, I'd appreciate some input on the error messages since I'm not convinced on how I wrote them. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice. Looks good to me and seems to work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
Objective
Solution
image
0.25.