-
Notifications
You must be signed in to change notification settings - Fork 6.7k
Link fixes4 #16764
Link fixes4 #16764
Conversation
iblislin
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Julia part is fine for me. 👍
julia/docs/src/tutorial/char-lstm.md
Outdated
| We will be using fixed-length input sequence for training. The code is | ||
| adapted from the [char-rnn example for MXNet's Python | ||
| binding](https://github.com/dmlc/mxnet-notebooks/blob/master/python/tutorials/char_lstm.ipynb), | ||
| binding](/api/r/docs/tutorials/char_rnn_model), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah, is there a Python tutorial on the web site?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@iblis17 there is this one: https://mxnet.apache.org/api/r/docs/tutorials/char_rnn_model.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
well, I think it's R's, not Python.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah I overlooked that. What about this? https://github.com/apache/incubator-mxnet/blob/8004a027ad6a73f8f6eae102de8d249fbdfb9a2d/example/rnn/old/char-rnn.ipynb
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, nice, that's the correct one.
TEChopra1000
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed site-link for python char-rnn tutorial.
|
@aaronmarkham @TEChopra1000 Your error - MKLDNN lib not found - #16629 Steps to merge This updates your master Hope this helps. |
ee698e7 to
40099d7
Compare
|
@aaronmarkham would you be willing to merge these link fixes? |
ChaiBapchya
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's get this merged!
|
|
||
| Here we will briefly introduce how to run inference using Module API in Python. There is more detailed explanation available in the [Predict Image Tutorial](https://mxnet.apache.org/tutorials/python/predict_image.html). | ||
| In general, prediction consists of the following steps: | ||
| Here we will briefly introduce how to run inference using Module API in Python. In general, prediction consists of the following steps: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the broken link discussed in #16724
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you want to add it back? or the image-classification shouldn't exist anymore? What's the solution for that issue?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ChaiBapchya #16724 is still open, I wondered if something is pending. I personally don't know the expect solution, if everything is good we can close it.
But following link was mentioned in the issue, not sure if this need to be considered.
True link should be -- https://github.com/dmlc/mxnet-notebooks/blob/master/python/tutorials/predict_imagenet.ipynb
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see there was a link which previously existed. Now the link has been removed. Have a word with Aaron/Talia they have a better idea.
A new round of link fixes.