Skip to content

parameterization of filters and features benchmarks (v2)#322

Merged
rapids-bot[bot] merged 8 commits intorapidsai:branch-22.08from
grlee77:benchmarks-refactor
Jul 27, 2022
Merged

parameterization of filters and features benchmarks (v2)#322
rapids-bot[bot] merged 8 commits intorapidsai:branch-22.08from
grlee77:benchmarks-refactor

Conversation

@grlee77
Copy link
Copy Markdown
Contributor

@grlee77 grlee77 commented Jun 28, 2022

Update of #278 for branch-22.08 with minor additional fixes. I have tested this locally and it seems to be operating as expected.

Anas Zainul Abidin and others added 5 commits June 28, 2022 16:08
@grlee77 grlee77 added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Jun 28, 2022
@jakirkham
Copy link
Copy Markdown
Member

Looks like there are some conflicts. Could you please take a look at these @grlee77? 🙂

@jakirkham
Copy link
Copy Markdown
Member

@gpucibot merge

@grlee77
Copy link
Copy Markdown
Contributor Author

grlee77 commented Jul 26, 2022

Looks like there are some conflicts. Could you please take a look at these @grlee77? slightly_smiling_face

Resolved. should be good to go now. No point to wait on CI as it doesn't run these benchmarks.

@jakirkham
Copy link
Copy Markdown
Member

Yeah it won't merge until CI passes. So we just need to wait unfortunately. On the bright side it should merge as soon as CI passes

@rapids-bot rapids-bot bot merged commit 3f7069f into rapidsai:branch-22.08 Jul 27, 2022
rapids-bot bot pushed a commit that referenced this pull request Jul 28, 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)

Authors:
  - Gregory Lee (https://github.com/grlee77)
  - https://github.com/aasthajh

Approvers:
  - https://github.com/jakirkham

URL: #324
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants