Skip to content

Adding error messaging for unsupported tensor shapes #2089

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Apr 27, 2021

Conversation

charliebudd
Copy link
Collaborator

@charliebudd charliebudd commented Apr 26, 2021

Signed-off-by: chaliebudd [email protected]

Address an issue raised in #2077/#2083 where some tensor shapes cause the filtering to silently fail. Errors have been added to explain that these shapes are unsupported.

P.S. A move to JIT compilation will allow us to provide optimised code that is more flexible in this regard.

Status

Ready

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • Integration tests passed locally by running ./runtests.sh -f -u --net --coverage.
  • Quick tests passed locally by running ./runtests.sh --quick --unittests.

@charliebudd charliebudd requested a review from wyli April 26, 2021 11:22
@wyli
Copy link
Contributor

wyli commented Apr 27, 2021

/black

@wyli wyli merged commit 1f2247e into Project-MONAI:master Apr 27, 2021
wyli pushed a commit that referenced this pull request May 26, 2021
* adding error messaging for unsupported tensor shapes

Signed-off-by: chaliebudd <[email protected]>

* [MONAI] python code formatting

Signed-off-by: monai-bot <[email protected]>
wyli pushed a commit that referenced this pull request May 26, 2021
* adding error messaging for unsupported tensor shapes

Signed-off-by: chaliebudd <[email protected]>

* [MONAI] python code formatting

Signed-off-by: monai-bot <[email protected]>
wyli pushed a commit that referenced this pull request May 26, 2021
* adding error messaging for unsupported tensor shapes

Signed-off-by: chaliebudd <[email protected]>

* [MONAI] python code formatting

Signed-off-by: monai-bot <[email protected]>
wyli pushed a commit that referenced this pull request May 27, 2021
* adding error messaging for unsupported tensor shapes

Signed-off-by: chaliebudd <[email protected]>

* [MONAI] python code formatting

Signed-off-by: monai-bot <[email protected]>
wyli pushed a commit that referenced this pull request May 27, 2021
* adding error messaging for unsupported tensor shapes

Signed-off-by: chaliebudd <[email protected]>

* [MONAI] python code formatting

Signed-off-by: monai-bot <[email protected]>
wyli pushed a commit that referenced this pull request May 27, 2021
* adding error messaging for unsupported tensor shapes

Signed-off-by: chaliebudd <[email protected]>

* [MONAI] python code formatting

Signed-off-by: monai-bot <[email protected]>
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