Skip to content
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

fix: Fix setServiceName python method in OpenAI #1498

Merged
merged 1 commit into from May 4, 2022

Conversation

mhamilton723
Copy link
Collaborator

@mhamilton723 mhamilton723 commented May 4, 2022

Summary

Briefly describe the changes included in this Pull Request.

Tests

Describe what tests have you performed to validate your changes before submitting the Pull Request. Use 'N/A' if not applicable.

Dependency changes

If you needed to make any changes to dependencies of this project, please describe them here.

AB#1776083

@mhamilton723
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@codecov-commenter
Copy link

codecov-commenter commented May 4, 2022

Codecov Report

Merging #1498 (a9e4b99) into master (7bf6b53) will decrease coverage by 0.11%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1498      +/-   ##
==========================================
- Coverage   84.21%   84.10%   -0.12%     
==========================================
  Files         296      289       -7     
  Lines       14923    14788     -135     
  Branches      708      708              
==========================================
- Hits        12568    12438     -130     
+ Misses       2355     2350       -5     
Impacted Files Coverage Δ
.../microsoft/azure/synapse/ml/cognitive/OpenAI.scala 55.22% <ø> (ø)
...org/apache/spark/ml/param/JsonEncodableParam.scala 57.89% <0.00%> (-26.32%) ⬇️
...g/apache/spark/ml/param/PythonWrappableParam.scala 66.66% <0.00%> (-8.34%) ⬇️
...crosoft/azure/synapse/ml/io/http/HTTPClients.scala 67.64% <0.00%> (-7.36%) ⬇️
...re/src/main/python/synapse/ml/core/schema/Utils.py 67.10% <0.00%> (-5.27%) ⬇️
...soft/azure/synapse/ml/cognitive/TextToSpeech.scala 84.84% <0.00%> (-3.04%) ⬇️
...oft/azure/synapse/ml/cognitive/TextAnalytics.scala 86.59% <0.00%> (-2.69%) ⬇️
...crosoft/azure/synapse/ml/lightgbm/TrainUtils.scala 83.64% <0.00%> (-2.52%) ⬇️
.../azure/synapse/ml/cognitive/TextAnalyticsSDK.scala 86.01% <0.00%> (-1.40%) ⬇️
...ft/azure/synapse/ml/cognitive/FormRecognizer.scala 75.00% <0.00%> (-0.93%) ⬇️
... and 9 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7bf6b53...a9e4b99. Read the comment docs.

@mhamilton723 mhamilton723 merged commit 6b9dadb into master May 4, 2022
@mhamilton723 mhamilton723 deleted the mhamilton723-patch-1 branch May 4, 2022 01:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants