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

tflite build changes #688

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

tflite build changes #688

wants to merge 22 commits into from

Conversation

chayim
Copy link
Contributor

@chayim chayim commented Apr 18, 2021

Updating RedisAI to the latest version of tflite. tflite is built on the Jetson devices, as well as x86, as part of this series of changes. For now, as agreed we've removed OSX support - it will be added back in another card.

@chayim chayim changed the title added new build script tflite 2.4.0 build scripts Apr 26, 2021
@codecov
Copy link

codecov bot commented Apr 26, 2021

Codecov Report

Merging #688 (6678b33) into master (a874faf) will decrease coverage by 0.04%.
The diff coverage is 82.84%.

❗ Current head 6678b33 differs from pull request most recent head 9b6cc1b. Consider uploading reports for the commit 9b6cc1b to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master     #688      +/-   ##
==========================================
- Coverage   78.90%   78.85%   -0.05%     
==========================================
  Files          43       43              
  Lines        6850     6863      +13     
==========================================
+ Hits         5405     5412       +7     
- Misses       1445     1451       +6     
Impacted Files Coverage Δ
src/backends/libtflite_c/tflite_c.cpp 59.70% <ø> (-0.39%) ⬇️
src/execution/utils.c 68.96% <16.66%> (-2.47%) ⬇️
tests/module/LLAPI.c 74.46% <79.50%> (-0.26%) ⬇️
tests/module/DAG_utils.c 88.23% <91.58%> (-0.74%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 920dbcb...9b6cc1b. Read the comment docs.

@chayim chayim changed the title tflite 2.4.0 build scripts tflite build changes Apr 26, 2021
@chayim chayim requested review from rafie and DvirDukhan April 26, 2021 13:12
@chayim chayim marked this pull request as ready for review April 27, 2021 15:12
@chayim chayim added the ci-test label Apr 29, 2021
@chayim chayim added ci-test and removed ci-test labels May 3, 2021
@DvirDukhan
Copy link
Collaborator

@chayim
which cuda version is used to build the TFLITE
I'm running my env with cuda 11.0, and I can't load the delegate

@chayim
Copy link
Contributor Author

chayim commented May 31, 2021

@chayim
which cuda version is used to build the TFLITE
I'm running my env with cuda 11.0, and I can't load the delegate

On Jetson it was the local 10.2. On x86-64 I think 10.2 as well.

@CLAassistant
Copy link

CLAassistant commented Sep 22, 2022

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ DvirDukhan
❌ chayim
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add to docs - build backends from source on different platforms update tensorflow lite
4 participants