o3de / o3de

Open 3D Engine (O3DE) is an Apache 2.0-licensed multi-platform 3D engine that enables developers and content creators to build AAA games, cinema-quality 3D worlds, and high-fidelity simulations without any fees or commercial obligations.
https://o3de.org
Other
7.81k stars 2.21k forks source link

Changing Translation Offset in entity with Noise Gradient is not reflected on Terrain Layer Spawner entity #13398

Closed LB-KacperKapusta closed 1 year ago

LB-KacperKapusta commented 1 year ago

Describe the bug Modifying Translation Offset in entity with Noise Gradient component targeting a Terrain Layer Spawner entity does not change the terrain being generated. Although, the change is reflected in the Preview window of the Noise Gradient component.

Prerequisites

Steps to reproduce

  1. Add Terrain World and Terrain World Renderer components to Level entity.
  2. Create Entity1 with Terrain Layer Spawner, Axis Aligned Box Shape and Terrain Height Gradient List components.
  3. Change Axis Aligned Box Shape's Dimensions to e.g. X:10, Y:10 and move the Entity1 above Z:0 for the terrain to be rendered.
  4. Create Entity2 with Perlin or Random Noise Gradient component and add the required components to it (as shape use Axis Aligned Box Shape, Box Shape, Capsule Shape or Sphere Shape component).
  5. Select Entity1 and choose Entity2 as target in Terrain Height Gradient List component.
  6. Select Entity2 and modify Translation Offset in its shape component.

Expected behavior The generated terrain updates as the Noise Gradient entity's Preview does.

Actual behavior The Noise Gradient entity's Preview updates but the change is not reflected on the generated terrain.

Screenshots/Video

https://user-images.githubusercontent.com/86953659/204005712-d006eff2-6d85-4c04-8550-32be06efb94b.mp4

Found in Branch Development (da550ef)

Desktop/Device

Additional context Add any other context about the problem here.

greerdv commented 1 year ago

Talking to @mbalfour-amzn it seems like this is the expected behaviour, although it did lead to discovering a couple of other issues (https://github.com/o3de/o3de/pull/14648 https://github.com/o3de/o3de/issues/14650)

greerdv commented 1 year ago

Closing as by design

LB-KacperKapusta commented 1 year ago

Confirming the issue has been closed as by design as per comment above.