Skip to content
This repository has been archived by the owner on Oct 13, 2021. It is now read-only.

Update Readme for applications #109

Merged
merged 4 commits into from
Jun 7, 2019
Merged

Update Readme for applications #109

merged 4 commits into from
Jun 7, 2019

Conversation

jiafatom
Copy link
Collaborator

@jiafatom jiafatom commented Jun 7, 2019

No description provided.

@jiafatom jiafatom requested a review from vinitra-zz June 7, 2019 17:06
Copy link
Member

@vinitra-zz vinitra-zz left a comment

Choose a reason for hiding this comment

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

Made some wording suggestions! Other than that, LGTM.

applications/mrcnn_onnx/README.md Outdated Show resolved Hide resolved
pip install -e .
python MaskedRCNN.py
```
Currently we have some code that are not ready to check-in.
Copy link
Member

Choose a reason for hiding this comment

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

Suggestion: maybe phrase this as "conversion code for this model will be coming soon" ?

applications/yolov3/README.md Outdated Show resolved Hide resolved
@jiafatom jiafatom merged commit 3d4bb6c into onnx:master Jun 7, 2019
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.

3 participants