Qiskit Metal v0.5: PySide6, Removing GDSPY to GDSTK, PYAEDT v1.0, and many more changes

[shanto268]

What are the issues this pull addresses (issue numbers / links)?

Porting Qiskit Metal from PySide2 to PySide6 so that it can work natively on M* Macs (Apple Silicon).

Issues addressed:

• [#798](Updating qiskit-metal to use PySide6 #798 (comment))
• Python 3.10 only #1001 (comment)
• Installation on MacOS Silicon depending on PySide2 #991 (comment)
• Limit PySide2 load to just the GUI #690 (comment)
• Port Qiskit Metal from PySide2 to PySide6 #908 (comment)

Did you add tests to cover your changes (yes/no)?

No. N/A

Did you update the documentation accordingly (yes/no)?

Yes

Did you read the CONTRIBUTING document (yes/no)?

Yes

Summary

Fixed changes from PR #908 to pass automated CI workflows and tests, ensuring compatibility with updated dependencies, environment configurations, and the latest (11/17/2024) main branch.

Details and comments

0. Built on top of changes made by @obrienpja in PR #908.

1. Changes to QWidget_PlaceholderText and QTableView_AllComponents:

   • Refactored the QWidget_PlaceholderText class to ensure proper initialization and compatibility with PySide6.
   • Resolved issues with placeholder text styling by updating palette usage to align with PySide6 standards.
   • Updated QTableView_AllComponents to correctly inherit and integrate with QWidget_PlaceholderText, ensuring both functionality and visual styling work seamlessly.
   • Fixed initialization errors and ensured that the placeholder label behaves as intended when no components are available.

2. Requirements and environment updates:

   • Updated the environment.yml file to use stable, OS-agnostic dependencies for Python 3.10.
   • Added missing dependencies, such as pyaedt, to ensure a consistent build across different operating systems.
   • Ensured that all dependencies align with the workflows and requirements in the lab's testing environment.

3. Testing and validation:

   • Verified that the changes integrate seamlessly with the existing workflows and CI processes used in our lab.
   • Tested the updated branch against the current main branch workflows to confirm identical behavior.
   • All CI tests now pass successfully across all supported platforms (Windows, macOS, Ubuntu) and Python versions (3.9 and 3.10).

[CLAassistant]

All committers have signed the CLA.

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 3 committers have signed the CLA.

✅ obrienpja
✅ priti-ashvin-shah-ibm
❌ shanto268
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[zlatko-minev]

Thank you, @shanto268! The tests are running. This is a giant PR, so it would be good to have more eyes on it than just you and me. Has anyone else from your lab tried this? I think after merging this, we could bump the Qiskit Metal version from 0.1.5 to 0.2. I just want to make sure it runs.

[PositroniumJS]

pyaedt has been updated, it can be imported again on Mac devices.
Hence, the last tests are fixed in shanto268#9.
@shanto268 @zlatko-minev

[zlatko-minev]

Great, thank you very much.Just checking in on what else we might have left before we feel ready to make the mergeSent from my iPhoneOn Oct 9, 2025, at 12:43 PM, PositroniumJS ***@***.***> wrote:PositroniumJS left a comment (qiskit-community/qiskit-metal#1002) pyaedt has been updated, it can be imported again on Mac devices. Hence, the last tests are fixed in shanto268#9. @shanto268 @zlatko-minev —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>

[SamWolski]

Cheesing appears to be broken when rendering to GDS. It can be bypassed as a workaround by selectively enabling/disabling the cheese and no-cheese layers:

design.renderers.gds.options.update({
	"cheese": {"view_in_file": {"main": {1: False},}},
	"no_cheese": {"view_in_file": {"main": {1: True}}},
})

but then there is obviously no cheesing generated.

I've also noticed that the cell and layer structure of the generated GDS file has changed. The ground_main_1 cell and 1/0 layer used to correspond to the ground plane NOT including any central conductors, but they are both gone. The layer has now been merged into 1/10. I haven't had a chance to look into it more deeply, but I suspect the cheesing algorithm is trying to find the missing cell and/or layer, at which point the code crashes.

The merging of the ground plane and central conductor elements is problematic for those of us who need to do our own cheesing and other post-processing, as the ground plane needs to be treated differently to other metallic elements like central conductors.

[PositroniumJS]

I've also noticed that the cell and layer structure of the generated GDS file has changed. The ground_main_1 cell and 1/0 layer used to correspond to the ground plane NOT including any central conductors, but they are both gone. The layer has now been merged into 1/10. I haven't had a chance to look into it more deeply, but I suspect the cheesing algorithm is trying to find the missing cell and/or layer, at which point the code crashes.

I am not using the cheesing but I had made some comits to port it to gdstk earlier on this PR, so it should still be working I guess. In any case, in the GDS that I am making, I still have the layer 1/0. Are you doing something special? 🤔

[zlatko-minev]

Thank you @shanto268 @PositroniumJS and @SamWolski , I feel we are on good track here. Excited to get close in the next week or two to merge this super-mega pull request and version update into main.

We will then advertise in the Quantum device community and have more people who would like to test it and try it out. After a few small patches, and when we feel it's in a stable in a version that it could be used in a tutorials for next year's quantum device workshop, we can tag it and release it as unofficial new version across the distribution channels. This will be a big moment for the community, the community has been waiting this for maybe two years.

Of course, we will make further improvements for the Quantum device workshop tutorials down the line after that, but it will be good to have this in a stable enough place.

[SamWolski]

I am not using the cheesing but I had made some comits to port it to gdstk earlier on this PR, so it should still be working I guess.

Clean python env created with conda, only qiskit-metal (either main or this PR) installed:

conda create -n <env_name> python=3.10
conda activate <env_name>
pip install <path_to_cloned_repo>

Generate a minimal design that will actually export something:

import qiskit_metal as metal
from qiskit_metal.qlibrary.terminations.launchpad_wb import LaunchpadWirebond

design = metal.designs.DesignPlanar()
_ = LaunchpadWirebond(design, "pad_1", options={},)
design.renderers.gds.export_to_gds("new_metal_test.gds")

On main, this is successful and generates the GDS file as expected.
On this PR, this fails with

  File "<env>/lib/python3.10/site-packages/qiskit_metal/renderers/renderer_gds/make_cheese.py", line 443, in _subtract_holes_from_ground
    ground_cell = next(
StopIteration

[SamWolski]

In any case, in the GDS that I am making, I still have the layer 1/0. Are you doing something special? 🤔

If I run the same minimal script, but bypass the cheesing just before the export with

design.renderers.gds.options.update({
	"cheese": {"view_in_file": {"main": {1: False},}},
	"no_cheese": {"view_in_file": {"main": {1: True}}},
})

The main version creates layers 1/0, 1/10, and 1/99, whereas the PR version generates only 1/10 and 1/99, with the ground plane merged into the 1/10 layer.

[SamWolski]

I've managed to solve both the issues for positive masks; it's just a few lines that were dropped in the port to gdstk. I'll give it my best shot for negative masks and PR my fork with the fixes.

[SamWolski]

Just verified, no changes need to be made for the negative mask (no separate ground plane layer was ever created, and the generated files appear to match). PR is submitted.

[1ucian0]

Suggested change

_debug/

[1ucian0]

Suggested change

	venvz/
	venvz/

[zlatko-minev]

Passing tutorial 1 and 2, with some minor issues to be addressed later:

• GDS: Junction and size / orientation in GDS export. There may be an issue with the rescaling when movign from gdspy to gdstk loader. Have to debug logic.
• GDS: Cheesing? I turned off for the tutorial, was producing some issue. To investigate later.
• GUI QGeometry table change path/poly type filter