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

Make RootMotionView use ArrayMesh instead of ImmediateGeometry #48568

Conversation

TokageItLab
Copy link
Member

Fixed #48389

2021-05-09.10.19.38-1.mov

@Calinou
Copy link
Member

Calinou commented May 9, 2021

If we reach an agreement on this PR, it may be worth opening a similar PR for the 3.x branch to improve editor performance. ImmediateGeometry is known to be slow in the GLES3 and GLES2 renderers.

@TokageItLab TokageItLab force-pushed the adopt-rootmationview-to-arraymesh branch 3 times, most recently from f6e6598 to cae4981 Compare May 9, 2021 01:43
@TokageItLab TokageItLab changed the title Make RootMotionView use ArrayMesh instead of ImmidiateGeometry Make RootMotionView use ArrayMesh instead of ImmediateGeometry May 9, 2021
@TokageItLab TokageItLab force-pushed the adopt-rootmationview-to-arraymesh branch 2 times, most recently from 62a1705 to 19d7597 Compare May 9, 2021 02:15
@TokageItLab TokageItLab force-pushed the adopt-rootmationview-to-arraymesh branch from 19d7597 to 2f15c33 Compare May 9, 2021 02:37
@clayjohn
Copy link
Member

clayjohn commented May 9, 2021

I think we need to wait and see how efficient ImmediateGeometry is in Vulkan before evaluating this PR. Recreating an ArrayMesh each frame will be an unnecessary bottleneck. Ideally ImmediateGeometry will be implemented in a way that is optimized for per-frame updates.

@TokageItLab
Copy link
Member Author

Indeed. We are still waiting for the implementation of ImmidiateGeometry to be done as soon as possible.

@TokageItLab
Copy link
Member Author

TokageItLab commented May 10, 2021

If changes to the original implementation are allowed, drawing the grid with the shader may be the most performant.

@TokageItLab TokageItLab closed this Jul 1, 2021
@TokageItLab
Copy link
Member Author

TokageItLab commented Jul 1, 2021

fixed by #50014

@TokageItLab TokageItLab deleted the adopt-rootmationview-to-arraymesh branch January 21, 2022 10:20
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.

Vulkan: RootMotionView's grid is not displayed
4 participants