-
Notifications
You must be signed in to change notification settings - Fork 68
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
TRT support for MAISI #701
base: dev
Are you sure you want to change the base?
Conversation
Signed-off-by: Boris Fomitchev <[email protected]>
Signed-off-by: Boris Fomitchev <[email protected]>
Signed-off-by: Boris Fomitchev <[email protected]>
Signed-off-by: Boris Fomitchev <[email protected]>
Signed-off-by: Boris Fomitchev <[email protected]>
Signed-off-by: Boris Fomitchev <[email protected]>
Signed-off-by: Boris Fomitchev <[email protected]>
Signed-off-by: Boris Fomitchev <[email protected]>
Hi @yiheng-wang-nv , Is the CI pipeline broken? Thanks. |
Signed-off-by: Yiheng Wang <[email protected]>
for more information, see https://pre-commit.ci
Signed-off-by: Yiheng Wang <[email protected]>
@@ -34,8 +34,8 @@ | |||
"spatial_dims": 3, | |||
"image_channels": 1, | |||
"latent_channels": 4, | |||
"output_size_xy": 512, |
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 @borisfom , could you revert this size change? The default values are set by MAISI author
@@ -241,7 +241,6 @@ def ldm_conditional_sample_one_image( | |||
# synthesize latents | |||
noise_scheduler.set_timesteps(num_inference_steps=num_inference_steps) | |||
for t in tqdm(noise_scheduler.timesteps, ncols=110): | |||
# Get controlnet output |
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.
same here, the comment may be useful, could you revert the changes? Thanks!
Hi @KumoLiu , just FYI, the MAISI tensorrt enhancement PR contains the content of this PR: Project-MONAI/MONAI#8153 We may need to merge this one first before merging the MAISI one |
Signed-off-by: Boris Fomitchev <[email protected]>
Hi @borisfom ,
Thanks, |
Should be fine for MAISI as I tested.
Thanks, |
@binliunls : how come image_decoding is much slower with TRT? How do I run a test for that ? |
I was running the command line like |
Description
TRT optimization support for MAISI.
Depends on Project-MONAI/MONAI#8153
Status
Work in progress