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

[jsk_perception/doc/vqa_node] add explanation about model argument #2850

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kochigami
Copy link
Contributor

@kochigami kochigami commented Nov 6, 2024

When I executed the below command following https://jsk-docs.readthedocs.io/projects/jsk_recognition/en/latest/jsk_perception/nodes/vqa_node.html#run-inference-container-on-another-host-or-another-terminal

kochigami@kochigami-ThinkPad-P16-Gen-1:~/catkin_ws/src/jsk_recognition/jsk_perception/docker$ ./run_jsk_vil_api --port 8888 --ofa_task caption --ofa_model_scale huge

I got the following error:

usage: run_jsk_vil_api [-h] [-p PORT] [--ofa_task OFA_TASK] [--ofa_model_scale {base,large,huge}] {ofa,clip}
run_jsk_vil_api: error: the following arguments are required: model

It seems that we need to set a model argument, but it seems that doc forgets it.
Thus, I added an explanation about model argument.
I'm not sure whether it is a good explanation, so please check this.

Related issue: #2849

@kochigami kochigami requested a review from mqcmd196 November 6, 2024 08:07
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