-
Notifications
You must be signed in to change notification settings - Fork 70
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
Update vista3d to use MONAI components #627
Update vista3d to use MONAI components #627
Conversation
Signed-off-by: Yiheng Wang <[email protected]>
/build |
Signed-off-by: Yiheng Wang <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR, looks good to me.
Did you try with the model weight to see if it can loaded correctly and the results make sense?
/build |
Hi @KumoLiu , |
/build |
/build |
@@ -243,8 +243,8 @@ def _iteration(self, engine: SupervisedEvaluator, batchdata: dict[str, torch.Ten | |||
label_set = np.arange(output_classes).tolist() | |||
label_prompt = torch.tensor(label_set).to(engine.state.device).unsqueeze(-1) | |||
# point prompt is generated withing vista3d,provide empty points | |||
points = torch.zeros(label_prompt.shape[0], 1, 3) | |||
point_labels = -1 + torch.zeros(label_prompt.shape[0], 1) | |||
points = torch.zeros(label_prompt.shape[0], 1, 3).to(inputs.device) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
HI @heyufan1995 , could you help to confirm if the changes here are correct? I got device mismatch error thus changed here, thanks in advance.
Fixes # .
Description
A few sentences describing the changes proposed in this pull request.
Status
Ready/Work in progress/Hold
Please ensure all the checkboxes:
./runtests.sh --codeformat
.version
andchangelog
inmetadata.json
if changing an existing bundle.CONTRIBUTING.md
).monai
,pytorch
andnumpy
are correct inmetadata.json
.eval_metrics
of the provided weights and TorchScript modules.large_file.yml
./home/your_name/
for"bundle_root"
).