You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2021/04/13 22:44:57 UTC

[GitHub] [incubator-mxnet] deepakkumar1984 opened a new pull request #20162: Adding MxNet.Sharp package to the ecosystem page

deepakkumar1984 opened a new pull request #20162:
URL: https://github.com/apache/incubator-mxnet/pull/20162


   As discussed here: https://github.com/apache/incubator-mxnet/issues/16167#issuecomment-817845948
   
   I have added one more package to the list of the ecosystem. 
   
   Changes
   1) Add new image for MxNet.Sharp  to the asset/img folder
   2) Update the ecosystem.html to add project details


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #20162: Adding MxNet.Sharp package to the ecosystem page

Posted by GitBox <gi...@apache.org>.
mxnet-bot commented on pull request #20162:
URL: https://github.com/apache/incubator-mxnet/pull/20162#issuecomment-819099792


   Hey @deepakkumar1984 , Thanks for submitting the PR 
   All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following commands: 
   - To trigger all jobs: @mxnet-bot run ci [all] 
   - To trigger specific jobs: @mxnet-bot run ci [job1, job2] 
   *** 
   **CI supported jobs**: [clang, unix-cpu, windows-cpu, centos-cpu, miscellaneous, centos-gpu, sanity, edge, unix-gpu, windows-gpu, website]
   *** 
   _Note_: 
    Only following 3 categories can trigger CI :PR Author, MXNet Committer, Jenkins Admin. 
   All CI tests must pass before the PR can be merged. 
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-mxnet] deepakkumar1984 commented on pull request #20162: Adding MxNet.Sharp package to the ecosystem page

Posted by GitBox <gi...@apache.org>.
deepakkumar1984 commented on pull request #20162:
URL: https://github.com/apache/incubator-mxnet/pull/20162#issuecomment-821657131


   Thanks @szha for the detailed info, I have removed the csharp-package and only the ecosystem page changes is there in this PR. I can start another RFC thread for adding the c# package and continue with further discussions. Indeed its a big step and your guidance will be needed and hopefully will be able to merge it smootly.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-mxnet] szha merged pull request #20162: Adding MxNet.Sharp package to the ecosystem page

Posted by GitBox <gi...@apache.org>.
szha merged pull request #20162:
URL: https://github.com/apache/incubator-mxnet/pull/20162


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-mxnet] szha commented on pull request #20162: Adding MxNet.Sharp package to the ecosystem page

Posted by GitBox <gi...@apache.org>.
szha commented on pull request #20162:
URL: https://github.com/apache/incubator-mxnet/pull/20162#issuecomment-821251723


   @deepakkumar1984 thanks for the contribution. Looks like you included the c# package itself in this PR. Since that's a large change and has implication to the commitment of this community, it would be better to have a request-for-comment (RFC) for adding the c# support.
   
   With the RFC, we want to make sure the new language binding and examples will be well maintained in this code base. Information such as the current maintainers of the package, the necessary CI support, license information, intended release cycle and distribution channels, and expected user groups, are all relevant. Once these factors are clarified, the assimilation of the code base can be done as a one-time donation via software grant (see https://www.apache.org/foundation/how-it-works/legal.html#incoming-code), with which we could include you or intended maintainers as committers once the community agrees. In addition, note that mxnet is a trademark of the Apache Software Foundation which has specific naming requirements. The guideline is here: https://www.apache.org/foundation/marks/faq/#products.
   
   I know it's a lot to take in and adding a language binding is a big step. I will offer help wherever I can and feel free to let me know if you have questions. In this PR, shall we focus on adding the reference to the package in the ecosystem page first?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org