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

Problems when running the experiment with docker #284

Closed
DeeperMind opened this issue May 9, 2019 · 4 comments
Closed

Problems when running the experiment with docker #284

DeeperMind opened this issue May 9, 2019 · 4 comments

Comments

@DeeperMind
Copy link

DeeperMind commented May 9, 2019

When trying to run the experiment with docker for TFLite example, I encountered the following error:

+ python /tmp/FAI-PEP/benchmarking/run_bench.py -b /tmp/FAI-PEP/specifications/models/tflite/mobilenet_v2/mobilenet_v2_0.35_96.json --config_dir /tmp/config
usage: run_bench.py [-h] [--app_id APP_ID] [-b BENCHMARK_FILE] [--lab]
                    [--logger_level {info,warning,error}] [--remote]
                    --root_model_dir ROOT_MODEL_DIR [--token TOKEN]
                    [-c CUSTOM_BINARY] [--pre_built_binary PRE_BUILT_BINARY]
                    [--user_string USER_STRING]
run_bench.py: error: argument --root_model_dir is require

Does anyone know how to resolve this? Did I missed anything before running the example?

@hl475
Copy link
Contributor

hl475 commented May 9, 2019

I checked docker_tflite.sh, and I think root_model_dir is missing here. A quick solution is to add root_model_dir like this one into docker_tflite.sh file. Can you try that out to see if that could solve the problem? If so, it will be much appreciated if you can create a PR to fix this as well as the one in docker_pytorch.sh.

@sf-wind
Copy link
Contributor

sf-wind commented May 9, 2019

Right, please file a PR :) Thanks.

@DeeperMind
Copy link
Author

PR created. Thank you for your help!

facebook-github-bot pushed a commit that referenced this issue May 10, 2019
Summary:
Resolve a missing parameter (`--root_model_dir`) in tflite docker example.

Solve #284
Pull Request resolved: #285

Differential Revision: D15288828

Pulled By: hl475

fbshipit-source-id: 756e68b46ed8756ed75091eba5d45e1fbea6a0e5
@hl475 hl475 closed this as completed May 10, 2019
@hl475
Copy link
Contributor

hl475 commented May 10, 2019

PR landed. Thanks again!

alexvk added a commit to alexvk/models that referenced this issue Aug 1, 2019
run_bench.py: error: the following arguments are required: --root_model_dir

See: facebook/FAI-PEP#284
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

No branches or pull requests

3 participants