Skip to content

Conversation

@GPUtester
Copy link
Contributor

❄️ Code freeze for branch-22.10 and v22.10 release

What does this mean?

Only critical/hotfix level issues should be merged into branch-22.10 until release (merging of this PR).

What is the purpose of this PR?

  • Update documentation
  • Allow testing for the new release
  • Enable a means to merge branch-22.10 into main for the release

raydouglass and others added 26 commits July 22, 2022 10:54
[gpuCI] Forward-merge branch-22.08 to branch-22.10 [skip gpuci]
[gpuCI] Forward-merge branch-22.08 to branch-22.10 [skip gpuci]
[gpuCI] Forward-merge branch-22.08 to branch-22.10 [skip gpuci]
[gpuCI] Forward-merge branch-22.08 to branch-22.10 [skip gpuci]
[gpuCI] Forward-merge branch-22.08 to branch-22.10 [skip gpuci]
[gpuCI] Forward-merge branch-22.08 to branch-22.10 [skip gpuci]
[gpuCI] Forward-merge branch-22.08 to branch-22.10 [skip gpuci]
[gpuCI] Forward-merge branch-22.08 to branch-22.10 [skip gpuci]
[gpuCI] Forward-merge branch-22.08 to branch-22.10 [skip gpuci]
[gpuCI] Forward-merge branch-22.08 to branch-22.10 [skip gpuci]
[gpuCI] Forward-merge branch-22.08 to branch-22.10 [skip gpuci]
[gpuCI] Forward-merge branch-22.08 to branch-22.10 [skip gpuci]
[gpuCI] Forward-merge branch-22.08 to branch-22.10 [skip gpuci]
These functions had been implemented, but were missing from the `__init__.py` in `skimage.measure`. This was not caught because the test cases had imported them from the private submodule!

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

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

URL: #386
[gpuCI] Forward-merge branch-22.08 to branch-22.10 [skip gpuci]
Forward-merge branch-22.08 to branch-22.10 [skip gpuci]
Fixes #397
Fixes #395

* Symlinks top-level README and CHANGELOG into `python/cucim/`
* Adds links to the developer page on PyPI and Anaconda
* Also adds links for docs, source, etc.

cc @grlee77 @gigony

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

Approvers:
  - Gigon Bae (https://github.com/gigony)
  - Joseph (https://github.com/jolorunyomi)

URL: #415
…rable convolution (#409)

closes #408

An error in the keyword arguments passed to the shared memory calculation is fixed in the first commit here. In the second commit, a second fallback is added in case of any other anticipated compilation failure.

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

Approvers:
  - https://github.com/jakirkham
  - Gigon Bae (https://github.com/gigony)

URL: #409
closes #193

This PR adds `cucim.skimage.feature.match_descriptors`. It is a very straightforward adaptation of the scikit-image code, only substituting numpy->cupy. The only differences of note are:
- when new enough CuPy and pylibraft are not available it warns and falls back to `cdist` on the CPU
- test cases involving BRIEF temporarily involve round trip to the host until we implement BRIEF here (looks not too hard)

This PR also removes the deprecated `masked_register_translation` and `register_translation` (these were moved to cucim.skimage.registration.phase_cross_correlation`) from the `feature` module. Removing those was missed when previously updating the API here to match scikit-image 0.19.

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

Approvers:
  - Gigon Bae (https://github.com/gigony)

URL: #338
This PR adds benchmarks for a few of the recently introduced functions.

It also uses more realistic nD label object generation for the segmentation benchmarks.

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

Approvers:
  - Gigon Bae (https://github.com/gigony)
  - https://github.com/jakirkham

URL: #378
@GPUtester GPUtester requested review from a team as code owners October 7, 2022 15:03
@raydouglass raydouglass merged commit f1a7764 into main Oct 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants