bevyengine/bevy

Are the `random_sampling` and `sampling_primitives` examples redundant?

Opened this issue · 2 comments

Bevy version

main

What you did

cargo run --example random_sampling
Added in #13507

cargo run --example sampling_primitives
Added in #13519

What went wrong

These examples appear to be extremely similar, and their docs even have identical descriptions.

//! This example shows how to sample random points from primitive shapes.

Additional information

It appears that this isn't a mistake, so I would suggest updating the description of either or both examples to explain what each of them is accomplishing.

But they seem totally redundant to me, so removing one may be an option.

I vote redundant. Remove sampling_primitives.

For some context: I originally wrote the random_sampling example just to demonstrate what ShapeSample was and how to use it. The intention of sampling_primitives (which came shortly after) was to show off the range of primitives for which this sampling feature was supported. I don't have strong feelings about whether one of them gets removed or something. :)