Skip to content

Remove widget from client (DO NOT MERGE - NEED DEPRECATION FIRST)#2229

Open
ItamarGoldman wants to merge 4 commits into
mainfrom
remove_widget_from_client
Open

Remove widget from client (DO NOT MERGE - NEED DEPRECATION FIRST)#2229
ItamarGoldman wants to merge 4 commits into
mainfrom
remove_widget_from_client

Conversation

@ItamarGoldman

@ItamarGoldman ItamarGoldman commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Summary

As part of the work in refactoring the client, removing the widget from the client.

Implementation Checklist

  • Remove client/qiskit_serverless/visualization/ directory
  • Remove widget() method from BaseClient
  • Check if ipywidgets dependency can be removed from requirements
  • Add to release notes as breaking change
  • Run full test suite to ensure no hidden dependencies

Details and comments

Files to Remove

1. Widget Module Files

Files to Update

1. BaseClient Class

File: client/qiskit_serverless/core/client.py:191-198

Action: Delete lines 187-198 (entire widget section)

Impact Assessment

✅ No Impact Areas

  1. Main Package API - client/qiskit_serverless/__init__.py

    • Widget is NOT exported in the main package
    • No changes needed to package imports
  2. Documentation - docs/

    • No documentation references to widget found
    • No .rst files mention widget or visualization
  3. Tests - client/tests/

    • No test files reference widget or visualization
    • No test coverage to remove
  4. Dependencies

    • Widget uses ipywidgets but this is likely already a dependency
    • Check requirements.txt or setup.py to see if ipywidgets can be removed

@ItamarGoldman ItamarGoldman changed the title Remove widget from client [WIP] Remove widget from client Jun 11, 2026
@ItamarGoldman ItamarGoldman changed the title [WIP] Remove widget from client Remove widget from client Jun 11, 2026
@ItamarGoldman ItamarGoldman marked this pull request as ready for review June 11, 2026 13:47
@ItamarGoldman ItamarGoldman requested a review from a team as a code owner June 11, 2026 13:47
@ItamarGoldman ItamarGoldman changed the title Remove widget from client Remove widget from client (DO NOT MERGE - NEED DEPRECATION FIRST) Jun 11, 2026
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.

1 participant