Skip to content
This repository has been archived by the owner on Aug 10, 2019. It is now read-only.

Should it be a separate repo? #2

Open
dzhulgakov opened this issue Apr 2, 2018 · 1 comment
Open

Should it be a separate repo? #2

dzhulgakov opened this issue Apr 2, 2018 · 1 comment

Comments

@dzhulgakov
Copy link
Member

It's minor, just out of curiosity - why is it a separate repo? ONNX support itself seems to be native in https://github.com/Microsoft/CNTK , so it feels nicer to just point people to that instead of asking them to install another python package.

If it's only for testing purposes - can't we just fold it under CNTK repo as well? Or keep it separate somewhere so that people don't get confused on what they should use.

For reference, something like onnx-coreml or onnx-tensorflow exists as a separate repo but it includes both conversion and testing. For onnx-caffe2 it used to be similar, but as we merged the conversion into the main C2 repo we also moved tests in there.

@kit1980
Copy link

kit1980 commented Oct 2, 2018

This is now moved to the CNTK repo.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants