Skip to content

Numpy Implementation of SplitOnGrid#3378

Merged
bhashemian merged 12 commits into
Project-MONAI:devfrom
bhashemian:split-on-grid-numpy
Nov 22, 2021
Merged

Numpy Implementation of SplitOnGrid#3378
bhashemian merged 12 commits into
Project-MONAI:devfrom
bhashemian:split-on-grid-numpy

Conversation

@bhashemian

@bhashemian bhashemian commented Nov 22, 2021

Copy link
Copy Markdown
Member

Fixes #3376

Description

This PR implements numpy version of SplitOnGrid and update unittests for numpy.ndarray andtorch.Tensor on both CPU and CUDA.

Status

Ready

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.

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 requested review from Nic-Ma and wyli November 22, 2021 03:12
Comment thread monai/apps/pathology/transforms/spatial/array.py Outdated
Comment thread monai/apps/pathology/transforms/spatial/array.py Outdated
Comment thread tests/test_split_on_grid.py Outdated
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 requested a review from Nic-Ma November 22, 2021 04:08
@bhashemian

Copy link
Copy Markdown
Member Author

@Nic-Ma thanks for reviewing this. Do you have any other comments?

@bhashemian bhashemian enabled auto-merge (squash) November 22, 2021 04:39
@bhashemian

Copy link
Copy Markdown
Member Author

@Nic-Ma @wyli can you run blossom ci here?

@Nic-Ma

Nic-Ma commented Nov 22, 2021

Copy link
Copy Markdown
Contributor

/build

Comment thread monai/apps/pathology/transforms/spatial/dictionary.py Outdated
Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
@Nic-Ma

Nic-Ma commented Nov 22, 2021

Copy link
Copy Markdown
Contributor

/build

@Nic-Ma Nic-Ma disabled auto-merge November 22, 2021 15:47
@wyli

wyli commented Nov 22, 2021

Copy link
Copy Markdown
Contributor

/black

@bhashemian bhashemian merged commit 5adf30b into Project-MONAI:dev Nov 22, 2021
@bhashemian bhashemian deleted the split-on-grid-numpy branch November 23, 2021 02:54
@wyli wyli mentioned this pull request Nov 23, 2021
4 tasks
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.

Add numpy support to SplitOnGrid

3 participants