Skip to content

complete parametrization of cucim.skimage benchmarks#324

Merged
rapids-bot[bot] merged 25 commits intorapidsai:branch-22.08from
grlee77:benchmarks-refactor-2
Jul 28, 2022
Merged

complete parametrization of cucim.skimage benchmarks#324
rapids-bot[bot] merged 25 commits intorapidsai:branch-22.08from
grlee77:benchmarks-refactor-2

Conversation

@grlee77
Copy link
Copy Markdown
Contributor

@grlee77 grlee77 commented Jun 29, 2022

This PR expands the approach of #322 to the remaining cucim.skimage benchmarks as well

This PR is built on top of a few other open benchmarking PRs, so please review and merge those first: (#322, #323, #290)

@grlee77 grlee77 added improvement Improves an existing functionality non-breaking Introduces a non-breaking change performance Performance improvement labels Jun 29, 2022
@grlee77 grlee77 added this to the v22.08.00 milestone Jun 29, 2022
@jakirkham
Copy link
Copy Markdown
Member

Looks like there are some conflicts here. Could you please resolve these @grlee77? 🙂

@grlee77
Copy link
Copy Markdown
Contributor Author

grlee77 commented Jul 27, 2022

Fixed conflicts, added a requirements-bench.txt as discussed in #323 (review).

@grlee77
Copy link
Copy Markdown
Contributor Author

grlee77 commented Jul 27, 2022

might have to resolve conflicts again if #323 is merged (although all changes from #323 are also present here if you want to merge only this one)

@jakirkham
Copy link
Copy Markdown
Member

Would it make sense to close that PR then?

@grlee77
Copy link
Copy Markdown
Contributor Author

grlee77 commented Jul 27, 2022

Yeah, it is fine to close that one.

@grlee77
Copy link
Copy Markdown
Contributor Author

grlee77 commented Jul 27, 2022

Just fixed a few new conflicts here. Should be good to go now

@jakirkham
Copy link
Copy Markdown
Member

@gpucibot merge

@rapids-bot rapids-bot bot merged commit d03e03f into rapidsai:branch-22.08 Jul 28, 2022
@jakirkham
Copy link
Copy Markdown
Member

Thanks Greg! 🙏

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

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change performance Performance improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants