part number update & v1.3 mfgpack release #47
Conversation
NS5B1G385DTT1G is obsolete 74AHC1G66GV,125 is a direct replacement updated manufacturing pack
WalkthroughThe pull request involves updating component details across multiple files for two integrated circuits (IC1 and IC2). The changes primarily focus on modifying part numbers, manufacturer information, and supplier details. The README.md file has been updated to reflect the version release for v1.3, while the schematic and board files have been updated with new component attributes from ON Semiconductor to Nexperia USA Inc. Changes
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (3)
src/babelbabel.brd (1)
4263-4266: Consider supply chain resilience.Both IC1 and IC2 now use identical parts (74AHC1G66GV,125), which simplifies inventory but could increase supply chain risk. The dual sourcing from both Digi-Key and LCSC helps mitigate this risk.
Consider maintaining a list of additional compatible alternatives in the project documentation for future reference.
README.md (2)
68-69: Maintain consistent list style.The unordered list items use asterisks (*) while other sections in the document use dashes (-). Consider updating for consistency.
-* Flip the two rest switches. -* Replaced obsolete part: +- Flip the two rest switches. +- Replaced obsolete part:🧰 Tools
🪛 Markdownlint (0.37.0)
68-68: Expected: dash; Actual: asterisk
Unordered list style(MD004, ul-style)
69-69: Expected: dash; Actual: asterisk
Unordered list style(MD004, ul-style)
71-73: Document the part replacement details.The table clearly documents the part number replacement. However, consider adding:
- A brief explanation of why this part is a suitable replacement
- Any potential impact on functionality or compatibility
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
README.md(1 hunks)src/babelbabel.brd(2 hunks)src/babelbabel.sch(2 hunks)
🧰 Additional context used
🪛 Markdownlint (0.37.0)
README.md
68-68: Expected: dash; Actual: asterisk
Unordered list style
(MD004, ul-style)
69-69: Expected: dash; Actual: asterisk
Unordered list style
(MD004, ul-style)
🔇 Additional comments (5)
src/babelbabel.brd (1)
4245-4248: Verify electrical specifications of the replacement part.While both parts are analog switches in SOT23-5L packages, please verify:
- Operating voltage compatibility
- On-resistance specifications
- Switching time characteristics
- Temperature range requirements
Consider updating the documentation to include a note about the compatibility verification performed between NS5B1G385DTT1G and 74AHC1G66GV,125.
✅ Verification successful
Replacement part 74AHC1G66GV,125 is compatible with system requirements
The component meets all system specifications:
- Operating voltage: Compatible with USB power (4.5-5.5V)
- Temperature range: Exceeds required 0 to +40°C
- Package: Maintains SOT23-5L form factor
- Function: Direct analog switch replacement
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Search for voltage specifications in schematic files rg -l "VCC|VDD|power supply" --type txt # Search for temperature specifications in documentation fd -e md -e txt -e pdf | xargs rg -l "temperature|thermal|°C"Length of output: 275
Script:
#!/bin/bash # Check contents of requirements doc cat docs/requirements.md # Find schematic and board files fd -e sch -e brd # Search for component references in found files fd -e sch -e brd -x grep -l "IC1\|IC2\|NS5B1G385\|74AHC1G66"Length of output: 1579
src/babelbabel.sch (2)
12252-12255: Same deviceset naming concerns apply to IC1.The same issues regarding deviceset name mismatch and part compatibility verification apply to this component as well.
12240-12243: Verify component compatibility and update deviceset name.While the attributes have been updated for the new part 74AHC1G66GV,125, the deviceset name still references the old part (NS5B1G385). This mismatch could cause confusion in future maintenance.
Please:
- Confirm that the new part 74AHC1G66GV,125 is pin-compatible and electrically equivalent to NS5B1G385DTT1G
- Consider updating the deviceset name to match the new part number
README.md (2)
66-66: Verify the release year.The release year is set to 2025, which is in the future. Please confirm if this is intentional or if it should be set to the current year (2024).
68-68: Clarify the "rest switches" change.The entry "Flip the two rest switches" lacks context. Consider elaborating on:
- Which switches are being referred to
- The purpose of flipping them
- The impact on functionality
🧰 Tools
🪛 Markdownlint (0.37.0)
68-68: Expected: dash; Actual: asterisk
Unordered list style(MD004, ul-style)
| <attribute name="DIGIKEY#" value="1727-6763-1-ND" x="26.53" y="4.02" size="1.778" layer="27" display="off"/> | ||
| <attribute name="LCSC#" value="C554311" x="26.08" y="3.77" size="1.778" layer="27" display="off"/> | ||
| <attribute name="MANF" value="Nexperia USA Inc." x="26.53" y="4.02" size="1.778" layer="27" display="off"/> | ||
| <attribute name="MANF#" value="74AHC1G66GV,125" x="26.53" y="4.02" size="1.778" layer="27" display="off"/> |
There was a problem hiding this comment.
💡 Codebase verification
Update component technology definition in schematic file
The schematic file (src/babelbabel.sch) contains inconsistent part references:
- Component instances correctly use the new part (74AHC1G66GV,125)
- But the technology definition still references the old part (NS5B1G385DTT1G)
Please update the following attributes in the technology definition:
- DIGIKEY#: Change from "NS5B1G385DTT1GOSDKR-ND" to "1727-6763-1-ND"
- MANF: Change from "ON Semiconductor" to "Nexperia USA Inc."
- MANF#: Change from "NS5B1G385DTT1G" to "74AHC1G66GV,125"
🔗 Analysis chain
Verify consistency across related files.
The component updates look good, but please ensure consistent updates across:
- Schematic file (src/babelbabel.sch)
- BOM/manufacturing documentation
- README.md version history
Also applies to: 4263-4266
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Verify consistent part numbers across files
rg -l "NS5B1G385|74AHC1G66GV|1727-6763-1-ND|C554311"
# Check for any remaining references to old part
rg "NS5B1G385DTT1G"
Length of output: 344
Script:
#!/bin/bash
# Get context around the old part references in schematic
rg -B 2 -A 2 "NS5B1G385DTT1G" src/babelbabel.sch
# Check for new part number in schematic to see if both exist
rg -B 2 -A 2 "74AHC1G66GV,125" src/babelbabel.sch
Length of output: 898
NS5B1G385DTT1G is obsolete
74AHC1G66GV,125 is a direct replacement
will update manufacturing pack
Summary by CodeRabbit
Documentation
Chores