Skip to content
This repository was archived by the owner on Nov 11, 2025. It is now read-only.

Integrate ICCC with KiCad using Rust#8

Merged
luxas merged 6 commits into
mainfrom
kicad-evaluator
Jun 15, 2021
Merged

Integrate ICCC with KiCad using Rust#8
luxas merged 6 commits into
mainfrom
kicad-evaluator

Conversation

@twelho
Copy link
Copy Markdown
Member

@twelho twelho commented Jun 15, 2021

Initialize the next generation of ICCC (#4) now integrated directly into KiCad and written in Rust. You may define expressions for component attributes in the "Symbol Properties" view in KiCad, and ICCC Rust edition (nicknamed evaluator) will parse the EESchema (.sch) file of KiCad and update the values.

This work also unites the parser (#6) and evaluator to facilitate code sharing, and updates the .gitignore to work better with multi-binary Cargo workspaces.

@twelho twelho added the enhancement New feature or request label Jun 15, 2021
@twelho twelho self-assigned this Jun 15, 2021
Copy link
Copy Markdown
Member

@luxas luxas left a comment

Choose a reason for hiding this comment

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

Thanks! I'm happy to merge this now, and then we can continue to iterate.

use kicad_rs::types::*;

// Main function, can return different kinds of errors
fn main() -> Result<(), Box<dyn Error>> {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'll move the logic from here to the library after this has been merged.

// use crate::resolver;

// Main function, can return different kinds of errors
fn main() -> Result<(), Box<dyn Error>> {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Will figure out this code after merge, to use the parsing library.

Comment thread .gitignore

# Temporary files
*.000
*.bak
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why remove these three?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

IDE warned about duplicate entries so I removed the less relevant duplicates.

@luxas luxas marked this pull request as ready for review June 15, 2021 17:19
@luxas luxas merged commit 6d72e74 into main Jun 15, 2021
@twelho twelho deleted the kicad-evaluator branch June 15, 2021 17:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants