Side note, we need to figure out why the bounce lighting isn't super visible in this test scene. Maybe @DGriffin91 knows?

Unfortunately this is a bit more expensive, likely because of the extra atomics. I'll have to test if copying the lifetime before the cell update to avoid the atomicLoad() improves things.

Side note, we need to figure out why the bounce lighting isn't super visible in this test scene.

        DirectionalLight {
            illuminance: light_consts::lux::FULL_DAYLIGHT,
            shadows_enabled: false, // Solari replaces shadow mapping
            ..default()
        },

With default exposure.

Unless we can find actual visual issues with using a store instead of a max, the store is probably acceptable. It just means we will be a bit more aggressive at clearing than is really necessary.

atomicMax vs atomicStore doesn't seem to matter. I'm guessing the new atomicLoad is the problem, which is easy enough to fix.

Side note, we need to figure out why the bounce lighting isn't super visible in this test scene. Maybe @DGriffin91 knows?

Which version of the scene is this? By default all the materials are 100% metallic.

Side note: I'm not able to open the scene in bevy 0.17: #21490

Figured out the performance changes, it's just from adjusting the maximum allowed cell lifetime. Not a big deal.

Which version of the scene is this?

@SparkyPotato's on github. Your's wasn't updated at the time I set this up.