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

Generalize correction pattern file handling #438

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mairas
Copy link
Sponsor Contributor

@mairas mairas commented Oct 25, 2022

I hadn't quite finished the correction pattern work so far. This PR fixes at least some of the issues:

  • The unused Part Id field is removed from the file format
  • Pattern is matched against the reference, value and footprint name fields (in this order)
  • Error handling when reading the file is improved a bit
  • Documentation has been added

Check that the number of fields is correct for each row.
The correction pattern expression is now tried against the reference,
value and footprint name attributes of each footprint (in this order).
@mairas
Copy link
Sponsor Contributor Author

mairas commented Mar 2, 2023

Could this PR be considered for merging? At the moment I need to maintain a separate local fork to support my workflow, and it gets old soon... :-)

@mairas
Copy link
Sponsor Contributor Author

mairas commented Mar 31, 2023

@yaqwsx ping?

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.

None yet

1 participant