This repository was archived by the owner on Jun 29, 2019. It is now read-only.
Update MachineLearningSamples-SentimentAnalysis#7
Open
ZhouFang928 wants to merge 6 commits into
Open
Conversation
Add a guide on how to install CNTK before installing Keras in section 3 CNTK Backend
The original ModelingAndEvaluation.md is not actually linked to the file related to execute SentimentExtraction.py. The original ModelingAndEvaluationDocker is actually a guide on model details and modeling with CNTK as Backend. Be good to specify the difference between executing on local and docker compute context. Now, I modify the names of the markdown files and corresponding python files. See if this is better.
ZhouFang928
changed the title
Add a comment on how to upgrade pip before installing azure-cli and azure-cli-ml. Add sudo before the installation command.
add step-by-step instructions.
specify Linux log in status
hning86
approved these changes
Sep 27, 2017
Contributor
|
@ZhouFang928, please resolve conflicts. |
Author
|
Hi Hai,
It shows that I am not authorized
<https://help.github.com/articles/about-protected-branches/> to merge this
pull request and I have deleted the folked repository a few days ago. Can I
still resolve the conflicts and how? If there is no way to do that, I will
have to close the pull request.
Thanks a lot!
Fang
…On Wed, Sep 27, 2017 at 9:27 AM, Hai Ning ***@***.***> wrote:
@ZhouFang928 <https://github.com/zhoufang928>, please resolve conflicts.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#7 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AKEdw5aJ-z8GP5fTrkxjVB4sBdqRYQFBks5smaSOgaJpZM4PahLp>
.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add a guide on how to install CNTK before installing Keras in section 3 CNTK Backend.
Modify the names of the markdown files and corresponding python files in scenario-sentiment-analysis.md.
Add pip upgrade before installing az-cli and az-cli-ml; do a sudo install on Linux env.
Add step-by-step instructions on how to set up azure ml environment.