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.

Sign up for GitHub

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

Jump to bottom

feat: Vertex tools support nil params and multiple tools #1133

Open
wants to merge 20 commits into
base: main
Choose a base branch
from
Open

feat: Vertex tools support nil params and multiple tools #1133

wants to merge 20 commits into from

Conversation

ImPedro29
Copy link

Support multiple tols

I've changed the function convertTools;

  • It now handle nil params
  • Support enum
  • Better logs for array of functions in array of tools
  • Rewrite to future support for other tools than function

ImPedro29 and others added 19 commits June 9, 2024 17:08
@ImPedro29
add partial support for system prompt
7a61e4f
@ImPedro29
adding comments doc
78e536e
@ImPedro29
Merge branch 'tmc:main' into main
6f5cddb
@ImPedro29
adding comments doc
b57ff5a
@ImPedro29
add enum support for google
cc3ea41
@ImPedro29
fix new tool logic
c14bf1f
@ImPedro29
check if params is nil
7474ca6
@ImPedro29
Merge remote-tracking branch 'refs/remotes/origin/main' into main-ver…
646636c 
…beux

# Conflicts:
#	go.mod
#	llms/googleai/vertex/vertex.go
@ImPedro29
chore: set parts to max 1 and update vertexai
89e085f
@ImPedro29
fix: convert function error log
8b7d5b0
@ImPedro29
add fileContent
702fb32
@ImPedro29
Merge branch 'tmc:main' into main
95ed9d3
@ImPedro29
Merge branch 'tmc:main' into main
1ac5bbe
@ImPedro29
feat: add enum and treat if hasn't params for vertex
8a4720d
@ImPedro29
Merge tag 'v0.1.16' into main-verbeux
f8c9510 
# Conflicts:
#	examples/vertex-completion-example/go.mod
#	examples/vertex-completion-example/go.sum
#	go.sum
#	llms/googleai/vertex/vertex.go
@ImPedro29
chore: fix logs for vertex convert function
d42c256
@ImPedro29
remove comment, now it support multiple
4bd94fa
@ImPedro29
chore: remove FileContent accidentally sent
22e4460
@ImPedro29
chore: removing not used errors
0c2ab14
@tmc
Copy link
Owner

tmc commented Feb 13, 2025

@ImPedro29 Thanks for your contribution! can you rebase this to resolve conflicts?

@ImPedro29
Merge branch 'main' into main-verbeux
13c336e 
# Conflicts:
#	examples/mistral-embedding-example/go.sum
#	examples/vertex-completion-example/go.sum
@ImPedro29
Copy link
Author

ImPedro29 commented Feb 13, 2025 •
edited
Loading

@ImPedro29 Thanks for your contribution! can you rebase this to resolve conflicts?

I've merged, done!
I recommend squash merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ImPedro29 @tmc