Skip to content
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

error in Pre-Trained Segmentation Models #1239

Merged
merged 1 commit into from
Oct 4, 2021
Merged

Conversation

maximofn
Copy link
Contributor

@maximofn maximofn commented Oct 1, 2021

CLI Argument for Multi-Human must be fcn-resnet18-mhp-640x360 not fcn-resnet18-mhp-512x320

CLI Argument for Multi-Human must be fcn-resnet18-mhp-640x360 not fcn-resnet18-mhp-512x320
@dusty-nv
Copy link
Owner

dusty-nv commented Oct 1, 2021

fcn-resnet18-mhp-640x360 argument should work:

else if( strcasecmp(modelName, "fcn-resnet18-mhp-640x360") == 0 )

Maybe it's just that you needed to download the model?

@maximofn
Copy link
Contributor Author

maximofn commented Oct 4, 2021

Sorry, I think I have explained myself wrong, what I mean is that in the Pre-Trained Segmentation Models Available table it is written twice fcn-resnet18-mhp-512x320, but the second would have to be fcn-resnet18-mhp-640x360

@dusty-nv dusty-nv merged commit 9bf5549 into dusty-nv:master Oct 4, 2021
@dusty-nv
Copy link
Owner

dusty-nv commented Oct 4, 2021

Oh I see now - thank you @maximofn for fixing that in the docs!

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.

2 participants