Skip to content
This repository has been archived by the owner on Aug 25, 2024. It is now read-only.

WIP:Adding tensorflow regression model #226

Merged
merged 22 commits into from
Oct 25, 2019
Merged

Conversation

aghinsa
Copy link
Contributor

@aghinsa aghinsa commented Oct 18, 2019

resolves#75 and part of #29

@lgtm-com
Copy link

lgtm-com bot commented Oct 18, 2019

This pull request introduces 6 alerts when merging 633ee5e into 2a23f55 - view on LGTM.com

new alerts:

  • 6 for Unused import

Copy link

@johnandersen777 johnandersen777 left a comment

Choose a reason for hiding this comment

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

Very nice! You've make quick work of this. Just a few minor things.

model/tensorflow/dffml_model_tensorflow/dnnr.py Outdated Show resolved Hide resolved
model/tensorflow/dffml_model_tensorflow/dnnr.py Outdated Show resolved Hide resolved
model/tensorflow/dffml_model_tensorflow/dnnr.py Outdated Show resolved Hide resolved
model/tensorflow/dffml_model_tensorflow/dnnr.py Outdated Show resolved Hide resolved
model/tensorflow/tests/test_dnnr.py Outdated Show resolved Hide resolved
model/tensorflow/tests/test_dnnr.py Outdated Show resolved Hide resolved
@lgtm-com
Copy link

lgtm-com bot commented Oct 20, 2019

This pull request introduces 1 alert when merging 652bd22 into 8577ca5 - view on LGTM.com

new alerts:

  • 1 for Unused import

@codecov-io
Copy link

Codecov Report

Merging #226 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #226      +/-   ##
==========================================
- Coverage   91.76%   91.76%   -0.01%     
==========================================
  Files          57       57              
  Lines        3607     3606       -1     
  Branches      420      421       +1     
==========================================
- Hits         3310     3309       -1     
  Misses        240      240              
  Partials       57       57
Impacted Files Coverage Δ
dffml/source/source.py 86.66% <0%> (-0.22%) ⬇️

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 2a23f55...9624dbd. Read the comment docs.

John Andersen and others added 3 commits October 24, 2019 15:38
Signed-off-by: John Andersen <[email protected]>
Signed-off-by: John Andersen <[email protected]>
Signed-off-by: John Andersen <[email protected]>
@johnandersen777 johnandersen777 merged commit 3991622 into intel:master Oct 25, 2019
@aghinsa aghinsa deleted the tfdnnr branch October 25, 2019 09:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants