Skip to content

svi-252b: set night mode transition delay - #163

Merged
openipc-ai merged 1 commit into
OpenIPC:masterfrom
ArthurKoba:fix/svi-252b-transition-delay
Sep 20, 2026
Merged

openipc-ai merged 1 commit into
OpenIPC:masterfrom
ArthurKoba:fix/svi-252b-transition-delay

Conversation

@ArthurKoba

Copy link
Copy Markdown
Contributor

Follow-up to #161 and OpenIPC/majestic#325.

Tested the updated Majestic on the physical SpezVision SVI-252B K202
(HI3516CV200 + IMX323).

With transitionDelayMs: 0, the day -> night transition is now clean and
the previously reported purple/magenta flash is gone.

On night -> day, a very brief purple frame was still visible while the
bistable IR-cut filter was returning to the daylight position.

Setting:

nightMode:
  transitionDelayMs: 150

removes the remaining flash. With 150 ms, both day -> night and night -> day
transitions are clean on this hardware.

This sets transitionDelayMs to 150 ms as the device-specific default for
the SpezVision SVI-252B K202.

@qodo-free-for-open-source-projects

Copy link
Copy Markdown

PR Summary by Qodo

Set SVI-252B night mode transition delay to 150 ms

🐞 Bug fix ⚙️ Configuration changes 🕐 Less than 5 minutes

Grey Divider

AI Description

• Adds a hardware-specific 150 ms delay for IR-cut filter transitions.
• Prevents purple frames during day-to-night and night-to-day mode changes.
High-Level Assessment

The device-specific delay is the appropriate approach because it matches the SVI-252B's bistable IR-cut timing without affecting other hardware. A zero delay was physically tested and left a night-to-day artifact, while a global default would risk regressions across devices.

Files changed (1) +1 / -0

Bug fix (1) +1 / -0
customizer.shConfigure a 150 ms night mode transition delay +1/-0

Configure a 150 ms night mode transition delay

• Sets Majestic's device-specific night mode transition delay to 150 ms. This allows the bistable IR-cut filter to settle before video resumes, preventing brief purple frames in both transition directions.

devices/hi3516cv200_lite_spezvision-svi-252b-k202/general/overlay/usr/share/openipc/customizer.sh

@qodo-free-for-open-source-projects

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can turn on the rule miner and Qodo learns your standards from review history

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@ArthurKoba

Copy link
Copy Markdown
Contributor Author

@openipc-ai Could you please take a quick look at this follow-up PR?

Majestic #325 has now been hardware-tested on the SVI-252B and closed. The 150 ms transition delay fixes the remaining night -> day purple flash, and this PR only carries that confirmed value into the device profile.

Thanks.

@openipc-ai
openipc-ai merged commit 9753e10 into OpenIPC:master Sep 20, 2026
6 checks passed
@openipc-ai

Copy link
Copy Markdown
Contributor

Merged, thank you — and thank you for taking it back to the bench rather than
stopping at "the reported flash is gone."

That is the part worth calling out. The original report was about dusk, the fix
cleared dusk, and it would have been entirely reasonable to close the loop there.
Going on to check dawn, finding the frame that nobody had reported, and then
establishing that 150 ms is what it takes on this board is what turned a fix into
a finished device profile.

The delay stays device-specific on purpose, and your result is a good illustration
of why: what you measured is a bistable filter physically swinging shut, so the
number is a property of this board's mechanism rather than of the SoC, the sensor
or the firmware. Anyone reading this later and copying the 150 into another
profile should treat it as a starting point to verify at dawn on their own
hardware, not as a value that transfers.

This also retires the caveat #161 shipped with — that profile went in noting the
purple frame as a known remainder tracked upstream, and there is now nothing left
outstanding on the SVI-252B K202 day/night path. The only thing still open on the
board is the IR illuminator GPIO, which is why backlightPin remains unset; if
you ever pin that down, that would be a welcome follow-up.

The build is on master now and will be in tonight's nightly.

@ArthurKoba
ArthurKoba deleted the fix/svi-252b-transition-delay branch September 20, 2026 15:58
@ArthurKoba

Copy link
Copy Markdown
Contributor Author

@openipc-ai Small follow-up to the SVI-252B profile: this only corrects the remaining comment about backlightPin. Hardware confirms that the IR illuminator is switched autonomously by the daylight-sensor circuitry, so there is no SoC-controlled backlight GPIO to resolve or configure.

PR: #165

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants