aaronmarkham commented on a change in pull request #18783:
URL: https://github.com/apache/incubator-mxnet/pull/18783#discussion_r461234484
##########
File path: docs/static_site/src/pages/api/api.html
##########
@@ -123,26 +74,11 @@ <h4>Python-first API</h4>
{%- endfor -%}
<h2>Other Bindings</h2>
<div class="row">
-{%- for doc in page.docs -%}
- {%- if doc.tag != 'python' -%}
- <div class="col-4">
- <div class="docs-card">
- <div class="docs-logo-container">
- <img class="docs-logo-image" src="{{doc.icon | relative_url}}">
- </div>
- <div class="docs-action-btn">
- <a href="{{doc.guide_link | relative_url}}"> <img
src="{{'assets/img/compass.svg' | relative_url}}"
class="docs-logo-docs">{{doc.title}} Guide <span
class="span-accented">›</span></a>
- </div>
- <div class="docs-action-btn">
- <a href="{{doc.tutorial_link | relative_url}}"> <img
src="{{'assets/img/video-tutorial.svg' | relative_url}}"
class="docs-logo-docs">{{doc.title}} Tutorials <span
class="span-accented">›</span></a>
- </div>
- <div class="docs-action-btn">
- <a href="{{doc.api_link | relative_url}}"> <img
src="{{'assets/img/api.svg' | relative_url}}"
class="docs-logo-docs">{{doc.title}} API Reference <span
class="span-accented">›</span></a>
- </div>
- </div>
+ <div class="language-binding-banner">
+ <h4>Call for Contribution</h4>
+ The language bindings of MXNet 1.x were deleted in 2.x due to some C APIs
being deprecated and the bindings did rely on the deprecated APIs.
Review comment:
```suggestion
The Clojure, Java, Julia, R, and Scala language bindings of MXNet v1.x
were removed in v2.x due to some C APIs being deprecated and the bindings rely
on the deprecated APIs. You can still use these language bindings in v1.x.
```
##########
File path: docs/static_site/src/pages/api/api.html
##########
@@ -123,26 +74,11 @@ <h4>Python-first API</h4>
{%- endfor -%}
<h2>Other Bindings</h2>
<div class="row">
-{%- for doc in page.docs -%}
- {%- if doc.tag != 'python' -%}
- <div class="col-4">
- <div class="docs-card">
- <div class="docs-logo-container">
- <img class="docs-logo-image" src="{{doc.icon | relative_url}}">
- </div>
- <div class="docs-action-btn">
- <a href="{{doc.guide_link | relative_url}}"> <img
src="{{'assets/img/compass.svg' | relative_url}}"
class="docs-logo-docs">{{doc.title}} Guide <span
class="span-accented">›</span></a>
- </div>
- <div class="docs-action-btn">
- <a href="{{doc.tutorial_link | relative_url}}"> <img
src="{{'assets/img/video-tutorial.svg' | relative_url}}"
class="docs-logo-docs">{{doc.title}} Tutorials <span
class="span-accented">›</span></a>
- </div>
- <div class="docs-action-btn">
- <a href="{{doc.api_link | relative_url}}"> <img
src="{{'assets/img/api.svg' | relative_url}}"
class="docs-logo-docs">{{doc.title}} API Reference <span
class="span-accented">›</span></a>
- </div>
- </div>
+ <div class="language-binding-banner">
+ <h4>Call for Contribution</h4>
+ The language bindings of MXNet 1.x were deleted in 2.x due to some C APIs
being deprecated and the bindings did rely on the deprecated APIs.
+ Given that MXNet has C APIs and other language bindings can be added. Your
contribution is welcome.
Review comment:
```suggestion
MXNet's new C APIs in v2.x can be used to reestablish your preferred
language binding. Your contribution is welcome!
```
Can you double check that folks should be using the CPP API instead?
Previously this was always a debate since the C API was supposed to be internal
only.
##########
File path: docs/static_site/src/pages/api/api.html
##########
@@ -123,26 +74,11 @@ <h4>Python-first API</h4>
{%- endfor -%}
<h2>Other Bindings</h2>
<div class="row">
-{%- for doc in page.docs -%}
- {%- if doc.tag != 'python' -%}
- <div class="col-4">
- <div class="docs-card">
- <div class="docs-logo-container">
- <img class="docs-logo-image" src="{{doc.icon | relative_url}}">
- </div>
- <div class="docs-action-btn">
- <a href="{{doc.guide_link | relative_url}}"> <img
src="{{'assets/img/compass.svg' | relative_url}}"
class="docs-logo-docs">{{doc.title}} Guide <span
class="span-accented">›</span></a>
- </div>
- <div class="docs-action-btn">
- <a href="{{doc.tutorial_link | relative_url}}"> <img
src="{{'assets/img/video-tutorial.svg' | relative_url}}"
class="docs-logo-docs">{{doc.title}} Tutorials <span
class="span-accented">›</span></a>
- </div>
- <div class="docs-action-btn">
- <a href="{{doc.api_link | relative_url}}"> <img
src="{{'assets/img/api.svg' | relative_url}}"
class="docs-logo-docs">{{doc.title}} API Reference <span
class="span-accented">›</span></a>
- </div>
- </div>
+ <div class="language-binding-banner">
+ <h4>Call for Contribution</h4>
Review comment:
```suggestion
<h4>Language Bindings Call for Contributions</h4>
```
----------------------------------------------------------------
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:
us...@infra.apache.org
