Skip to content
This repository was 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

@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.

pip install -e .
python MaskedRCNN.py
```
Currently we have some code that are not ready to check-in.

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" ?

@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