-
Notifications
You must be signed in to change notification settings - Fork 171
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
[filter] Support snpe filter in android #4487
Conversation
- Let `custom=Runtime:[CPU|GPU|DSP|NPU]` set each snpe runtime. Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>
- Support snpe (SDK v2) in Android. Some additional env variables are requires to be set. REF: https://docs.qualcomm.com/bundle/publicresource/topics/80-63442-2/dsp_runtime.html Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>
a65c156
to
250a833
Compare
- Determine proper source code out of `tensor_filter_snpe.cc` and `tensor_filter_snpe_v1.cc` Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>
ifeq ($(shell test -d ${SNPE_ROOT}/include/zdl; echo $$?),0) | ||
NNSTREAMER_FILTER_SNPE_SRCS := \ | ||
$(NNSTREAMER_EXT_HOME)/tensor_filter/tensor_filter_snpe_v1.cc | ||
endif |
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.
Isn't this too dangerous assumption?
I'd recommend:
- build snpe(v2) if v2 is available
- build snpe(v1) if v1 is available AND v2 is not available.
The current logic is
- build snpe(v2) if v2 is available AND v1 is NOT available
- build snpe(v1) if v1 is available AND.
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.
I assume $SNPE_ROOT is unique and a single SNPE SDK does not have both header directories for V1 and V2. I think there is no possibility to both v1 and v2 available.
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.
ok.. if !exist(zdl)
means !v2, let's proceed.
Some additional env variables are requires to be set.
REF: https://docs.qualcomm.com/bundle/publicresource/topics/80-63442-2/dsp_runtime.html
Self evaluation: