Skip to content

fit to qop350#346

Open
4hsiangQM wants to merge 27 commits intomainfrom
350fetching
Open

fit to qop350#346
4hsiangQM wants to merge 27 commits intomainfrom
350fetching

Conversation

@4hsiangQM
Copy link
Copy Markdown
Contributor

This PR updates the quantum control application to use the new qop350 API and change config file.

  1. Replaced all fetching_tool from the qualang with the new API’s fetch_result.
  2. Updated configuration file: change config to full_config = controller_config | logical_config

Copy link
Copy Markdown
Contributor

@KevinAVR KevinAVR left a comment

Choose a reason for hiding this comment

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

Jack and I discussed during the quarter. Almost all files are changing.

The change were to configs that apply to OPX1k, but OPX+ and Octaves config are not changes.

Program files have the new fetching tool, in replace to the qualang-tools fetching tool.

There are likely bugs but they need to be corrected as we go with CSPs or customer interaction.

@TheoLaudatQM
Copy link
Copy Markdown
Collaborator

Is the fetching tool working for both OPX+ and OPX1000?
Also, I am not very comfortable with

There are likely bugs but they need to be corrected as we go with CSPs or customer interaction.
In my opinion it is problematic if we can ensure that the scripts are working, and I would rather not push it until it has been tested.
@KevinAVR what makes you think that things are broken?

counts_handle.wait_for_values(1)
time = []
counts = []
res_handles = job.result_handles
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Duplicated with line 65

data_list = ["I_g", "Q_g", "I_e", "Q_e"]
# Waits (blocks the Python console) until all results have been acquired
res_handles.wait_for_all_values()
results = res_handles.fetch_results(wait_until_done=False, timeout=60)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Based on the commands and the script, I'd say that wait_until_done should be True, no?

@4hsiangQM
Copy link
Copy Markdown
Contributor Author

截圖 2025-10-02 下午3 26 01

@4hsiangQM
Copy link
Copy Markdown
Contributor Author

截圖 2025-10-02 晚上8 09 32

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