larq / compute-engine

Highly optimized inference engine for Binarized Neural Networks
https://docs.larq.dev/compute-engine
Apache License 2.0
242 stars 34 forks source link

Permission denied when benchmarking on Android using ADB #665

Closed rameshKrSah closed 3 years ago

rameshKrSah commented 3 years ago

Hi, I tried the benchmark tool for Android using ADB, and I couldn't run the tool because of the permission denied error.

image

The solution was to make the benchmark tool executable using the ADB shell like this:

image

After this, the tool runs fine and prints out the evaluation results.

image

I just want to put this out here to help others facing the same problem.

An update to the docs here (https://docs.larq.dev/compute-engine/benchmark/) will be the permanent fix.

lgeiger commented 3 years ago

Thanks for catching this. I wasn't aware that the permissions might get lost on some phones or adb servers when pushing the benchmarking binary.

@rameshKrSah Do you want to sent a PR to add this to the docs here?

lgeiger commented 3 years ago

I wasn't aware that the permissions might get lost on some phones or adb servers when pushing the benchmarking binary.

Actually this might be due to Windows. I think we only tested the docs on Linux and macOS. Either way we should update the docs to change the permissions on the phone instead 👍

rameshKrSah commented 3 years ago

Thanks for catching this. I wasn't aware that the permissions might get lost on some phones or adb servers when pushing the benchmarking binary.

@rameshKrSah Do you want to sent a PR to add this to the docs here?

Sure, I can do that. Will send the PR soon. :)

rameshKrSah commented 3 years ago

The pull request was merged