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.

Sign up for GitHub

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

Jump to bottom

Add seeds to the randomness-related function #820

Closed
tnemoz opened this issue Oct 2, 2024 · 2 comments · Fixed by #842
Closed

Add seeds to the randomness-related function #820

tnemoz opened this issue Oct 2, 2024 · 2 comments · Fixed by #842
Milestone
v1.1.0

Comments

@tnemoz
Copy link
Contributor

tnemoz commented Oct 2, 2024

Currently, the functions in toqito/rand don't take a seed. It could be useful to add the functionality. This would also allow to remove the skips in the doctests.
@purva-thakre purva-thakre mentioned this issue Oct 2, 2024
Closed
@purva-thakre
Copy link
Collaborator

Yes, this has been on my todos for a while. Thank you for the issue!

@vprusso
Copy link
Owner

vprusso commented Oct 2, 2024

Yes, great capture, @tnemoz ! Agree with @purva-thakre !

@tnemoz tnemoz mentioned this issue Oct 7, 2024
Merged
7 tasks
@purva-thakre purva-thakre added this to the v1.1.0 milestone Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v1.1.0
Development

Successfully merging a pull request may close this issue.

Removed qiskit as a dependency and added seeds to randomness-related functions tnemoz/toqito
3 participants
@vprusso @tnemoz @purva-thakre