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

Low end pipeline does not render terrain #18468

Open
michalpelka opened this issue Nov 13, 2024 · 6 comments
Open

Low end pipeline does not render terrain #18468

michalpelka opened this issue Nov 13, 2024 · 6 comments
Labels
kind/bug-2409 Used for stabilization/24.09 issues sig/graphics-audio Categorizes an issue or PR as relevant to SIG graphics-audio.

Comments

@michalpelka
Copy link
Contributor

Bug Description

Using low-end pipeline causes to terrain to be rendered black.

Assets-Required

https://gist.github.com/michalpelka/618984b2e35e7c780cc9d27d4905e0f7

Steps to Reproduce

  • Get a level from a link
  • start gamelauncher with parameters:
-r_fullscreen=false
-bg_ConnectToAssetProcessor=0
-r_width=960
-r_height=960
+LoadLevel
levels/o3de_terrain_level/o3de_terrain_level.spawnable --r_renderPipelinePath=passes/LowEndRenderPipeline.azasset

Expected Behavior

image

Actual Behavior

image

Screenshots/Videos

Already attached

Found In Branch

2409 release.

Commit ID From

commit e8570f9 (HEAD, tag: 2409.0, origin/main)

Desktop/Device

ubuntu 22.04

Additional Context

We have a similar issue with o3de/o3de-extras#792
@michalpelka michalpelka added needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. sig/graphics-audio Categorizes an issue or PR as relevant to SIG graphics-audio. kind/bug-2409 Used for stabilization/24.09 issues labels Nov 13, 2024
@moudgils moudgils removed the needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. label Nov 13, 2024
@moudgils
Copy link
Contributor

Can you try with -rhi-validation-mode=enable and attach log to this ticket?

@michalpelka
Copy link
Contributor Author

@moudgils thanks for suggestions!
Logs are here :
https://gist.github.com/michalpelka/d482a4712105e616d4b280292328f7ae

@michalpelka
Copy link
Contributor Author

I've updated logs, I've used --rhi-device-validation=enable parameter.

@michalpelka
Copy link
Contributor Author

Also, the depth image is rendered correctly:
image

@moudgils
Copy link
Contributor

moudgils commented Nov 19, 2024 •
edited
Loading

Adding some more context - I think the issue is that terrain was always tested and built with main render pipeline in mind. In low end render pipeline we enable UNIFIED_FORWARD_OUTPUT to simplify the main render pipeline considerably by unifying the output from the forward pass into a single lighting buffer instead of splitting it into multiple lighting related GBuffer RTs. In order to support terrain with the low end render pipeline we would need to add support for UNIFIED_FORWARD_OUTPUT within the terrain forward shader.

@michalpelka
Copy link
Contributor Author

Thanks for clarification @moudgils. I will add PR to docs on that Terrain is not supported in LowEnd pipeline : https://docs.o3de.org/docs/atom-guide/dev-guide/render-pipelines/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/bug-2409 Used for stabilization/24.09 issues sig/graphics-audio Categorizes an issue or PR as relevant to SIG graphics-audio.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@michalpelka @moudgils