-
Notifications
You must be signed in to change notification settings - Fork 267
[Work In Progress] Java API #1185
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: develop
Are you sure you want to change the base?
Conversation
| */ | ||
| public fun <Input, Output> functionalStrategy( | ||
| public fun <Input, Output> graphStrategy( | ||
| name: String = "funStrategy", |
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.
Should the name be changed as well? Should it be unique, use UUID then?
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.
Is there a reason why it’s Java and not Kotlin?
| */ | ||
| public suspend fun execute( | ||
| prompt: Prompt, | ||
| model: LLModel, | ||
| tools: List<ToolDescriptor> = emptyList() | ||
| tools: List<ToolDescriptor> |
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.
is tools required now?
| version = run { | ||
| // our version follows the semver specification | ||
|
|
||
| val baseVersion = project.property("version") as String | ||
| val main = "0.6.0-java-1" |
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.
you have outdated file. Please, change this in gradle.properties, if required
kpavlov
left a comment
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.
How about we divide up the PR? The first thing that jumps out is adding @JvmOverloads.
Motivation and Context
Breaking Changes
Type of the changes
Checklist
developas the base branchAdditional steps for pull requests adding a new feature