-
feat: correct the way to get used connection names for csharp (#1442)
发布于
2023-12-13 05:59:30 +00:00 Description
This pull request includes several changes to improve code readability,
maintainability, and functionality. The most important changes include
refactoring code in the_flow_operations.pyfile to improve
readability and maintainability, various changes in theutils.pyfile
to improve code clarity and future modifications, and the addition of a
new class method in the_base_executor_proxy.pyfile for generating
tool metadata.Code readability and maintainability improvements:
src/promptflow/promptflow/_sdk/operations/_flow_operations.py:
Refactored code to improve readability and maintainability by using the
load_flowfunction instead of directly instantiating classes. [1]
[2]src/promptflow/promptflow/_sdk/_submitter/utils.py:
Various changes to improve code clarity and future code modifications.
[1]
[2]
[3]
[4]
[5]src/promptflow/promptflow/batch/_base_executor_proxy.py:
Added a new class method_get_tool_metadatato generate the tool
metadata file for a flow.
Other important changes:
src/promptflow/promptflow/_sdk/_submitter/test_submitter.py:
Changes to improve code readability and maintainability. [1]
[2]
[3]
[4]
[5]
[6]src/promptflow/promptflow/_sdk/entities/_flow.py:
Added a new propertylanguageto theFlowclass.
All Promptflow Contribution checklist:
- The pull request does not introduce [breaking changes].
- CHANGELOG is updated for new features, bug fixes or other
significant changes. - I have read the contribution guidelines.
- Create an issue and link to the pull request to get dedicated
review from promptflow team. Learn more: suggested
workflow.
General Guidelines and Best Practices
- Title of the pull request is clear and informative.
- There are a small number of commits, each of which have an
informative message. This means that previously merged commits do not
appear in the history of the PR. For more information on cleaning up the
commits in your PR, see this
page.
Testing Guidelines
- Pull request includes test coverage for the included changes.
下载附件