Skip to content

Fix netlist regression tests for gdsfactory 9.32.0 compatibility#223

Merged
nikosavola merged 2 commits intodependabot/pip/gdsfactory-gte-9.29.1-and-lt-9.32.1from
copilot/sub-pr-222
Jan 26, 2026
Merged

Fix netlist regression tests for gdsfactory 9.32.0 compatibility#223
nikosavola merged 2 commits intodependabot/pip/gdsfactory-gte-9.29.1-and-lt-9.32.1from
copilot/sub-pr-222

Conversation

Copy link
Contributor

Copilot AI commented Jan 23, 2026 •
edited
Loading

gdsfactory 9.32.0 changed netlist serialization: instance names are now simplified (no hash suffixes) and the top-level name field is removed.

Changes

  • Regenerated 35 netlist regression test reference files in tests/test_pdk/ to match new naming format

Example diff in test_netlists_double_pad_transmon_with_bbox_.yml:

# Before (gdsfactory <9.32.0)
instances:
  double_pad_transmon_PS250_400_PG15_JSsquid_junction_JDN_cc5ec781_0_0:
    component: double_pad_transmon
name: double_pad_transmon_with_bbox_BE200_PS250_400_PG15_JSsq_14ec26ba

# After (gdsfactory >=9.32.0)
instances:
  double_pad_transmon:
    component: double_pad_transmon
# 'name' field removed

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI mentioned this pull request Jan 23, 2026
Merged
@nikosavola
Regenerate netlist regression test files for gdsfactory 9.32.x compat…
6723afe 
…ibility

Co-authored-by: nikosavola <7860886+nikosavola@users.noreply.github.com>
Copilot AI changed the title [WIP] Update gdsfactory requirement to allow latest version Fix netlist regression tests for gdsfactory 9.32.0 compatibility Jan 23, 2026
Copilot AI requested a review from nikosavola January 23, 2026 06:49
@nikosavola nikosavola marked this pull request as ready for review January 26, 2026 08:01
@github-actions github-actions bot added the tests Relating to testing label Jan 26, 2026
@nikosavola nikosavola merged commit 532c640 into dependabot/pip/gdsfactory-gte-9.29.1-and-lt-9.32.1 Jan 26, 2026
16 checks passed
@nikosavola nikosavola deleted the copilot/sub-pr-222 branch January 26, 2026 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nikosavola nikosavola Awaiting requested review from nikosavola

Assignees

@nikosavola nikosavola

Copilot code review Copilot

Labels

tests Relating to testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@nikosavola