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

Configurable nniManager log path and log level #644

Merged
merged 29 commits into from
Jan 24, 2019
Merged

Configurable nniManager log path and log level #644

merged 29 commits into from
Jan 24, 2019

Conversation

chicm-ms
Copy link
Contributor

@chicm-ms chicm-ms commented Jan 22, 2019

  1. Configure nniManager log path
  2. Configure nniManager log level
  3. Add --debug command line parameter to nnictl

chicm-ms and others added 26 commits November 13, 2018 14:53
Fix nniManager unit test (#515)
* Support distributed job for frameworkcontroller (#612)

support distributed job for frameworkcontroller

* Multiphase doc (#519)

* multiPhase doc

* updates

* updates

* Add time parser for 'nnictl update duration' (#632)

Current nnictl update duration only support seconds unit, add a parser for this command to support {s, m, h, d}

* fix experiment state bug (#629)

* update top README.md (#622)

* Update README.md

* update (#634)

* Integration tests refactoring (#625)

* Integration test refactoring (#21) (#616)

* Integration test refactoring (#21)

* Refactoring integration tests

* test metrics

* update azure pipeline

* updates

* updates

* updates

* updates

* updates

* updates

* updates

* updates

* updates

* updates

* updates

* updates

* updates

* updates

* updates

* updates

* updates

* updates

* updates

* updates

* update trigger

* Integration test refactoring (#618)

* updates

* updates

* update pipeline (#619)

* update pipeline

* updates

* updates

* updates

* updates

* updates

* test pipeline (#623)

* test pipeline

* updates

* updates

* updates

* Update integration test (#624)

* Update integration test

* updates

* updates

* updates

* updates

* updates

* updates
This reverts commit 62fc165.
@chicm-ms chicm-ms requested a review from xuehui1991 January 22, 2019 09:56
@chicm-ms chicm-ms changed the title Configurable nniManager log path Configurable nniManager log path and log level Jan 22, 2019
Copy link
Contributor

@yds05 yds05 left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@chicm-ms chicm-ms merged commit d9c83c0 into microsoft:master Jan 24, 2019
@chicm-ms chicm-ms deleted the config_log_path branch January 24, 2019 06:12
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