Skip to content

Commit

Permalink
fix: radix test must use radix sort
Browse files Browse the repository at this point in the history
  • Loading branch information
mosure committed Dec 15, 2023
1 parent 3499dd6 commit e9fe460
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
2 changes: 1 addition & 1 deletion Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,7 @@ path = "tests/gpu/gaussian.rs"
[[bin]]
name = "test_radix"
path = "tests/gpu/radix.rs"
required-features = ["debug_gpu"]
required-features = ["debug_gpu", "sort_radix"]


[[bench]]
Expand Down
7 changes: 5 additions & 2 deletions tests/gpu/radix.rs
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,10 @@ fn setup(
commands.spawn((
GaussianSplattingBundle {
cloud,
settings: GaussianCloudSettings {
sort_mode: SortMode::Radix,
..default()
},
..default()
},
Name::new("gaussian_cloud"),
Expand Down Expand Up @@ -203,8 +207,7 @@ impl Node for RadixTestNode {
let position_vec3 = Vec3::new(position[0], position[1], position[2]);
let depth = (position_vec3 - camera_position).length();

let depth_tolerance = 1.0;
let depth_is_non_decreasing = depth_acc <= depth + depth_tolerance;
let depth_is_non_decreasing = depth_acc <= depth;
if !depth_is_non_decreasing {
println!(
"radix keys: [..., {:#010x}, {:#010x}, {:#010x}, ...]",
Expand Down

0 comments on commit e9fe460

Please sign in to comment.