Skip to content

ROI Inference pipeline for HoVerNet #1055

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 33 commits into from
Dec 8, 2022
Merged

ROI Inference pipeline for HoVerNet #1055

merged 33 commits into from
Dec 8, 2022

Conversation

bhashemian
Copy link
Member

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
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
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