grom/benilla-octo
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

MONKEY (fix-wind): log the live foliageWind tier; test the player-path rows

Browse Source 
The review report (grass and tree sway not moving in the play build) did not
reproduce: the release/play exe publishes grass and tree strength 1 with
foliageWind absent from the config, and capture pairs show sway. The tier is
now logged at boot and on every change so a player run shows what the
receivers were given, and a test holds the no-pin default path: moving
travel, grass and tree rows set, tier 0 clearing them.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
pkuzic
2026-09-26 16:21:45 +01:00
co-authored by Claude Opus 5.5
parent 5f78d55794
commit 2776a5c1b2
1 changed files with 42 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
crates/benilla-world/src/wind/mod.rs
+42
View File
@@ -190,7 +190,19 @@ fn update_wind(
mut frame: ResMut<MonkeyFrame>,
mut clock: Local<WindClock>,
mut offset: Local<Option<f64>>,
mut logged_tier: Local<Option<u8>>,
) {
// MONKEY (fix-wind): name the live tier in the log (boot and every change), so a player run
// shows what the foliage receivers were given.
if *logged_tier != Some(quality.0) {
*logged_tier = Some(quality.0);
info!(
"MONKEY wind: foliageWind tier {} (grass {}, trees {})",
quality.0,
quality.0 >= 1,
quality.0 >= 2
);
}
let offset = *offset.get_or_insert_with(capture_wind_offset);
let seconds = time.elapsed_secs_f64() + offset;
let storm = storm_blend(weather.sky_density);
@@ -331,6 +343,36 @@ mod tests {
}
}
/// MONKEY (fix-wind): the player path — no capture pin, the registered tier — publishes live
/// grass and tree rows every frame, and the travel keeps moving; tier 0 clears them.
#[test]
fn default_tier_publishes_moving_grass_and_tree_rows() {
use bevy::ecs::system::RunSystemOnce;
use std::time::Duration;
assert_eq!(FoliageWind::REGISTERED, 2);
let mut world = World::new();
world.insert_resource(Time::<()>::default());
world.insert_resource(WeatherState::default());
world.insert_resource(FoliageWind(FoliageWind::REGISTERED));
world.insert_resource(WindField::default());
world.insert_resource(MonkeyFrame::default());
let mut travel = Vec::new();
for _ in 0..3 {
world
.resource_mut::<Time>()
.advance_by(Duration::from_millis(500));
world.run_system_once(update_wind).unwrap();
let rows = world.resource::<MonkeyFrame>().pack(0.5, 0.0);
assert_eq!(rows[5][1..], [1.0, 1.0, 1.0], "sway, grass, tree strength");
travel.push(rows[5][0]);
}
assert!(travel[0] < travel[1] && travel[1] < travel[2], "{travel:?}");
world.insert_resource(FoliageWind(0));
world.run_system_once(update_wind).unwrap();
let rows = world.resource::<MonkeyFrame>().pack(0.5, 0.0);
assert_eq!(rows[5][1..], [0.0, 0.0, 0.0]);
}
#[test]
fn travel_rate_stays_bounded_late_in_a_session() {
// MONKEY (fix-wind): the published travel advances at the wind speed, never speed x age.