Skip to content

Docs improvements from @jyu00 and rename Latest updates to Changelog#4964

Open
abbycross wants to merge 10 commits intomainfrom
ajc/cleanup-for-jessie
Open

Docs improvements from @jyu00 and rename Latest updates to Changelog#4964
abbycross wants to merge 10 commits intomainfrom
ajc/cleanup-for-jessie

Conversation

@abbycross
Copy link
Copy Markdown
Collaborator

@abbycross abbycross commented Apr 9, 2026

Closes #4937
Closes #4938
Closes #4680
Closes #4959

@abbycross abbycross requested review from a team and jyu00 April 9, 2026 18:33
@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@qiskit-bot
Copy link
Copy Markdown
Contributor

One or more of the following people are relevant to this code:

@abbycross abbycross changed the title Docs improvements from @jyu00 Docs improvements from @jyu00 and rename Latest updates to Changelog Apr 9, 2026
kaelynj
kaelynj previously approved these changes Apr 9, 2026
Copy link
Copy Markdown
Collaborator

@kaelynj kaelynj left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

Copy link
Copy Markdown
Collaborator

@jyu00 jyu00 left a comment

Choose a reason for hiding this comment

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

Some minor adjustment because I didn't test the code first 🙈

Co-authored-by: Jessie Yu <jessieyu@us.ibm.com>
Copy link
Copy Markdown
Collaborator

@jyu00 jyu00 left a comment

Choose a reason for hiding this comment

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

We also need to initialize the duration & sum it up in the loop:

# Schedule the circuit to get more accurate timing
pm = generate_preset_pass_manager(
    target=backend.target,
    optimization_level=0,
    scheduling_method="alap"
)

scheduled_circuits = pm.run(isa_circuits)

init_duration = backend.target["reset"][(0,)].duration
rep_delay = sampler.options.execution.rep_delay or backend.default_rep_delay

circuit_duration = 0

for circuit in scheduled_circuits:
    # Estimate circuit length
    circuit_duration += circuit.estimate_duration(backend.target)

    # Add INIT time
    if sampler.options.execution.init_qubits:
        circuit_duration += init_duration

    # Add rep_delay
    circuit_duration += rep_delay

total_time = 2 + (circuit_duration*shots)
print(f"Total estimated usage is {math.ceil(total_time)} seconds")

Co-authored-by: Jessie Yu <jessieyu@us.ibm.com>
@abbycross
Copy link
Copy Markdown
Collaborator Author

@jyu00 there were a couple small discrepancies between the recent code block you shared and the recent commits you suggested, so I chose the code block as the source of truth - lmk if I should have done otherwise :D

Copy link
Copy Markdown
Collaborator

@jyu00 jyu00 left a comment

Choose a reason for hiding this comment

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

Thanks Abby!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

Rename Latest updates Clarify job splitting Add note about device drift Add code example for cost estimation

4 participants