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.

Sign up for GitHub

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

Jump to bottom

Improve multithreading support for render command encoding #17733

Open
PPakalns opened this issue Feb 7, 2025 · 0 comments
Open

Improve multithreading support for render command encoding #17733

PPakalns opened this issue Feb 7, 2025 · 0 comments
Labels
C-Feature A new feature, making something new possible S-Needs-Triage This issue needs to be labelled

Comments

@PPakalns
Copy link
Contributor

PPakalns commented Feb 7, 2025

What problem does this solve or what need does it fill?

Multi-threading support for rendering command encoding was added in #9172 .

Support for it was added to some subset of nodes. Currently for example MainOpaquePass2dNode has been parallelized, but MainTransparentPass2dNode has not been parallelized.

Parallelized command encoding is especially important for setups where multiple camera views are rendered. Currently graph is executed in serial and no parallelization is done for unsupported nodes.

What solution would you like?

Continue adding support for render node parallelization.

What alternative(s) have you considered?

Currently none.

Additional context

See discussion on discord rendering channel: https://discord.com/channels/691052431525675048/866787577687310356/1337553149208035359
@PPakalns PPakalns added C-Feature A new feature, making something new possible S-Needs-Triage This issue needs to be labelled labels Feb 7, 2025
@PPakalns PPakalns mentioned this issue Feb 7, 2025
Merged
github-merge-queue bot pushed a commit that referenced this issue Feb 9, 2025
@PPakalns
main_transparent_pass_2d render node command encoding parallelization (…
2d62026 
…#17735)

# Objective

- Add command encoding parallelization to transparent 2d pass render
node.
- Improves #17733

## Solution

Using functionality added in
#9172

## Testing

- Tested in personal project where multiple cameras are rendered with
objects rendered in transparent 2d pass.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-Feature A new feature, making something new possible S-Needs-Triage This issue needs to be labelled
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@PPakalns