Skip to content

Backends build #851

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

Merged
merged 54 commits into from
Oct 14, 2021
Merged

Backends build #851

merged 54 commits into from
Oct 14, 2021

Conversation

alonre24
Copy link
Collaborator

This PR is holding the work that been done until 1.2 version GA release for building RedisAI backends for different platforms. In the current state, we only use our own build for Onnxruntime backend.

In the future, we may continue this work and have our own build for additional platforms and architectures.

@codecov
Copy link

codecov bot commented Oct 12, 2021

Codecov Report

Merging #851 (09d914c) into master (de0f302) will increase coverage by 1.02%.
The diff coverage is 91.54%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #851      +/-   ##
==========================================
+ Coverage   79.97%   80.99%   +1.02%     
==========================================
  Files          53       55       +2     
  Lines        8009     8151     +142     
==========================================
+ Hits         6405     6602     +197     
+ Misses       1604     1549      -55     
Impacted Files Coverage Δ
...backends/libtorch_c/torch_extensions/torch_redis.h 90.90% <ø> (ø)
src/serialization/AOF/rai_aof_rewrite.c 0.00% <0.00%> (ø)
.../serialization/RDB/decoder/previous/v2/decode_v2.c 46.56% <0.00%> (+9.81%) ⬆️
src/execution/parsing/deprecated.c 81.18% <66.66%> (+1.48%) ⬆️
...c/serialization/RDB/decoder/current/v4/decode_v4.c 71.42% <71.42%> (ø)
.../serialization/RDB/decoder/previous/v0/decode_v0.c 63.33% <72.72%> (+1.69%) ⬆️
tests/module/LLAPI.c 75.91% <79.06%> (+1.44%) ⬆️
src/serialization/RDB/encoder/v4/encode_v4.c 85.96% <81.81%> (ø)
src/backends/tensorflow.c 72.16% <91.00%> (+3.12%) ⬆️
src/backends/onnxruntime.c 73.50% <91.95%> (-3.88%) ⬇️
... and 24 more

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 b04ec83...09d914c. Read the comment docs.

@DvirDukhan DvirDukhan merged commit 09d914c into master Oct 14, 2021
@DvirDukhan DvirDukhan deleted the backends_build branch October 14, 2021 10:31
@alonre24 alonre24 restored the backends_build branch October 17, 2021 11:21
@DvirDukhan DvirDukhan deleted the backends_build branch October 17, 2021 11:31
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.

3 participants