Repository navigation
Conversation
The old figure counted from the first camera after the one on screen, so it ramped up from 0 during each pan and read a steady delay as one that grows with the pan's length. The picture from before a motion is no longer counted, and the wait for the motion's first picture is logged apart. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Nobody sees the view's own camera when the app starts it, yet it rendered the scene at 1920x1080 among the trees, sun shadows included. It now draws a 64x36 picture of the sky where it stands. In a game the view goes from 7-9 to 16-18 frames a second. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…d for The view alone: pictures 155 ms after their request, from 203 ms, at the same frame rate. Results in plans/battlefield-latency-results.md. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part of #95 (plan:
plans/battlefield-latency.md). Follows a review of the earlier conclusions; details and numbers inplans/battlefield-latency-results.md.What changes
teleoperation_battlefield::lag): a paint now says how old its picture is (time since the app asked for the camera it shows). The old figure ramped from 0 at the start of each pan, which made a steady ~500 ms delay look like a queue that grows with the pan's length. The picture from before a motion is not counted; the wait for the motion's first picture is logged apart.RenderScale::parked, set bytowers_view --hidden): the view's own camera, which nobody sees when the app starts it, draws a 64×36 picture of the sky where it stands instead of the full scene at 1920×1080 among the trees. It can't be turned off (Bevy's atmosphere needs a live view).pacing::IN_FLIGHT2 → 1), and the pacer runs inPostCleanup, after Bevy'smap_buffers, so its wait also completes that frame's readback.Numbers
The battlefield camera has no sun shadows to cut: Bevy's shadows reach 150 m and it sits 450 m from the ground. No look changes in this PR.
Tests
--hiddenparks it. Three old lag tests changed on purpose for the new definition.cargo clippy --workspace --all-targets,cargo fmtclean.cargo test --workspace --release: all pass exceptcognition_league's acceptance suite, which ran out of its 120 s budget before its tests started (unrelated to this change; the machine was busy).Status: pending: it still fails, and the app-side figures from an automated run are unreliable (the window is off screen). Next: an on-screen run.🤖 Generated with Claude Code