fix warning for homepage build #680
Conversation
QuanluZhang
requested review from
squirrelsc,
scarlett2018 and
yds05
and removed request for
squirrelsc and
scarlett2018
| nni.report_final_result(metrics) | ||
| ``` | ||
| `metrics` also could be any python object. If users use NNI built-in tuner/assessor, `metrics` follows the same format rule as that in `report_intermediate_result`, the number indicates the model's performance, for example, the model's accuracy, loss etc. This `metrics` is reported to [tuner](Builtin-Tuner.md). | ||
| `metrics` also could be any python object. If users use NNI built-in tuner/assessor, `metrics` follows the same format rule as that in `report_intermediate_result`, the number indicates the model's performance, for example, the model's accuracy, loss etc. This `metrics` is reported to [tuner](tuners.md). |
There was a problem hiding this comment.
Another issue found in https://nni.readthedocs.io/en/latest/FAQ.html
refer is not found: https://github.com/Microsoft/nni/blob/master/docs/ExperimentConfig
There was a problem hiding this comment.
https://nni.readthedocs.io/en/latest/KubeflowMode.html
Let's remove the email in the last para
Any problems when using NNI in kubeflow mode, plesae create issues on NNI github repo, or send mail to nni@microsoft.com
There was a problem hiding this comment.
https://nni.readthedocs.io/en/latest/KubeflowMode.html
The "here" in below para reached nowhere.
We already build a docker image msranni/nni on Docker Hub. It contains NNI python packages, Node modules and javascript artifact files required to start experiment, and all of NNI dependencies. The docker file used to build this image can be found at here. You can either use this image directly in your config file, or build your own image based on it.
https://nni.readthedocs.io/en/deployment/Dockerfile.build.base
Got this:
SORRY /
\ /
\ This page does /
] not exist yet. [ ,'|
] [ / |
]___ [ ,' |
] ]\ /[ [ |: |
] ] \ / [ [ |: |
] ] ] [ [ [ |: |
] ] ] __[ [ [ |: |
] ] ] ]\ _ /[ [ [ [ |: |
] ] ] ] (#) [ [ [ [ :===='
] ] ]].nHn.[_[ [ [
] ] ] HHHHH. [ [ [
] ] / `HH("N \ [ [
]]/ HHH " [[
] NNN [
] N/" [
] N H [
/ N
/ q,
/ \
There was a problem hiding this comment.
https://nni.readthedocs.io/en/latest/AdvancedNAS.html#weight-sharing-among-trials
"simple weight sharing example" goes no where.
For details, please refer to this simple weight sharing example. We also provided a practice example for reading comprehension, based on previous ga_squad example.
There was a problem hiding this comment.
@QuanluZhang actually it's tuners.rst, but recommonmark only recognize .md as hyperlink, and will convert it to tuners.html anyway.
There was a problem hiding this comment.
@scarlett2018 for FAQ & email issue, please refer to e439d42
and 1f5f04e
Other deadlinks are already fixed in previous commits.
All those changes can take effect only after this PR is merged.
also
hide
sdk_reference.rstfix source code deadlink w/ https global link
url_resolvernot working for github code links readthedocs/recommonmark#137 fixed, will be able to reference relative path again.fix deadlink for
README.mdAlso suggest that using links to homepage instead in
README.md