Skip to content

ROI Inference pipeline for HoVerNet#1055

Merged
bhashemian merged 33 commits intoProject-MONAI:mainfrom
bhashemian:wsi-inference-hovernet
Dec 8, 2022
Merged

ROI Inference pipeline for HoVerNet#1055
bhashemian merged 33 commits intoProject-MONAI:mainfrom
bhashemian:wsi-inference-hovernet

Conversation

@bhashemian
Copy link
Copy Markdown
Member

@bhashemian bhashemian commented Nov 17, 2022

Fixes Project-MONAI/MONAI#5539

Description

A few sentences describing the changes proposed in this pull request.

Checks

  • Notebook runs automatically ./runner [-p <regex_pattern>]

@bhashemian
Add a draft of inference
979e05a 
Signed-off-by: Behrooz <[email protected]>
@bhashemian
Uncomment load weights
d3b77de 
Signed-off-by: Behrooz <[email protected]>
@bhashemian
Add infer_roi
fc271ef 
Signed-off-by: Behrooz <[email protected]>
@bhashemian
Major updates
6582442 
Signed-off-by: Behrooz <[email protected]>
@bhashemian bhashemian mentioned this pull request Nov 18, 2022
Merged
2 tasks
@bhashemian bhashemian mentioned this pull request Nov 21, 2022
Merged
6 tasks
@bhashemian bhashemian marked this pull request as ready for review November 21, 2022 18:18
@bhashemian
Remove test-oly lines
e58050c 
Signed-off-by: Behrooz <[email protected]>
@bhashemian
Add sw batch size
123b721 
Signed-off-by: Behrooz <[email protected]>
@bhashemian
Change settings:
04c1f45 
Signed-off-by: Behrooz <[email protected]>
Nic-Ma
Nic-Ma reviewed Nov 22, 2022
View reviewed changes
Nic-Ma
Nic-Ma reviewed Nov 22, 2022
View reviewed changes
Copy link
Copy Markdown
Contributor

@Nic-Ma Nic-Ma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as @KumoLiu 's PR, I think we should put a README doc for these python code.

Thanks.

KumoLiu
KumoLiu reviewed Nov 22, 2022
View reviewed changes
@bhashemian
Copy link
Copy Markdown
Member Author

bhashemian commented Nov 22, 2022 •
edited
Loading

Same as @KumoLiu 's PR, I think we should put a README doc for these python code.

Thanks.

I will update the same README in #999 is (after it merged) as they are going to be in the same folder.

@bhashemian bhashemian requested a review from KumoLiu December 8, 2022 02:17
@bhashemian bhashemian requested review from Nic-Ma and wyli December 8, 2022 15:21
@bhashemian
Remove device
d03067a 
Signed-off-by: Behrooz <[email protected]>
@bhashemian
few improvements
0450963 
Signed-off-by: Behrooz <[email protected]>
@bhashemian
improvments and bug fix
56d2e98 
Signed-off-by: Behrooz <[email protected]>
@bhashemian
Update default run
8df9008 
Signed-off-by: Behrooz <[email protected]>
@bhashemian bhashemian enabled auto-merge (squash) December 8, 2022 20:07
@bhashemian bhashemian merged commit 430cc81 into Project-MONAI:main Dec 8, 2022
@bhashemian bhashemian deleted the wsi-inference-hovernet branch December 8, 2022 20:36
boneseva pushed a commit to boneseva/MONAI-tutorials that referenced this pull request Apr 21, 2024
@bhashemian
ROI Inference pipeline for HoVerNet (Project-MONAI#1055)
8de3193 
Fixes Project-MONAI/MONAI#5539

### Description
A few sentences describing the changes proposed in this pull request.

### Checks
<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [ ] Notebook runs automatically `./runner [-p <regex_pattern>]`

Signed-off-by: Behrooz <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@KumoLiu KumoLiu KumoLiu approved these changes

@SachidanandAlle SachidanandAlle Awaiting requested review from SachidanandAlle

@Nic-Ma Nic-Ma Awaiting requested review from Nic-Ma

@wyli wyli Awaiting requested review from wyli

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Pathology HoVerNet: Inference pipeline (large ROI)

3 participants

@bhashemian @Nic-Ma @KumoLiu