Skip to content

Conversation

@bhashemian
Copy link
Member

@bhashemian bhashemian commented Sep 12, 2021

Description

This PR implements a wrapper for cuCIM transforms to be used in combination with MONAI transforms.

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • New tests added to cover the changes.
  • Integration tests passed locally by running ./runtests.sh -f -u --net --coverage.
  • Quick tests passed locally by running ./runtests.sh --quick --unittests.
  • In-line docstrings updated.
  • Documentation updated, tested make html command in the docs/ folder.

Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
@bhashemian
Copy link
Member Author

@wyli @Nic-Ma do you know why the min tests are failing?
https://github.com/Project-MONAI/MONAI/pull/2932/checks?check_run_id=3591023115#step:8:115

Is it because of following dependency?

if TYPE_CHECKING:
from cupy import ndarray as cp_ndarray
else:
cp_ndarray, _ = optional_import("cupy", name="ndarray")

Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
@Nic-Ma
Copy link
Contributor

Nic-Ma commented Sep 14, 2021

Hi @wyli ,

I remember we faced this error before, but I can't clearly remember the details.. Do you have any idea?

Thanks in advance.

@wyli
Copy link
Contributor

wyli commented Sep 14, 2021

@wyli @Nic-Ma do you know why the min tests are failing?
https://github.com/Project-MONAI/MONAI/pull/2932/checks?check_run_id=3591023115#step:8:115

Is it because of following dependency?

if TYPE_CHECKING:
from cupy import ndarray as cp_ndarray
else:
cp_ndarray, _ = optional_import("cupy", name="ndarray")

yes, cp_ndarray can't be used as a type annotation (such as def __call__(self, data: cp_ndarray):) because it'll be evaluated on some user environment and raise an error when there's no cupy installation.

Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
@bhashemian bhashemian mentioned this pull request Sep 14, 2021
5 tasks
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
@bhashemian bhashemian marked this pull request as ready for review September 16, 2021 02:25
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
@bhashemian bhashemian requested review from Nic-Ma and wyli September 16, 2021 02:34
Signed-off-by: Wenqi Li <wenqil@nvidia.com>
@wyli wyli enabled auto-merge (squash) September 16, 2021 13:18
@wyli wyli merged commit 38403ed into Project-MONAI:dev Sep 16, 2021
@bhashemian bhashemian deleted the cucim-transform branch September 16, 2021 15:08
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.

3 participants