Closed th0mpy closed 7 months ago
Got an error on the CLI action:
Let me take a look
Few points of feedback:
Don't commit the changes just yet. Just a few housecleaning things to get out of the way first. In the meantime, I updated per @bartlammers suggestions.
I'll get to the screenshots tonight and will push a new update as well as do just a little cleaning to tidy up the structure of the KiCad and regenerate the production files.
I made a couple updates to the schematics, per some design considerations.
@bartlammers all updates accounted for. Please review and let me know what you think.
Regarding the STLs. Yes, the Voron 0 STLs are 100% the same as before. I can just remove those STLs.
I just received a few boards yesterday with all of the changes. I'll snap a few new pics tonight and upload with updated filenames.
Images updated. Please have a look and let me know if you approve of the changes. Side note, I confirmed that the new version of KiCad will not allow you to change the extension names. It really messes things up and I had to restore from a backup. All should be good now.
@th0mpy
Only thing i noticed was the Neopixel out silkscreen looks to overlap. May be best to make the pin names a bit larger and shift the title up a bit.
(this may just be my font though, but your image looks to have the same issue)
I updated that label.
I updated that label.
ok great, than i see nothing else to nitpick =D
This PR seems to have removed V0_Display/KiCad/V0_Display.kicad_pcb
, which was the PCB layout for the STM32 V0 display. Was this intentional?
The V0_Display.kicad_pcb was an accident. I've fixed it, but it still shows as a deletion in the PR.
Although, I do see that the file was indeed deleted. I can do a new pull request to fix that.
This is my remix of the V0_display, based on the RP2040 (RaspberryPi Pico).