Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error: Unexpected key "include_keys" within field "pcbs.top_plate"! #22

Closed
thoni56 opened this issue Jul 17, 2022 · 3 comments
Closed

Comments

@thoni56
Copy link

thoni56 commented Jul 17, 2022

I ran the build.sh script locally (with ergogen locally). Unfortunately it failed too (#22) with

> ergogen samoklava.yaml

Ergogen v3.1.2 CLI

Interpreting format: YAML
Preprocessing input...
Calculating variables...
Parsing points...
Generating outlines...
Extruding cases...
Scaffolding PCBs...
Error: Unexpected key "include_keys" within field "pcbs.top_plate"!
    at exports.assert (C:\Users\Thomas\AppData\Roaming\npm\node_modules\ergogen\src\as
sert.js:12:15)
    at Object.exports.unexpected (C:\Users\Thomas\AppData\Roaming\npm\node_modules\erg
ogen\src\assert.js:35:9)
    at Object.exports.parse (C:\Users\Thomas\AppData\Roaming\npm\node_modules\ergogen\
src\pcbs.js:243:11)
    at Object.process (C:\Users\Thomas\AppData\Roaming\npm\node_modules\ergogen\src\er
gogen.js:84:27)
    at C:\Users\Thomas\AppData\Roaming\npm\node_modules\ergogen\src\cli.js:58:29
    at Object.<anonymous> (C:\Users\Thomas\AppData\Roaming\npm\node_modules\ergogen\sr
c\cli.js:126:3)

This is also evident when pasting the yaml into webversion of ergogen.

Perhaps this is what's causing #21 but we're not seeing the ergogen error. The build script finished with the same XIO error...

@thoni56
Copy link
Author

thoni56 commented Jul 17, 2022

This is with ergogen 3.1.2, but I don't get this error with 3.1.0 which is in the repos package.json, so it is probably not causing #21.

@soundmonster
Copy link
Owner

How did you install Ergogen? I would recommend using Samoklava's Makefile:

  • make setup will install my Ergogen fork
  • make clean all will generate and autoroute the board; requires Docker or Podman

@thoni56
Copy link
Author

thoni56 commented Aug 28, 2022

Thanks, yes, I've since learned that Samoklava has a patched Ergogen. Not sufficiently obvious, obviously not for me at least. Thanks for reminding me to close this.

@thoni56 thoni56 closed this as completed Aug 28, 2022
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

No branches or pull requests

2 participants