Skip to content
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

Add transform to handle empty box as training data #6170

Merged
merged 25 commits into from
Mar 25, 2023

Conversation

Can-Zhao
Copy link
Collaborator

@Can-Zhao Can-Zhao commented Mar 18, 2023

Fixes #5990 .

Description

Add transforms to convert empty box with shape (0,M) or (0,) into (0,4) or (0,6).
Provide format checking inside detector so users can know how to solve the format issue with empty box input.

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • Breaking change (fix or new feature that would cause existing functionality to change).
  • 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 --disttests.
  • In-line docstrings updated.
  • Documentation updated, tested make html command in the docs/ folder.

Can-Zhao and others added 12 commits March 18, 2023 05:16
Signed-off-by: Can Zhao <canz@nvidia.com>
Signed-off-by: Can Zhao <canz@nvidia.com>
…ng, especially when input box is empty

Signed-off-by: Can-Zhao <volcanofly@gmail.com>
for more information, see https://pre-commit.ci

Signed-off-by: Can-Zhao <volcanofly@gmail.com>
Signed-off-by: Can-Zhao <volcanofly@gmail.com>
Signed-off-by: Can-Zhao <volcanofly@gmail.com>
Signed-off-by: Can-Zhao <volcanofly@gmail.com>
Signed-off-by: Can-Zhao <volcanofly@gmail.com>
Signed-off-by: Can-Zhao <volcanofly@gmail.com>
Signed-off-by: Can-Zhao <volcanofly@gmail.com>
@Can-Zhao Can-Zhao force-pushed the negbox branch 3 times, most recently from ed52ac9 to 7a92b22 Compare March 22, 2023 18:48
Can-Zhao and others added 4 commits March 22, 2023 11:48
Signed-off-by: Can-Zhao <volcanofly@gmail.com>
Signed-off-by: Can-Zhao <volcanofly@gmail.com>
@Can-Zhao Can-Zhao marked this pull request as ready for review March 22, 2023 19:36
@Can-Zhao Can-Zhao requested review from wyli and Nic-Ma March 22, 2023 20:02
Can-Zhao and others added 7 commits March 23, 2023 09:31
Signed-off-by: Can-Zhao <volcanofly@gmail.com>
Signed-off-by: Can-Zhao <volcanofly@gmail.com>
Signed-off-by: Can-Zhao <volcanofly@gmail.com>
Signed-off-by: Can-Zhao <volcanofly@gmail.com>
Signed-off-by: Can-Zhao <volcanofly@gmail.com>
@wyli
Copy link
Contributor

wyli commented Mar 24, 2023

/build

@wyli wyli enabled auto-merge (squash) March 24, 2023 11:56
@wyli
Copy link
Contributor

wyli commented Mar 24, 2023

/build

1 similar comment
@wyli
Copy link
Contributor

wyli commented Mar 25, 2023

/build

@wyli wyli merged commit c0aace5 into Project-MONAI:dev Mar 25, 2023
@Can-Zhao Can-Zhao deleted the negbox branch March 25, 2023 19:28
jak0bw pushed a commit to jak0bw/MONAI that referenced this pull request Mar 28, 2023
Fixes Project-MONAI#5990 .

### Description

Add transforms to convert empty box with shape (0,M) or (0,) into (0,4)
or (0,6).
Provide format checking inside detector so users can know how to solve
the format issue with empty box input.

### Types of changes
<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [x] Non-breaking change (fix or new feature that would not break
existing functionality).
- [ ] Breaking change (fix or new feature that would cause existing
functionality to change).
- [x] 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 --disttests`.
- [ ] In-line docstrings updated.
- [ ] Documentation updated, tested `make html` command in the `docs/`
folder.

---------

Signed-off-by: Can Zhao <canz@nvidia.com>
jak0bw pushed a commit to jak0bw/MONAI that referenced this pull request Mar 28, 2023
Fixes Project-MONAI#5990 .

### Description

Add transforms to convert empty box with shape (0,M) or (0,) into (0,4)
or (0,6).
Provide format checking inside detector so users can know how to solve
the format issue with empty box input.

### Types of changes
<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [x] Non-breaking change (fix or new feature that would not break
existing functionality).
- [ ] Breaking change (fix or new feature that would cause existing
functionality to change).
- [x] 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 --disttests`.
- [ ] In-line docstrings updated.
- [ ] Documentation updated, tested `make html` command in the `docs/`
folder.

---------

Signed-off-by: Can Zhao <canz@nvidia.com>
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.

ConvertBoxToStandardModed cannot handle empty boxes
2 participants