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

Qubit specific extension to qibo noise model #1158

Closed
jykhoo1987 opened this issue Jan 18, 2024 · 1 comment · Fixed by #1160
Closed

Qubit specific extension to qibo noise model #1158

jykhoo1987 opened this issue Jan 18, 2024 · 1 comment · Fixed by #1160
Labels
enhancement New feature or request

Comments

@jykhoo1987
Copy link
Contributor

The current composite NoiseModel is limited to the same 'depolarizing error' for one- and two-qubit gates across all qubits. It would be good to have the ability to input different values of for different qubits, which is likely the case as well in hardware. The ‘excited population’ should also have this flexibility as well.

@rahula06 @shangtai

@jykhoo1987 jykhoo1987 added the enhancement New feature or request label Jan 18, 2024
@renatomello
Copy link
Contributor

renatomello commented Jan 19, 2024

That's already allowed, because the qubits variable from the NoiseModel.add method allows to specify qubits using integers and tuples. See https://qibo.science/qibo/stable/api-reference/qibo.html#noise. So one already could specify noise per gate and per qubit.

EDIT: the documentation is not very clear that integers are allowed in the qubits parameter, so at least that very small fix should be done.

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

Successfully merging a pull request may close this issue.

2 participants