Skip to content

Conversation

@pranetksharma
Copy link
Collaborator

Summary of changes

QIR requires .build() from autoqasm to work, made the change in relevant places.

@pranetksharma pranetksharma requested a review from ryanhill1 July 24, 2025 21:24
Pranet Sharma added 2 commits July 24, 2025 16:28
- Updated measurement counts retrieval from `measurement_counts()` to `data.get_counts()` in multiple Jupyter notebooks to ensure compatibility with the latest API changes.
- Modified the following notebooks:
  - qbraid-autoqasm-poc.ipynb
  - aws_sim_qiskit_bernstein_vazirani.ipynb
  - ibm_batch_jobs_grovers.ipynb
  - ibm_quantum_jobs_with_runtime.ipynb
  - qbraid_sdk_devices_and_jobs.ipynb
  - qbraid_runtime_qbraid_provider_aws.ipynb
@ryanhill1
Copy link
Member

ryanhill1 commented Jul 24, 2025

Also, be careful with older notebooks: if a notebook specifies a qbraid.__version__ (e.g. 0.7.0), we should not update the syntax unless we’re updating all code cells to match that newer version. get_counts() updates should only be made in notebooks that don’t specify a version, where the assumption is they use the latest version of the SDK.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The deutsch_jozsa_constant function is defined twice in cell 4

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove comments before merge

@pranetksharma pranetksharma requested a review from ryanhill1 July 31, 2025 14:56
@ryanhill1 ryanhill1 merged commit 8278539 into main Jul 31, 2025
1 check passed
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.

3 participants