Skip to content

Conversation

@chayim
Copy link
Contributor

@chayim chayim commented Jun 15, 2021

Corrected how we build onnx. We can now build and release on x64, and x64 gpu. This will build on osx x64 as well, and version 1.8 of onnx now builds (using these changes) on jetson. This PR is only for 1.7.2. A new developer should be able to use the accompanying instructions, in order to work on the onnx build should they so desire.

This would be the template for building back-ends. ci was built in full and the remaining run is for changing circleci back to master's norm.

@chayim chayim requested a review from DvirDukhan June 15, 2021 14:10
@chayim chayim changed the title jetson 1.8 onnxruntime 1.8 build and documentation Jun 15, 2021
@codecov
Copy link

codecov bot commented Jun 15, 2021

Codecov Report

Merging #785 (2014340) into master (3cf1354) will decrease coverage by 0.03%.
The diff coverage is 90.90%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #785      +/-   ##
==========================================
- Coverage   79.99%   79.96%   -0.04%     
==========================================
  Files          53       53              
  Lines        8032     8019      -13     
==========================================
- Hits         6425     6412      -13     
  Misses       1607     1607              
Impacted Files Coverage Δ
src/redisai.c 86.99% <ø> (ø)
src/execution/DAG/dag.c 91.73% <88.88%> (+1.47%) ⬆️
src/execution/parsing/dag_parser.c 96.01% <100.00%> (+0.01%) ⬆️
src/execution/parsing/deprecated.c 79.70% <100.00%> (-1.69%) ⬇️

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 db9857c...2014340. Read the comment docs.

@chayim chayim changed the title onnxruntime 1.8 build and documentation onnxruntime 1.7.2 build and documentation Jul 26, 2021
@@ -1,134 +1,16 @@
REDIS_ONNX_VERSION?=rel-1.7.2_RCE

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is the branch name?

#----------------------------------------------------------------------------------------------
{% if REDIS_ARCH == 'x64' %}
{% if REDIS_GPU is defined %}
FROM nvidia/cuda:11.0-cudnn8-devel-ubuntu18.04

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe allow user to define their cuda version?
we use 11.0
might want to compile it on newer versions

@alonre24 alonre24 mentioned this pull request Oct 5, 2021
@alonre24
Copy link
Collaborator

alonre24 commented Oct 5, 2021

moved to #847

@alonre24 alonre24 closed this Oct 5, 2021
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.

4 participants