Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

update readme #3827

Merged
merged 27 commits into from
Jun 15, 2021
Merged

update readme #3827

merged 27 commits into from
Jun 15, 2021

Conversation

QuanluZhang
Copy link
Contributor

  • update NAS in readme
  • remove ppo tuner for hpo related doc

@QuanluZhang QuanluZhang changed the base branch from master to v2.3 June 15, 2021 01:24
README.md Outdated
<li><a href="https://nni.readthedocs.io/en/stable/NAS/DARTS.html">DARTS</a></li>
<li><a href="https://nni.readthedocs.io/en/stable/NAS/SPOS.html">SPOS</a></li>
<li><a href="https://nni.readthedocs.io/en/stable/NAS/Proxylessnas.html">ProxylessNAS</a></li>
<li><a href="https://nni.readthedocs.io/en/stable/NAS/ExplorationStrategies.html">PPO</a></li>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's better to write RL instead of PPO for better coverage. I'm not sure everyone knows PPO, and I'm sure they've heard of RL somehow.

@kvartet
Copy link
Contributor

kvartet commented Jun 15, 2021

please update docs/en_US/_templates/index.html, which is the homepage of the readthedocs

@kvartet
Copy link
Contributor

kvartet commented Jun 15, 2021

please update docs/en_US/_templates/index.html, which is the homepage of the readthedocs

I update it according to the README and push to this pr directly~

@kvartet kvartet closed this Jun 15, 2021
@kvartet kvartet reopened this Jun 15, 2021
@ultmaster ultmaster merged commit 8450724 into microsoft:v2.3 Jun 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants