Skip to content

Refactored docker builder #602

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 4 commits into from
Feb 8, 2021
Merged

Refactored docker builder #602

merged 4 commits into from
Feb 8, 2021

Conversation

rafie
Copy link
Contributor

@rafie rafie commented Feb 7, 2021

No description provided.

@codecov
Copy link

codecov bot commented Feb 8, 2021

Codecov Report

Merging #602 (e45571c) into master (8c258e7) will increase coverage by 1.58%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #602      +/-   ##
==========================================
+ Coverage   74.34%   75.93%   +1.58%     
==========================================
  Files          39       39              
  Lines        6062     6062              
==========================================
+ Hits         4507     4603      +96     
+ Misses       1555     1459      -96     
Impacted Files Coverage Δ
src/DAG/dag.c 88.05% <0.00%> (+2.07%) ⬆️
src/backends/onnxruntime.c 67.74% <0.00%> (+6.72%) ⬆️
src/tensor.c 84.64% <0.00%> (+8.05%) ⬆️
src/backends/torch.c 80.25% <0.00%> (+8.58%) ⬆️

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 8c258e7...e45571c. Read the comment docs.

@rafie rafie requested a review from DvirDukhan February 8, 2021 06:44
@rafie rafie merged commit adcd882 into master Feb 8, 2021
@rafie rafie deleted the rafi-dockerhub2 branch February 8, 2021 10:55
DvirDukhan pushed a commit that referenced this pull request Feb 14, 2021
@chayim chayim mentioned this pull request Jul 22, 2021
@chayim chayim mentioned this pull request Nov 7, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants