Conversation
…ependencies package.json was accidentally deleted in commit d903845 (OpenAPI #1). Restored it and upgraded all dependencies to resolve peer conflicts: - Docusaurus 2.0.0-beta.8 to 3.9.2 - React 17 to 18 - @mdx-js/react 1.x to 3.x - prism-react-renderer 1.x to 2.x - Added missing deps: react-icons, redocusaurus, docusaurus-plugin-image-zoom - Removed bundled-in-v3 deps: @svgr/webpack, file-loader, url-loader Co-authored-by: Cursor <cursoragent@cursor.com>
There was a problem hiding this comment.
Pull request overview
This pull request adds a new landing page for the Grinn AstraSOM-1680 system-on-module to the Peridio documentation site. The AstraSOM-1680 is an edge AI vision platform featuring a Synaptics Astra SL1680 SoC with 7.9 TOPS NPU performance, designed for smart camera and computer vision deployments.
Changes:
- Added new solution page and comprehensive data for Grinn AstraSOM-1680
- Updated navigation menus to include Grinn AstraSOM-1680 in hardware listings
- Enhanced CSS styling for dropdown menu section titles to improve visual alignment
Reviewed changes
Copilot reviewed 6 out of 9 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/src/pages/solutions/grinn/astrasom-1680.js | New React component for the AstraSOM-1680 solution page |
| src/src/data/solutions/grinn/astrasom-1680.ts | Comprehensive solution data including specs, use cases, and features |
| src/src/css/custom.css | Enhanced dropdown menu section title styling with flex alignment |
| src/sidebars.js | Added Grinn vendor category with AstraSOM-1680 entry |
| src/docusaurus.config.js | Updated hardware dropdown menu to include Grinn AstraSOM-1680 and renamed section to "SEMICONDUCTORS & MODULES" |
| package.json | Appears unchanged (likely a diff artifact) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
cesartevisual
force-pushed
the
cesar-grr-190
branch
from
faeb370 to
68f4dfb
Compare
Co-authored-by: Cursor <cursoragent@cursor.com>
cesartevisual
force-pushed
the
cesar-grr-190
branch
from
68f4dfb to
b3b643f
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 7 out of 10 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| stats: [ | ||
| { value: '7.9', label: 'TOPS NPU Performance' }, | ||
| { value: 'Dual', label: 'MIPI CSI Cameras' }, | ||
| { value: '37×42mm', label: 'Ultra-Compact LGA' }, |
There was a problem hiding this comment.
Inconsistent module dimension specification. This line shows '37×42mm' while other parts of the file (lines 6, 38, and 89) show '37×42.6mm'. The dimensions should be consistent throughout the file. Please verify the correct dimensions and update all occurrences to match.
| { | ||
| "name": "peridio-documentation", | ||
| "version": "0.0.0", | ||
| "private": true, | ||
| "scripts": { | ||
| "docusaurus": "docusaurus", | ||
| "start": "docusaurus start", | ||
| "build": "docusaurus build", | ||
| "swizzle": "docusaurus swizzle", | ||
| "deploy": "docusaurus deploy", | ||
| "clear": "docusaurus clear", | ||
| "serve": "docusaurus serve", | ||
| "write-translations": "docusaurus write-translations", | ||
| "write-heading-ids": "docusaurus write-heading-ids" | ||
| }, | ||
| "dependencies": { | ||
| "@docusaurus/core": "^3.9.2", | ||
| "@docusaurus/plugin-google-gtag": "^3.9.2", | ||
| "@docusaurus/preset-classic": "^3.9.2", | ||
| "@mdx-js/react": "^3.0.0", | ||
| "clsx": "^2.0.0", | ||
| "docusaurus-plugin-image-zoom": "^2.0.0", | ||
| "prism-react-renderer": "^2.3.0", | ||
| "react": "^18.2.0", | ||
| "react-dom": "^18.2.0", | ||
| "react-icons": "^5.5.0", | ||
| "redocusaurus": "^2.1.1" | ||
| }, | ||
| "browserslist": { | ||
| "production": [ | ||
| ">0.5%", | ||
| "not dead", | ||
| "not op_mini all" | ||
| ], | ||
| "development": [ | ||
| "last 1 chrome version", | ||
| "last 1 firefox version", | ||
| "last 1 safari version" | ||
| ] | ||
| } | ||
| } |
There was a problem hiding this comment.
This package.json file appears to already exist at the repository root with identical content. The diff suggests it's being added as a new file, but the file already exists. This may be a git issue where the file was accidentally staged after being deleted and re-added, or it could be an artifact of how the diff was generated. Please verify whether this change is intentional and remove it from the commit if it's not needed.
| { | ||
| type: 'category', | ||
| label: 'Grinn', | ||
| items: [ | ||
| { | ||
| type: 'category', | ||
| label: 'AstraSOM-1680', | ||
| items: [ | ||
| { | ||
| type: 'html', | ||
| value: | ||
| '<a href="/solutions/grinn/astrasom-1680" target="_blank" rel="noopener noreferrer" class="menu__link" style="display: block;">Solution overview</a>', | ||
| }, | ||
| ], | ||
| }, | ||
| ], | ||
| }, |
There was a problem hiding this comment.
The Grinn sidebar entry lacks an autogenerated section for hardware documentation, unlike most other hardware entries (Advantech, NVIDIA, NXP, OnLogic, etc.). This is likely intentional since this is a WIP PR, but for consistency with other hardware vendors, consider adding a corresponding hardware documentation directory at 'src/docs/hardware/grinn/astrasom-1680' with provisioning and setup guides once the landing page is finalized.
1 participant
Added new landing page: Grinn AstraSOM line
Updated main navigation:
Updated Support Matrix: