Skip to content

[Merged by Bors] - Add TextureFormat::Rg16Unorm support for Image and derive Resource for SpecializedComputePipelines#5991

Closed
kurtkuehnert wants to merge 3 commits intobevyengine:mainfrom
kurtkuehnert:texture-format-rg16unorm
Closed

[Merged by Bors] - Add TextureFormat::Rg16Unorm support for Image and derive Resource for SpecializedComputePipelines#5991
kurtkuehnert wants to merge 3 commits intobevyengine:mainfrom
kurtkuehnert:texture-format-rg16unorm

Conversation

@kurtkuehnert
Copy link
Contributor

Objective

Currently some TextureFormats are not supported by the Image type.
The TextureFormat::Rg16Unorm format is useful for storing minmax heightmaps.
Similar to #5249 I now additionally require image to support the dual channel variant.

Solution

Added TextureFormat::Rg16Unorm support to Image.

Additionally this PR derives Resource for SpecializedComputePipelines, because for some reason this was missing.
All other special pipelines do derive Resource already.

@mockersf mockersf added the A-Rendering Drawing game state to the screen label Sep 15, 2022
@alice-i-cecile alice-i-cecile added the C-Usability A targeted quality-of-life change that makes Bevy easier to use label Sep 15, 2022
Copy link
Member

@alice-i-cecile alice-i-cecile left a comment

Choose a reason for hiding this comment

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

bors r+

bors bot pushed a commit that referenced this pull request Sep 15, 2022
…r SpecializedComputePipelines (#5991)

# Objective

Currently some TextureFormats are not supported by the Image type.
The `TextureFormat::Rg16Unorm` format is useful for storing minmax heightmaps.
Similar to #5249 I now additionally require image to support the dual channel variant.

## Solution

Added `TextureFormat::Rg16Unorm` support to Image.

Additionally this PR derives `Resource` for `SpecializedComputePipelines`, because for some reason this was missing.
All other special pipelines do derive `Resource` already.


Co-authored-by: Kurt Kühnert <51823519+Ku95@users.noreply.github.com>
@bors
Copy link
Contributor

bors bot commented Sep 15, 2022

@bors bors bot changed the title Add TextureFormat::Rg16Unorm support for Image and derive Resource for SpecializedComputePipelines [Merged by Bors] - Add TextureFormat::Rg16Unorm support for Image and derive Resource for SpecializedComputePipelines Sep 15, 2022
@bors bors bot closed this Sep 15, 2022
james7132 pushed a commit to james7132/bevy that referenced this pull request Oct 19, 2022
…r SpecializedComputePipelines (bevyengine#5991)

# Objective

Currently some TextureFormats are not supported by the Image type.
The `TextureFormat::Rg16Unorm` format is useful for storing minmax heightmaps.
Similar to bevyengine#5249 I now additionally require image to support the dual channel variant.

## Solution

Added `TextureFormat::Rg16Unorm` support to Image.

Additionally this PR derives `Resource` for `SpecializedComputePipelines`, because for some reason this was missing.
All other special pipelines do derive `Resource` already.


Co-authored-by: Kurt Kühnert <51823519+Ku95@users.noreply.github.com>
james7132 pushed a commit to james7132/bevy that referenced this pull request Oct 28, 2022
…r SpecializedComputePipelines (bevyengine#5991)

# Objective

Currently some TextureFormats are not supported by the Image type.
The `TextureFormat::Rg16Unorm` format is useful for storing minmax heightmaps.
Similar to bevyengine#5249 I now additionally require image to support the dual channel variant.

## Solution

Added `TextureFormat::Rg16Unorm` support to Image.

Additionally this PR derives `Resource` for `SpecializedComputePipelines`, because for some reason this was missing.
All other special pipelines do derive `Resource` already.


Co-authored-by: Kurt Kühnert <51823519+Ku95@users.noreply.github.com>
ItsDoot pushed a commit to ItsDoot/bevy that referenced this pull request Feb 1, 2023
…r SpecializedComputePipelines (bevyengine#5991)

# Objective

Currently some TextureFormats are not supported by the Image type.
The `TextureFormat::Rg16Unorm` format is useful for storing minmax heightmaps.
Similar to bevyengine#5249 I now additionally require image to support the dual channel variant.

## Solution

Added `TextureFormat::Rg16Unorm` support to Image.

Additionally this PR derives `Resource` for `SpecializedComputePipelines`, because for some reason this was missing.
All other special pipelines do derive `Resource` already.


Co-authored-by: Kurt Kühnert <51823519+Ku95@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Rendering Drawing game state to the screen C-Usability A targeted quality-of-life change that makes Bevy easier to use

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants