Skip to content

Add run_palace and combine_snp wrapper scripts for gds2palace integration#260

Draft
Copilot wants to merge 2 commits into
next_releasefrom
copilot/fix-mesh-generation-error
Draft

Add run_palace and combine_snp wrapper scripts for gds2palace integration#260
Copilot wants to merge 2 commits into
next_releasefrom
copilot/fix-mesh-generation-error

Conversation

Copy link
Copy Markdown

Copilot AI commented Apr 20, 2026

The gds2palace workflow auto-generates a run_sim script calling run_palace and combine_snp, but neither script was provided anywhere in the IIC-OSIC-TOOLS environment, causing immediate command not found failures when running any gds2palace simulation.

Changes

  • _build/images/palace/scripts/run_palace — bash wrapper around the palace binary; supports optional -np N for MPI parallelism
  • _build/images/palace/scripts/combine_snp — bash wrapper that invokes combine_extend_snp.py via readlink -f so it resolves correctly when called through the ${TOOLS}/bin/ symlink
  • _build/images/palace/scripts/combine_extend_snp.py — Python post-processor (adapted from gds2palace_ihp_sg13g2/scripts) that walks the output tree for port-S.csv (Palace) or scalar_results.names (Elmer), assembles the full S-matrix, writes Touchstone .snp, and optionally performs DC extrapolation and port de-embedding via scikit-rf
  • _build/images/palace/scripts/install.sh — installs the above to ${TOOLS}/palace/bin/ and ${TOOLS}/palace/lib/; the existing install_links.sh auto-symlinks ${TOOLS}/*/bin/*${TOOLS}/bin/, so both scripts land in PATH without further changes

Notes on the gmsh display errors

The Could not generate display lists / XRequest.148: BadValue errors visible in the screenshots are cosmetic X11/OpenGL warnings from gmsh's FLTK window under LIBGL_ALWAYS_INDIRECT=1 (set by start_x.sh in WSL mode). The mesh is generated and written successfully. Users can suppress the GUI entirely with settings['no_gui'] = True in their simulation script, or switch to VNC mode.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/VolkerMuehlhaus/gds2palace_ihp_sg13g2/contents/
    • Triggering command: /usr/bin/curl curl -s REDACTED (http block)
  • https://api.github.com/repos/VolkerMuehlhaus/gds2palace_ihp_sg13g2/contents/workflow
    • Triggering command: /usr/bin/curl curl -s REDACTED (http block)
  • https://api.github.com/repos/VolkerMuehlhaus/gds2palace_ihp_sg13g2/contents/workflow/scripts
    • Triggering command: /usr/bin/curl curl -s REDACTED (http block)
  • https://api.github.com/repos/awslabs/palace/commits/e72ad3b6aa8af6fc7a9e08a5cb1b387cc3c8beba
    • Triggering command: /usr/bin/curl curl -s REDACTED (http block)
  • https://api.github.com/repos/awslabs/palace/contents/
    • Triggering command: /usr/bin/curl curl -s REDACTED (http block)
  • https://api.github.com/repos/awslabs/palace/contents/scripts
    • Triggering command: /usr/bin/curl curl -s REDACTED conntrack --ctstate INVALID,NEW -j DROP (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI linked an issue Apr 20, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Fix mesh generation error in gds2palace integration Add run_palace and combine_snp wrapper scripts for gds2palace integration Apr 20, 2026
Copilot AI requested a review from hpretl April 20, 2026 12:08
@hpretl hpretl changed the base branch from main to next_release April 20, 2026 12:10
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.

gds2palace Integration

2 participants