diff --git a/device_firmware/switch_controller/README.md b/device_firmware/switch_controller/README.md index 5c62f1e..8c02b23 100644 --- a/device_firmware/switch_controller/README.md +++ b/device_firmware/switch_controller/README.md @@ -68,6 +68,29 @@ умноженный на `kBoundaryFactorNum / kBoundaryFactorDen` (по умолчанию 1.5). Пока минимум не набран, восстановление молчит — чередование корректно и без него. +### Надо ли перекалибровывать паузу под каждую настройку + +Нет, порог подстраивается сам, и вот что на него влияет, а что нет. + +**IFBW не влияет.** Он меняет длительность оцифровки, то есть интервал от фронта +до спада, а контроллер меряет паузу от спада до следующего фронта. Это разные +интервалы. Плюс переключение делается по спаду, когда оцифровка уже закончилась, +а детектор границы во время оцифровки вообще неактивен — так что сдвинуть ключ +посреди измерения он не может ни при каком IFBW. + +**Число точек, полоса, мощность не влияют** — паузы от них не зависят. + +**Влияют `dwell_us` и задержка PLL прибора**: они и есть время успокоения, то +есть сама пауза. Но подстройка на это и рассчитана — окно минимума короткое (16 +отсчётов), поэтому новое значение подхватывается за считанные точки. Ручной +калибровки не требуется. + +**Смена диапазона внутри развёртки безопасна.** На точках нижнего диапазона +прибор останавливает развёртку и работает с Si5351 по I2C — такие паузы намного +длиннее обычных. Логика от этого не ломается: остановка всегда происходит на +границе точки, перед стадией 0, а значит «длинная пауза → следующая стадия +нулевая → положение A» остаётся верным. + Множитель намеренно щедрый. Ложное срабатывание **посреди** точки испортит данные, а несрабатывание всего лишь отключит страховку. Если по отчёту видно, что границы не распознаются вообще (`point boundaries` близко к нулю при diff --git a/device_firmware/switch_controller/src/main.cpp b/device_firmware/switch_controller/src/main.cpp index 1f442f6..a59e661 100644 --- a/device_firmware/switch_controller/src/main.cpp +++ b/device_firmware/switch_controller/src/main.cpp @@ -63,9 +63,20 @@ constexpr bool kSwitchActiveLow = false; constexpr uint32_t kBoundaryFactorNum = 3; constexpr uint32_t kBoundaryFactorDen = 2; -// Окно, по которому берётся минимальная пауза. Двух окон хватает, чтобы -// оценка пережила смену настроек развёртки за пару секунд. -constexpr uint32_t kGapWindowSamples = 4096; +// Окно, по которому берётся минимальная пауза. Оно намеренно КОРОТКОЕ. +// +// Межстадийная пауза — самая частая: при четырёх стадиях на точку это три паузы +// из четырёх, при двух — одна из двух. Значит минимум даже по десятку отсчётов +// уже равен ей, и длинное окно не даёт никакой точности (пауза задаётся +// счётчиком в FPGA, она детерминирована). +// +// Зато длинное окно вредит. Если пауза ВЫРОСЛА — подняли dwell_us или задержку +// PLL прибора — устаревший маленький минимум держит порог ниже реальной +// межстадийной паузы, и детектор границы начинает срабатывать на каждой стадии, +// залипая ключ в положении A. С таким окном это самоустраняется за считанные +// точки, а не за развёртку. По той же причине короткое окно быстро вымывает +// случайную короткую паузу от помехи. +constexpr uint32_t kGapWindowSamples = 16; // Защита от шума: пауза короче этого не считается настоящей стадией. constexpr uint32_t kMinPlausibleGapMicroseconds = 2;