Skip to content

News: Ambarella S3L owners wanted, in all three languages - #558

Merged
openipc-ai merged 1 commit into
masterfrom
news-ambarella-s3l-owners
Nov 7, 2026
Merged

openipc-ai merged 1 commit into
masterfrom
news-ambarella-s3l-owners

Conversation

@openipc-ai

Copy link
Copy Markdown
Collaborator

A news post asking owners of Ambarella S3L cameras to send an ipctool report from stock firmware, in English, Russian and Chinese.

Why this, and why not a download

An audit of what the site says about Ambarella found:

  • No route for testers. The S3L row on /cameras says "Not available — in progress" and points to the chat. Nothing asks owners for anything, or says what works.
  • The safe thing to ask for already exists. /cameras/report takes ipctool's output from stock firmware, over uget and telnet. The ipctool served at openipc.org/ipctool (4e28973) includes ambarella: detect the SoC from the machine string ipctool#267's Ambarella detection.
  • The published image isn't safe to flash yet. s3l_ultimate from OpenIPC/firmware carries no board device tree (# CONFIG_ARM_APPENDED_DTB is not set), and the one camera tested, an IP4M-1026E, needs the DTB from ambarella: s3l: Amcrest IP4M-1026E, with the board DTB appended to the uImage linux#178. So the post asks people not to flash the nightly image.

Every factual claim comes from a public PR: firmware #2781, #2782, #2785, #2787 and #2788, linux#177 and #178, and the device list in firmware#2767. Where a claim couldn't be checked, the post says less: it gives no release year for the chip, doesn't say ipctool tells S3L from S3Lm, and names camera models rather than firmware version ranges, since Dahua uses the same version numbers on other chips. It says the IP4M-1026E result came from the contributor's own board profile and streamer, not from Majestic.

Not changed here

The catalogue entry for S3L (data/catalogue/ambarella.yml) is left as it is, deliberately:

  • linux_filename is openipc.ambarella-s3l-nor-lite.tgz, a name that never existed. The availability code resolves it to board ambarella, so the S3L row will keep saying "none" even after the nightly publishes openipc.s3l-nor-ultimate.tgz. Correcting it would make the row offer that image as a download, the one this post asks people not to flash. It should change once a board profile boots from flash.
  • build_status_url points to ambarella-s3l_images.yml, a workflow that doesn't exist. The same is true of every per-SoC badge I checked (xm530_images.yml, for example). OpenIPC/firmware has only build.yml, so that's a site-wide fix, not an S3L one.

Checks

  • tools/news/translate.py --check ru and --check zh: links, code, items and headings all match the original.
  • npm run export -w @openipc/site regenerated news.json. sitemap.golden.xml is regenerated from the build, as its test asks, and the diff is the post's 21 lines.
  • npm run build: the post is at /news/ambarella-s3l-camera-owners-wanted, /ru/... and /zh/..., with /cameras/report and /community localised per language, and it's in the atom feed.
  • npm run lint passes. npm test has 77 failures, the same 77 as origin/master on this host, which runs Node 26 where the repo expects 24. The post adds none.

Not deployed: deploying is a separate step.

S3L firmware builds and streams on one camera, but the image CI publishes
carries no board device tree, so it is not for flashing. What moves S3L
forward now is knowing which cameras exist, and the report page already
takes ipctool's output from stock firmware. The post asks for that, and
says plainly not to flash the nightly image.
@qodo-free-for-open-source-projects

Copy link
Copy Markdown

PR Summary by Qodo

Invite Ambarella S3L camera owners to submit reports in three languages

📝 Documentation 🕐 20-40 Minutes

Grey Divider

AI Description

• Publish English, Russian, and Chinese guidance on S3L progress and the nightly image’s flashing
 risk.
• Ask owners for ipctool reports from stock firmware to help identify cameras needing board
 profiles.
• Add the post to generated news data and the sitemap snapshot.
Diagram

graph TD
  A["Translated posts"] --> B["News export"] --> C["News JSON"] --> D["Localized pages"]
  C --> E["Atom feeds"]
  C --> F["Sitemap"]
Loading
High-Level Assessment

Using the existing multilingual news pipeline and report page is appropriate: it gives owners a way to contribute without presenting the untested nightly image as a safe download. A catalogue availability change should wait for a flash-tested board profile.

Files changed (5) +199 / -0

Tests (1) +21 / -0
sitemap.golden.xmlRecord all three localized post URLs +21/-0

Record all three localized post URLs

• Adds English, Russian, and Chinese URLs for the post, each with hreflang alternates, to the generated sitemap snapshot.

frontend/apps/site/src/lib/sitemap.golden.xml

Documentation (4) +178 / -0
2026-11-07-ambarella-s3l-camera-owners-wanted.mdPublish the English S3L owner request +61/-0

Publish the English S3L owner request

• Explains the current S3L build and single-camera result, warns against flashing the generic nightly image, and directs owners to submit a private ipctool report from stock firmware. Links developers to the community and existing device discussion.

data/news/2026-11-07-ambarella-s3l-camera-owners-wanted.md

2026-11-07-ambarella-s3l-camera-owners-wanted.ru.mdTranslate the S3L request into Russian +61/-0

Translate the S3L request into Russian

• Provides Russian metadata and article text with the same progress update, flashing warning, reporting instructions, and community links.

data/news/2026-11-07-ambarella-s3l-camera-owners-wanted.ru.md

2026-11-07-ambarella-s3l-camera-owners-wanted.zh.mdTranslate the S3L request into Chinese +32/-0

Translate the S3L request into Chinese

• Provides Chinese metadata and article text covering the current limitations and the request for stock-firmware ipctool reports.

data/news/2026-11-07-ambarella-s3l-camera-owners-wanted.zh.md

news.jsonExport the multilingual post into site data +24/-0

Export the multilingual post into site data

• Adds the new slug and English article with Russian and Chinese translations to the generated news data consumed by pages and feeds.

frontend/apps/site/src/data/news.json

@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 hide the parts of a finding you never read, like the evidence or the agent prompt

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@openipc-ai
openipc-ai merged commit 594523c into master Nov 7, 2026
2 checks passed
@openipc-ai
openipc-ai deleted the news-ambarella-s3l-owners branch November 7, 2026 10:27
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.

1 participant