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

feat: polymorphic gpt3 prompts #1541

Merged
merged 10 commits into from Jun 24, 2022

Conversation

niehaus59
Copy link
Contributor

@niehaus59 niehaus59 commented Jun 22, 2022

Summary

Add batch, index and batch-index prompt styles to the OpenAI gpt3 interface

Tests

Added tests to OpenAISuite.scala for each new prompt type

Dependency changes

No dependency changes.

AB#1840472

@niehaus59
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Pull request contains merge conflicts.

@niehaus59
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@codecov-commenter
Copy link

codecov-commenter commented Jun 22, 2022

Codecov Report

Merging #1541 (bb55aff) into master (04c22c4) will increase coverage by 1.98%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master    #1541      +/-   ##
==========================================
+ Coverage   81.57%   83.55%   +1.98%     
==========================================
  Files         301      308       +7     
  Lines       15337    15463     +126     
  Branches      707      713       +6     
==========================================
+ Hits        12511    12920     +409     
+ Misses       2826     2543     -283     
Impacted Files Coverage Δ
...oft/azure/synapse/ml/param/UntypedArrayParam.scala 62.50% <25.00%> (-5.36%) ⬇️
.../microsoft/azure/synapse/ml/cognitive/OpenAI.scala 57.50% <70.00%> (+1.52%) ⬆️
...ft/azure/synapse/ml/param/JsonEncodableParam.scala 64.28% <100.00%> (+1.05%) ⬆️
...crosoft/azure/synapse/ml/io/http/HTTPClients.scala 66.17% <0.00%> (-1.48%) ⬇️
...e/synapse/ml/vw/VowpalWabbitContextualBandit.scala 88.19% <0.00%> (-1.25%) ⬇️
...se/ml/cognitive/MultivariateAnomalyDetection.scala 87.03% <0.00%> (-0.75%) ⬇️
...thon/synapse/ml/vw/VowpalWabbitContextualBandit.py 45.45% <0.00%> (ø)
...ain/python/synapse/ml/vw/VowpalWabbitClassifier.py 83.33% <0.00%> (ø)
...ython/synapse/ml/vw/VowpalWabbitRegressionModel.py 62.50% <0.00%> (ø)
...n/synapse/ml/vw/VowpalWabbitClassificationModel.py 62.50% <0.00%> (ø)
... and 19 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 04c22c4...bb55aff. Read the comment docs.

@niehaus59
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@niehaus59
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@niehaus59
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@niehaus59
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mhamilton723 mhamilton723 merged commit e7cf85c into microsoft:master Jun 24, 2022
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

3 participants