Skip to content
This repository has been archived by the owner on Jan 3, 2023. It is now read-only.

add readme #139

Merged
merged 4 commits into from
Feb 26, 2018
Merged

add readme #139

merged 4 commits into from
Feb 26, 2018

Conversation

mbrookhart
Copy link
Contributor

With description of currently failing tests.

@mbrookhart mbrookhart requested review from adstraw and ashokei February 8, 2018 18:51
Copy link
Contributor

@ashokei ashokei left a comment

Choose a reason for hiding this comment

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

can you add some markdown formatting, perhaps sections/titles/header.

Also, a simple quick-start or installation section like MKL_README for example:
https://github.com/apache/incubator-mxnet/blob/master/MKL_README.md

@mbrookhart
Copy link
Contributor Author

Good call, thanks Ashok

Copy link
Contributor

@ashokei ashokei left a comment

Choose a reason for hiding this comment

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

LGTM.

@mbrookhart mbrookhart merged commit 1f90ebc into ngraph-integration-dev Feb 26, 2018
@mbrookhart mbrookhart deleted the mbrookhart/ngraph_readme branch February 26, 2018 16:46
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.

2 participants