microsoft / promptflow

Build high-quality LLM apps - from prototyping, testing to production deployment and monitoring.
https://microsoft.github.io/promptflow/
MIT License
8.43k stars 725 forks source link

Add possibility to select logging level. #2786

Closed nick863 closed 3 weeks ago

nick863 commented 1 month ago

Description

This PR address the issue from the work item 3096911. Essentially, we want to control the logging verbosity. I have made the changes in the logging and added unit test. To better test changes I have also added mypy configuration to pyproject.toml and fixed all the encountered issues.

All Promptflow Contribution checklist:

General Guidelines and Best Practices

Testing Guidelines

github-actions[bot] commented 1 month ago

promptflow-core test result

0 tests   0 :white_check_mark:  0s :stopwatch: 0 suites  0 :zzz: 0 files    0 :x:

Results for commit 06f89263.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 1 month ago

promptflow SDK CLI Azure E2E Test Result nirovins/change_logging_mechanism

  4 files    4 suites   5m 1s :stopwatch: 231 tests 202 :white_check_mark:  29 :zzz: 0 :x: 924 runs  808 :white_check_mark: 116 :zzz: 0 :x:

Results for commit 06f89263.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 1 month ago

SDK CLI Global Config Test Result nirovins/change_logging_mechanism

4 tests   4 :white_check_mark:  1m 13s :stopwatch: 1 suites  0 :zzz: 1 files    0 :x:

Results for commit 06f89263.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 1 month ago

promptflow-evals test result

12 files  12 suites   2m 13s :stopwatch:  3 tests  2 :white_check_mark:  1 :zzz: 0 :x: 36 runs  24 :white_check_mark: 12 :zzz: 0 :x:

Results for commit 06f89263.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 1 month ago

Executor Unit Test Result nirovins/change_logging_mechanism

779 tests   779 :white_check_mark:  3m 42s :stopwatch:   1 suites    0 :zzz:   1 files      0 :x:

Results for commit 06f89263.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 1 month ago

Executor E2E Test Result nirovins/change_logging_mechanism

238 tests   232 :white_check_mark:  5m 9s :stopwatch:   1 suites    5 :zzz:   1 files      1 :x:

For more details on these failures, see this check.

Results for commit 06f89263.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 1 month ago

SDK CLI Test Result nirovins/change_logging_mechanism

    4 files      4 suites   1h 5m 22s :stopwatch:   661 tests   637 :white_check_mark: 23 :zzz: 1 :x: 2 644 runs  2 551 :white_check_mark: 92 :zzz: 1 :x:

For more details on these failures, see this check.

Results for commit 06f89263.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 1 month ago

Hi, thank you for your interest in helping to improve the prompt flow experience and for your contribution. We've noticed that there hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by pushing some changes or leaving a comment.

github-actions[bot] commented 3 weeks ago

Hi, thank you for your contribution. Since there has not been recent engagement, we are going to close this out. Feel free to reopen if you'd like to continue working on these changes. Please be sure to remove the no-recent-activity label; otherwise, this is likely to be closed again with the next cleanup pass.