Skip to content

Conversation

@BillWagner
Copy link
Member

Use the new Column attribute for fields in the input data.

Change the argument names on the TextLoader constructor.

Update data path names to match instructions. (.NET Core applications run in ./bin/Debug/netcoreapp2.0 (or substitute current workload)

/cc @mairaw @aditidugar

Use the new `Column` attribute for fields in the input data.

Change the argument names on the `TextLoader` constructor.

Update data path names to match instructions. (.NET Core applications run in **./bin/Debug/netcoreapp2.0** (or substitute current workload)
@BillWagner BillWagner requested a review from mairaw May 7, 2018 21:06
The data attributes changed in the code, so update the text as well.
Copy link
Contributor

@rpetrusha rpetrusha left a comment

Choose a reason for hiding this comment

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

LGTM. This is ready to merge when the build finishes, @BillWagner.

@BillWagner
Copy link
Member Author

merging. The portal shows that the build completed, but status is not being reported.

@BillWagner BillWagner merged commit 523c3d3 into dotnet:master May 7, 2018
@BillWagner BillWagner deleted the update-sentiment-analysis-tutorial-with-released-bits branch May 8, 2018 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants