Skip to content

[Merged by Bors] - Helpers to check pipeline cache status#5796

Closed
mockersf wants to merge 3 commits into
bevyengine:mainfrom
mockersf:pipeline-cache-check-main
Closed

[Merged by Bors] - Helpers to check pipeline cache status#5796
mockersf wants to merge 3 commits into
bevyengine:mainfrom
mockersf:pipeline-cache-check-main

Conversation

@mockersf

@mockersf mockersf commented Aug 25, 2022

Copy link
Copy Markdown
Member

Objective

  • In WASM, creating a pipeline can easily take 2 seconds, freezing the game while doing so
  • Preloading pipelines can be done during a "loading" state, but it is not trivial to know which pipeline to preload, or when it's done

Solution

  • Add a log with shaders being loaded and their shader defs
  • add a function on PipelineCache to return the number of ready pipelines

@mockersf mockersf added A-Rendering Drawing game state to the screen C-Usability A targeted quality-of-life change that makes Bevy easier to use labels Aug 25, 2022
Comment thread crates/bevy_render/src/render_resource/pipeline_cache.rs Outdated

@djeedai djeedai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Change looks good but public items need documentation.

Comment thread crates/bevy_render/src/render_resource/pipeline_cache.rs

@djeedai djeedai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approving as #5600 does the documenting part already.

@djeedai djeedai added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Sep 2, 2022
@alice-i-cecile

Copy link
Copy Markdown
Member

bors r+

bors Bot pushed a commit that referenced this pull request Sep 2, 2022
# Objective

- In WASM, creating a pipeline can easily take 2 seconds, freezing the game while doing so
- Preloading pipelines can be done during a "loading" state, but it is not trivial to know which pipeline to preload, or when it's done

## Solution

- Add a log with shaders being loaded and their shader defs
- add a function on `PipelineCache` to return the number of ready pipelines
@bors

bors Bot commented Sep 2, 2022

Copy link
Copy Markdown

@bors bors Bot changed the title Helpers to check pipeline cache status [Merged by Bors] - Helpers to check pipeline cache status Sep 2, 2022
@bors bors Bot closed this Sep 2, 2022
james7132 pushed a commit to james7132/bevy that referenced this pull request Oct 28, 2022
# Objective

- In WASM, creating a pipeline can easily take 2 seconds, freezing the game while doing so
- Preloading pipelines can be done during a "loading" state, but it is not trivial to know which pipeline to preload, or when it's done

## Solution

- Add a log with shaders being loaded and their shader defs
- add a function on `PipelineCache` to return the number of ready pipelines
ItsDoot pushed a commit to ItsDoot/bevy that referenced this pull request Feb 1, 2023
# Objective

- In WASM, creating a pipeline can easily take 2 seconds, freezing the game while doing so
- Preloading pipelines can be done during a "loading" state, but it is not trivial to know which pipeline to preload, or when it's done

## Solution

- Add a log with shaders being loaded and their shader defs
- add a function on `PipelineCache` to return the number of ready pipelines
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 S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants