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

Jump to bottom

sort: avoid calling rand.Seed in test helpers #61142

Open
suyashtava opened this issue Jul 2, 2023 · 4 comments · May be fixed by #61143
Open

sort: avoid calling rand.Seed in test helpers #61142

suyashtava opened this issue Jul 2, 2023 · 4 comments · May be fixed by #61143
Labels
help wanted NeedsFix The path to resolution is known, but the work has not been done. Testing An issue that has been verified to require only test changes, not just a test failure.
Milestone
Go1.22

Comments

@suyashtava
Copy link

sort_benchmark_go
is using rand.Seed() which is depreciated, and we should be using latest procedure(Doc) for benchmarking.
@gopherbot gopherbot added this to the Proposal milestone Jul 2, 2023
@suyashtava suyashtava linked a pull request Jul 2, 2023 that will close this issue
Open
@suyashtava
Copy link
Author

Solving PR: #61143

@mvdan
Copy link
Member

mvdan commented Jul 2, 2023

This does not need to be a proposal.

@mvdan mvdan changed the title proposal: sort benchmark test is still using deprecated function. sort: benchmark test is still using deprecated function. Jul 2, 2023
@mvdan mvdan removed this from the Proposal milestone Jul 2, 2023
@mvdan mvdan removed the Proposal label Jul 2, 2023
@suyashtava
Copy link
Author

suyashtava commented Jul 2, 2023 •
edited

This does not need to be a proposal.

Contributing to go lang for the first time, might have misunderstood some guideline.

Can you pls also check the attached pr.

@bcmills bcmills added Documentation Testing An issue that has been verified to require only test changes, not just a test failure. help wanted NeedsFix The path to resolution is known, but the work has not been done. and removed Documentation labels Jul 5, 2023
@bcmills bcmills added this to the Backlog milestone Jul 5, 2023
@bcmills bcmills changed the title sort: benchmark test is still using deprecated function. sort: avoid calling rand.Seed in makeRandomInts test helper Jul 5, 2023
@bcmills bcmills changed the title sort: avoid calling rand.Seed in makeRandomInts test helper sort: avoid calling rand.Seed in test helpers Jul 5, 2023
@gopherbot
Copy link

Change https://go.dev/cl/507575 mentions this issue: slices: in benchmarks remove call to rand.Seed

@dmitshur dmitshur modified the milestones: Backlog, Go1.21 Jul 7, 2023
@gopherbot gopherbot modified the milestones: Go1.21, Go1.22 Aug 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted NeedsFix The path to resolution is known, but the work has not been done. Testing An issue that has been verified to require only test changes, not just a test failure.
Projects
None yet
Milestone
Go1.22
Development

Successfully merging a pull request may close this issue.

slices: in benchmarks remove call to rand.Seed suyashtava/go
5 participants
@dmitshur @mvdan @bcmills @gopherbot @suyashtava