Skip to content

5345 update output devices slidingwindow#5346

Merged
wyli merged 4 commits intoProject-MONAI:devfrom
wyli:5345-device-sliding-window
Oct 17, 2022
Merged

5345 update output devices slidingwindow#5346
wyli merged 4 commits intoProject-MONAI:devfrom
wyli:5345-device-sliding-window

Conversation

@wyli
Copy link
Copy Markdown
Contributor

@wyli wyli commented Oct 17, 2022

Signed-off-by: Wenqi Li [email protected]

Fixes #5345

sliding window output should prefer device instead of inputs.device

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.

Signed-off-by: Wenqi Li <[email protected]>
@wyli wyli force-pushed the 5345-device-sliding-window branch from 37408c5 to 3563e41 Compare October 17, 2022 14:56
Signed-off-by: Wenqi Li <[email protected]>
@wyli wyli force-pushed the 5345-device-sliding-window branch from 3563e41 to 135ae66 Compare October 17, 2022 15:02
Signed-off-by: Wenqi Li <[email protected]>
@wyli wyli requested a review from Nic-Ma October 17, 2022 15:09
@wyli wyli marked this pull request as ready for review October 17, 2022 15:09
@wyli wyli requested review from KumoLiu, ericspod and rijobro October 17, 2022 15:19
@wyli
Copy link
Copy Markdown
Contributor Author

wyli commented Oct 17, 2022

/build

@wyli wyli merged commit cd2f4e1 into Project-MONAI:dev Oct 17, 2022
@wyli wyli deleted the 5345-device-sliding-window branch October 17, 2022 18:00
wyli added a commit that referenced this pull request Oct 17, 2022
Signed-off-by: Wenqi Li <[email protected]>

Fixes #5345

sliding window output should prefer `device` instead of `inputs.device`


### 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).
- [ ] New tests added to cover the changes.
- [ ] Integration tests passed locally by running `./runtests.sh -f -u
--net --coverage`.
- [x] 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: Wenqi Li <[email protected]>
wyli added a commit that referenced this pull request Oct 18, 2022
Signed-off-by: Wenqi Li <[email protected]>

Fixes #5345

sliding window output should prefer `device` instead of `inputs.device`


### 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).
- [ ] New tests added to cover the changes.
- [ ] Integration tests passed locally by running `./runtests.sh -f -u
--net --coverage`.
- [x] 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: Wenqi Li <[email protected]>
KumoLiu pushed a commit that referenced this pull request Nov 2, 2022
Signed-off-by: Wenqi Li <[email protected]>

Fixes #5345

sliding window output should prefer `device` instead of `inputs.device`


### 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).
- [ ] New tests added to cover the changes.
- [ ] Integration tests passed locally by running `./runtests.sh -f -u
--net --coverage`.
- [x] 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: Wenqi Li <[email protected]>
Signed-off-by: KumoLiu <[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.

enhance the sliding window devices

2 participants