forked from microsoft/nni
-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
merge master #238
Merged
Merged
merge master #238
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…on. (#2223) * Fix broken english in Overview Fix a lot of akward or misleading phrasing. Fix a few spelling errors. Fixed past tence vs presen tense (can vs. could, supports vs supported) * Sentences shouldn't typically begin with "and", in installation.rst * Fixed a bit of bad grammar and awkward phrasing in Linux installation instructions. * Additional, single correction in linux instructions. * Fix a lot of bad grammar and phrasing in windows installation instructions * Fix a variety of grammar and spelling problems in docker instructions Lots of akward phrasing. Lots of tense issues (could vs can) Lots of spelling errors (especcially "offical") Lots of missing articles Docker is a proper noun and should be capitalized * Missing article in windows install instructions. * Change some "refer to this"s to "see here"s. * Fix a lot of bad grammar and confusing wording in Quick Start tab "something" should be the "something" tab. Tense issues (e.g., Modified vs. Modify). * Fix some akward phrasing in hyperparameter tuning directory. * Clean up grammar and phrasing in trial setup. * Fix broken english in tuner directory. * Correct a bunch of bad wording throughout the Hyperparameter Tuning overview Lots of missing articles. Swapped out "Example usage" for "Example config", because that's what it is. Usage isn't examplified at all. I have no idea what the note at the end of the TPE section is trying to say, so I left it untouched, but it should be changed to something that make sense. * Fixing, as best I canm weird wording in TPE, Random Search, Anneal Tuners Fixed many incomplete sentences and bad wording. The first sentence in the Parallel TPE optimization section doesn't make sense to me, but I left it in case it's supposed to be that way. That sentence was copied from the blog post. * Improve wording in naive evolution description. * Minor changes to SMAC page wording. * Improve some wording, but mostly formatting, on Metis Tuner page. * Minor grammatical fix in Matis page. * Minor edits to Batch tuner description. * Minor fixes to gridsearch description. * Better wording for GPTuner description. * Fix a lot of wording in the Network Morphism description. * Improve wording in hyperbanding description. * Fix a lot of confusing wording, spelling, and gramatical errors in BOHB description. * Fix a lot of confusing and some redundant wording in the PPOTunder description. * Improve wording in Builtin Assesors overview. * Fix some wording in Assessor overview. * Improved some wording in Median Assessor's description. * Improve wording and grammar on curve fitting assessor description. * Improved some grammar and wording the the WebUI tutorial page. * Improved wording and gammar in NAS overview. Also deletes one redundant copy of a note that was stated twice. * Improved grammar and wording in NAS quickstart. * Improve much of the wording and grammar in the NAS guide. * Replace "Requirement of classArg" with "classArgs requirements:" in two files tuner and builtin assessor.One instance in HyperoptTuner.md and BuiltinAssessor.md. Co-authored-by: AHartNtkn <AHartNtkn@users.noreply.github.com>
* add warning message when final data illegal * update * update * update * fix comments Co-authored-by: Lijiao <15910218274@163.com>
* support final result is NaN in webui * delete unuseful change * show NaN rather than NaN(latest) Co-authored-by: Lijiao <15910218274@163.com>
* check pylint for nni_cmd * fix id error * init * remove comments Co-authored-by: SparkSnail <Administrator@MININT-27KBA7M.fareast.corp.microsoft.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.