Skip to content

Add Mainboard Schematic#24

Draft
maxstrid wants to merge 4 commits into
mainfrom
hardware
Draft

Add Mainboard Schematic#24
maxstrid wants to merge 4 commits into
mainfrom
hardware

Conversation

@maxstrid
Copy link
Copy Markdown

Documentation Included (y/n)
Has Unit Tests (y/n)
Generative AI used (y/n)

Description of Changes

Explanation

Recommendations

Future Work

Questions

AI Usage

Signed-off-by: Maxwell Henderson <mxwhenderson@gmail.com>
@monty699 monty699 self-requested a review March 2, 2026 06:39
@monty699 monty699 self-assigned this Mar 2, 2026
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The original pr has changes in this form because its convinient to open and edit files, I'd prefer for schematic changes to go there instead of in the root because it keeps it organized.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This shouldn't be in a zip

@@ -0,0 +1,14888 @@
(kicad_sch
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The version here and in the zip are different

Comment thread electrical schematic
@@ -0,0 +1 @@
electrical schematic in here
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does this file mean

@@ -0,0 +1,14888 @@
(kicad_sch
(version 20260306)
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should label the schematic elements, right now unless you know the ic you don't know what sensor is what

@@ -0,0 +1,14888 @@
(kicad_sch
(version 20260306)
(generator "eeschema")
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are there 2 nrfs

(kicad_sch
(version 20260306)
(generator "eeschema")
(generator_version "10.0")
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is the lidar still in here? Are we still gonna use it?

(version 20260306)
(generator "eeschema")
(generator_version "10.0")
(uuid "5575cec3-c870-4eb0-ab2c-2d475ba00070")
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why does this board take in 22v or 12v, there's no power anything on this board unless you're planning to do power management on this board

(generator_version "10.0")
(uuid "5575cec3-c870-4eb0-ab2c-2d475ba00070")
(paper "A4")
(lib_symbols
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we be consistent with power up ground down on this pcb, some of them follow it and some dont

@maxstrid
Copy link
Copy Markdown
Author

Also you have the IC versions of these components, but we use breakout boards

@maxstrid
Copy link
Copy Markdown
Author

And you have filter caps and resistors here that I assume we won't have in the actual board because we're using breakouts

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.

4 participants