-
Notifications
You must be signed in to change notification settings - Fork 4
feat: Implement optimization code paths and functionality for initial release #140
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
base: main
Are you sure you want to change the base?
Changes from 35 commits
9859d08
1712e4f
ea596a7
2fd55e2
8481690
f8e5509
c032aaf
aee6aa7
59c7ac7
59f03f2
e2ff561
af2dd03
ea43575
2fecd54
d3e1f96
44c8c59
8f9f1e2
a17fd6e
67fdbf1
3042984
288336e
5d76276
4cb8859
ba369a2
149aa76
31c8385
55674ae
8f3468f
7074cfa
a386a27
8d1a868
937542a
8b3c69f
f7ce6d3
7468372
a387f83
312161f
542c135
3bce893
e8c6692
572a2aa
0ed243e
a8f14de
23baeb4
94de596
dc82818
92f51fa
365fa94
d267832
9bedf9e
53f455f
66bc1f0
f2f0894
9c1d8d7
4fc1ecf
32dc4d0
3b4baa3
d15c6bc
99dd7b8
849d41f
c51e235
f481ed3
27b248e
4545203
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,7 +1,7 @@ | ||
| [project] | ||
| name = "launchdarkly-server-sdk-ai-optimization" | ||
| name = "ldai_optimizer" | ||
| version = "0.1.0" # x-release-please-version | ||
| description = "LaunchDarkly AI SDK optimization helpers" | ||
| description = "LaunchDarkly AI tool — optimizer" | ||
| authors = [{name = "LaunchDarkly", email = "dev@launchdarkly.com"}] | ||
| license = {text = "Apache-2.0"} | ||
| readme = "README.md" | ||
|
|
@@ -21,6 +21,7 @@ classifiers = [ | |
| ] | ||
| dependencies = [ | ||
| "launchdarkly-server-sdk-ai>=0.16.0", | ||
| "coolname>=2.0.0", | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do we need this? Why such an old version? Is having cool names for variation keys worth the risk of this dependency? Would the 'cool' names even be useful/meaningful, vs a default date-based name?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Discussed in review meeting; will just hand roll this as its pretty simple to implement |
||
| ] | ||
|
|
||
| [project.urls] | ||
|
|
@@ -42,7 +43,7 @@ requires = ["hatchling"] | |
| build-backend = "hatchling.build" | ||
|
|
||
| [tool.hatch.build.targets.wheel] | ||
| packages = ["src/ldai_optimization"] | ||
| packages = ["src/ldai_optimizer"] | ||
|
|
||
| [tool.mypy] | ||
| python_version = "3.10" | ||
|
|
||
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,44 @@ | ||
| """LaunchDarkly AI SDK — optimization. | ||
|
|
||
| This package will provide helpers to run selected tools against the LaunchDarkly API from SDK-based workflows. | ||
| """ | ||
|
|
||
| from ldai.tracker import TokenUsage | ||
|
|
||
| from ldai_optimizer.client import OptimizationClient | ||
| from ldai_optimizer.dataclasses import ( | ||
| AIJudgeCallConfig, | ||
| GroundTruthOptimizationOptions, | ||
| GroundTruthSample, | ||
| LLMCallConfig, | ||
| LLMCallContext, | ||
| OptimizationContext, | ||
| OptimizationFromConfigOptions, | ||
| OptimizationJudge, | ||
| OptimizationJudgeContext, | ||
| OptimizationOptions, | ||
| OptimizationResponse, | ||
| ToolDefinition, | ||
| ) | ||
| from ldai_optimizer.ld_api_client import LDApiError | ||
|
|
||
| __version__ = "0.0.0" | ||
|
|
||
| __all__ = [ | ||
| '__version__', | ||
| 'AIJudgeCallConfig', | ||
| 'GroundTruthOptimizationOptions', | ||
| 'GroundTruthSample', | ||
| 'LDApiError', | ||
| 'LLMCallConfig', | ||
| 'LLMCallContext', | ||
| 'OptimizationClient', | ||
| 'OptimizationContext', | ||
| 'OptimizationFromConfigOptions', | ||
| 'OptimizationJudge', | ||
| 'OptimizationJudgeContext', | ||
| 'OptimizationOptions', | ||
| 'OptimizationResponse', | ||
| 'TokenUsage', | ||
| 'ToolDefinition', | ||
| ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I won't block on this but all other launchdarkly python packages have the launchdarkly-* name.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Renamed to
launchdarkly-ai-optimizerand claimed the package name. I'll clean up the older ones