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

[PCB Print][Added] Include Table in print #765

Merged
merged 5 commits into from
Jan 7, 2025

Conversation

nguyen-v
Copy link

@nguyen-v nguyen-v commented Jan 7, 2025

This PR adds the possibility to print tables (including drill tables) in PCB Print. The tables are not saved to the PCB (as opposed to the preflight).

In Include Table, if format_drill_table is set to True (which is the default), the drill tables will be printed with a drill marks column on the left and an extra bottom rule for the last row (Total).

A CSV generated either by gerb_drill or excellon is considered to be a drill table.

kibot:
  version: 1

outputs:
  - name: drill_table
    comment: "CSV Drill Tables"
    type: excellon
    dir: tables
    options:
      generate_drill_files: false 
      table:
        unify_pth_and_npth: 'no'
        group_slots_and_round_holes: true

  - name: 'print_drill_table'
    type: pcb_print
    options:
      include_table: true
      drill:  # must match the settings of the CSV table
        unify_pth_and_npth: 'no'
        group_slots_and_round_holes: true
      pages:
          repeat_for_layer: 'User.Comments'
          repeat_layers: 'drill_pairs'
          layers:
            - layer: 'User.Comments'
              color: '#000000'
            - layer: 'Edge.Cuts'
              color: '#000000'

Example of outputs:
image

@nguyen-v nguyen-v changed the title [PCB Print][Added] CSV Drill Tables in PDF [PCB Print][Added] Tables in PDF Jan 7, 2025
@set-soft set-soft merged commit 3aa99a4 into INTI-CMNB:dev Jan 7, 2025
17 checks passed
@nguyen-v nguyen-v changed the title [PCB Print][Added] Tables in PDF [PCB Print][Added] Include Table in print Jan 7, 2025
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.

2 participants