On Mon, Oct 7, 2019 at 6:32 PM Chaitanya Bapat <chai.ba...@gmail.com> wrote:
> Thanks a lot everyone for chiming in. > > Update: The issue in my PR was a bug (unrelated to ONNX) I introduced while > switching from GPU to non-GPU 2D transpose support. Thanks to Xingjian Shi > <https://github.com/sxjscience> for pointing that out. Apologies from my > side. > > As it stands now, the PR passes. I added a unit test to track that bug. So > we don't need any disabling. > > Since this wasn't related to ONNX per se, I guess the premise behind this > discussion is now null and void. > Also as it turns out, there are customers who use MXNet with ONNX so we > might have to continue testing and supporting (even if in maintenance mode) > > Note that 'we' there is a bit vague. Apache does not have customers, so there's no "we have to continue". If the community no longer has belief/trust/energy/interest in the code, then one approach is to remove it from the primary build and put it into a separate directory or repository. You can add a readme on how to re-integrate it and a note that if someone is interested in contributing to it then the community will do their best to respond. Or you can delete it. Amazon can always fork the code in a separate repo for their customers. Hen
