[Feature][Drill Map Drawing/Table] Generate drill drawings in PCB Print

[nguyen-v]

This PR addresses #733.
It adds the following:

• CSV drill table output to excellon output

outputs:
- name: csv_drill_table
  type: excellon
  options:
    generate_drill_files: false
    table:
      pth_and_npth_single_file: @PTH_NPTH@
      group_slots_and_round_holes: @GROUP_ROUND_SLOTS@

• Fixes a few issues with Include Table when setting has_header: false which was not working correctly before
• Include Table now also in PCB Print
• Mechanism to add a drill table and drill map drawing to the PCB Print using repeat_layers and setting the CSV as a drill table with a flag. Also new text expansions (%lp, %lpn) available. When using repeat_layers: 'drill_pairs', it will create as many pages as there are drill layer pairs (taking into account separation of PTH/NPTH). The drill table group has to be placed on the same layer as the repeat_layers layer.

outputs:
- name: pdf_fabrication
  type: pcb_print
  options:
    colored_pads: false
    colored_vias: false
    output: '%f-fabrication%I%v.%x'
    format: 'PDF'
    realistic_solder_mask: false
    dnf_filter: _kibom_dnf_Config
    frame_plot_mechanism: 'internal' 
    drill:
      pth_and_npth_single_file: @PTH_NPTH@
      group_slots_and_round_holes: @GROUP_ROUND_SLOTS@
    include_table:
      outputs:
        - name: 'csv_drill_table'
          is_drill: true

    pages:
      - scaling: @SCALING@
        sheet: 'Drill Drawing %lp'
        layer_var: 'Drill Drawing %lp (Scale @SCALING@:1)'
        sheet_reference_color: '#000000'
        colored_holes: true
        holes_color: "#FFFFFF"
        repeat_for_layer: '@LAYER_DRILL_MAP@'
        repeat_layers: 'drill_pairs'
        layers:
          - layer: '@LAYER_DRILL_MAP@'
            color: '#000000'

Example of what it can output:

[nguyen-v]

I seem to encounter some segmentation faults for some reason with two pdf pcb print in succession, will investigate

[nguyen-v]

Also with the current behavior, the PCB is modified by PCB Print (to save the tables in the PCB). I also needed to add a save just after restoring the image groups (otherwise it would be saved in a state where the image groups are still on the work layer as it is being saved after include table). I think this is fine, but maybe we should add a flag for it? It also looks like it's making one of the tests fail

[set-soft]

Hi @nguyen-v !
Lets separate the features in individual PRs. And also add some test cases, otherwise is too hard to review and debug.

[nguyen-v]

Hi @nguyen-v ! Lets separate the features in individual PRs. And also add some test cases, otherwise is too hard to review and debug.

Got it!