Skip to content

fixed invalid model get reponse #558

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 1 commit into from
Jan 12, 2021
Merged

fixed invalid model get reponse #558

merged 1 commit into from
Jan 12, 2021

Conversation

DvirDukhan
Copy link
Collaborator

closes #557

@codecov
Copy link

codecov bot commented Jan 12, 2021

Codecov Report

Merging #558 (699eb0c) into master (247c4e5) will increase coverage by 0.05%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #558      +/-   ##
==========================================
+ Coverage   74.52%   74.57%   +0.05%     
==========================================
  Files          35       35              
  Lines        5644     5648       +4     
==========================================
+ Hits         4206     4212       +6     
+ Misses       1438     1436       -2     
Impacted Files Coverage Δ
src/command_parser.c 91.83% <100.00%> (-0.05%) ⬇️
src/model.c 78.48% <100.00%> (ø)
src/redisai.c 83.15% <100.00%> (+0.12%) ⬆️
src/backends/tensorflow.c 67.77% <0.00%> (+0.60%) ⬆️

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 247c4e5...699eb0c. Read the comment docs.

Copy link
Contributor

@lantiga lantiga left a comment

Choose a reason for hiding this comment

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

LGTM

@lantiga lantiga merged commit 8243e8d into master Jan 12, 2021
@lantiga lantiga deleted the fix_error_replies branch January 12, 2021 23:30
# 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.

AI.DAGRUN / AI.DAGRUN_RO negative tests inconsistent state
2 participants