20260907 - Default the standalone Doppler span to +/-300 Hz - #58
Merged
Merged
Conversation
Keeps the standalone/development config in step with the fleet default, which moves to +/-300 Hz in retina-node's default.yml. This file is only the standalone/dev default: deployed nodes get theirs from the merger, which also carries the migration that lets the new span past an existing user.yml. test/regression's DOPPLER_MIN/DOPPLER_MAX stay at +/-200 on purpose. That harness fixes a reference the golden digests are cut against, so a tuning change here must not move it. The sdr-variants and config_test_* files still read +/-200. Nothing references them, and they are per-device templates rather than the working default, so they are left for whoever next touches the device they belong to. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Ticket: https://app.clickup.com/t/123zgec0m7t
Fleet half: offworldlabs/retina-node#36
Keeps the standalone/development config in step with the fleet default, which moves to +/-300 Hz in retina-node's
default.yml.This file is only the standalone/dev default. Deployed nodes get theirs from the merger, which also carries the migration that lets the new span past an existing
user.yml, so this PR on its own changes nothing in the field.Two things deliberately left at +/-200:
test/regression'sDOPPLER_MIN/DOPPLER_MAX. That harness fixes a reference the golden digests are cut against, so a tuning change here must not move it.config/sdr-variants/*.ymlandconfig_test_*.yml. Nothing references them and they are per-device templates rather than the working default, so they are left for whoever next touches the device they belong to.test/unit/process/ambiguity/TestAmbiguity.cppalready exercises +/-300 and asserts 301 bins, so no test changes were needed.Verified live on owl at 213 MHz: 301 Doppler bins spanning -300.02 to +300.02 Hz, with the off-round extent confirming nCorr 3322 and a 0.499961 s true CPI. Details in the retina-node PR.
🤖 Generated with Claude Code