Skip to content
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.

Sign up for GitHub

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

Jump to bottom

ML.Net and Tensorflow integration demo. #3

Open
wants to merge 19 commits into
base: master
Choose a base branch
from
Open

ML.Net and Tensorflow integration demo. #3

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
7d64c20
ML.Net and Tensorflow integration demo.
zeahmed Aug 10, 2018
1353772
Merge remote-tracking branch 'upstream/master' into tensorflow
zeahmed Aug 10, 2018
a6852ac
Got output in flatten array. Shapes and types are preloaded.
zeahmed Aug 11, 2018
d625f48
Added model serialization to disk using Tensorflow frozen model scheme.
zeahmed Aug 14, 2018
988c36e
Added support for different types.
zeahmed Aug 15, 2018
1d4c58f
Don't use TensorFlowSharp, instead carry our own subset of the TF# API
ericstj Aug 13, 2018
989e61b
Bring over copy of TF# source
ericstj Aug 13, 2018
6201aa8
Add license to TF# files and ifdef to make them build
ericstj Aug 13, 2018
b2a8016
Trim down TensorFlowSharp API
ericstj Aug 14, 2018
d616fe5
Enable StyleCop and fix issues
ericstj Aug 14, 2018
4714fe8
Enable ML Codeanalyzer
ericstj Aug 14, 2018
fcc7d75
Move TensorFlow support into Microsoft.ML.Transforms
ericstj Aug 14, 2018
78b8870
Put TensorFlow support under Microsoft.ML.Transforms.TensorFlow and m…
ericstj Aug 14, 2018
f632824
Create a generic method for creating TF tensors
ericstj Aug 15, 2018
07b586b
Merge pull request #4 from ericstj/wrapTF
zeahmed Aug 16, 2018
a1350d1
Addressed reviewers' comments.
zeahmed Aug 16, 2018
67d765c
Addressed reviewers' comments.
zeahmed Aug 16, 2018
25e477c
Removed batch size as configurable param and setting it to one for now.
zeahmed Aug 16, 2018
cff9552
Added TensorTypeFromType from TFSharp.
zeahmed Aug 16, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions src/Microsoft.ML.Transforms/Microsoft.ML.Transforms.csproj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,11 @@
<DefineConstants>CORECLR</DefineConstants>
</PropertyGroup>

<PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">
<NoWarn>CS8002;1591</NoWarn>
Copy link

@ericstj ericstj Aug 10, 2018 •
edited by zeahmed
Loading

Choose a reason for hiding this comment

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

Interesting that it isn't strong named. That will cause issues on desktop. We could ask for it to be strong named, or build our own copy.

Also this can be

<NoWarn>$(NoWarn);CS8002</NoWarn>

And placed in the main PropertyGroup @ line 7. #Resolved

Copy link

Choose a reason for hiding this comment

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

FYI this is fixed with my changes #4

In reply to: 209299533 [](ancestors = 209299533)

<AllowUnsafeBlocks>true</AllowUnsafeBlocks>
</PropertyGroup>

<ItemGroup>
<None Remove="Text\StopWords\Arabic.txt" />
<None Remove="Text\StopWords\Czech.txt" />
Expand Down Expand Up @@ -44,6 +49,10 @@
<EmbeddedResource Include="Text\StopWords\Swedish.txt" />
</ItemGroup>

<ItemGroup>
<PackageReference Include="TensorFlowSharp" Version="1.9.0" />
</ItemGroup>

<ItemGroup>
<ProjectReference Include="..\Microsoft.ML.Api\Microsoft.ML.Api.csproj" />
<ProjectReference Include="..\Microsoft.ML.Core\Microsoft.ML.Core.csproj" />
Expand Down
Loading