Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sequence_min_length_ratios for qiskit.transpiler.passes.PadDynamicalDecoupling #13689

Open
Arianvezvaee opened this issue Jan 18, 2025 · 1 comment
Labels
type: feature request New feature or request

Comments

@Arianvezvaee
Copy link

What should we add?

In the old version of PadDynamicalDecoupling there was an option sequence_min_length_ratios that allowed for inserting different sequences with shorter or longer delay times, set by the ratio of the delay time to the idling gap. This seems to have been removed/missing from the new PadDynamicalDecoupling. Is there a way to use this feature in this new version? Thank you!

@Arianvezvaee Arianvezvaee added the type: feature request New feature or request label Jan 18, 2025
@Arianvezvaee
Copy link
Author

Update: The feature can be added by adding the following to the _pad() function, and updating the __init__

`# Ensure idle time is long enough for DD
if time_interval < self._sequence_min_length_ratios * np.sum(self._dd_sequence_lengths[qubit]):
# If idle time is too short, insert only a delay
self._apply_scheduled_op(dag, t_start, Delay(time_interval, dag.unit), qubit)
return

slack = time_interval - np.sum(self._dd_sequence_lengths[qubit])
sequence_gphase = self._sequence_phase
`

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature request New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant