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

Rename llm-tool-functions to llm-tool, fixes to elisp-to-tool #141

Merged
merged 38 commits into from
Jan 21, 2025
Merged

Rename llm-tool-functions to llm-tool, fixes to elisp-to-tool #141

merged 38 commits into from
Jan 21, 2025
+184 −174

Conversation

ahyatt
Copy link
Owner

@ahyatt ahyatt commented Jan 21, 2025 •
edited
Loading

Also, make llm-make-tool accept arbitrary extra arguments, so no errors are thrown when getting cross-compatibility with gptel and llm tools.

This also will put out release 0.22.0. We need a new major version since there's an incompatible change (the renaming of a function).

ahyatt added 30 commits December 28, 2024 13:20
@ahyatt
Rename function calling to tool use and change the interface
3434a59
@ahyatt
Fix issues in tool use in conversations, make clearer struct names
fe691ca
@ahyatt
Get Gemini working with tool use
80fb034
@ahyatt
Port ollama to tool use, simplify plz requests
b741a5d
@ahyatt
Stop double pushing and use plist-put
b4ccd86
@ahyatt
Use tool calling in Claude, also add image support
ad69d2c
@ahyatt
Add async and array items
2d3dbd3
@ahyatt
Fix lint issues, bump version to 0.21
8e9b026
@ahyatt
Test for array
4c38e66
@ahyatt
Fix all tests
42439d0
@ahyatt
Update README.org to talk about tool use
1f73946
@ahyatt
Update copyright notices
2946987
@ahyatt
Rename and update the elisp converstion tool
bda5cf5
@ahyatt
Fix byte compilation
a831f03
@ahyatt
Fix extra quote in tester
a10d053
@ahyatt
Add to NEWS.org
0f64b1a
@ahyatt
Switch to json-serialize, which changes json schema
5df681c
@ahyatt
Fix issues causing integration tests to fail
b6efdd1
@ahyatt
Use json-parse-string instead of json-read-from-string
d9665a5
@ahyatt
Merge branch 'main' into tool-use
81f8a8a
@ahyatt
Remove plistp, which is too new to use
ecb92e6
@ahyatt
Update elisp tool utility with latest changes
54ff404
@ahyatt
Make enums specifications in tools vectors
e048acc
@ahyatt
Use :optional instead of :required
11f7f61
@ahyatt
Fix use of required in integration test
f52f093
@ahyatt
Fix issue in Open AI tool use, and fix tester's use of required
692f683 
This should fix #139.
@ahyatt
Fix to previous fix of Open AI's streaming tool use
b27cda0 
This provides a better fix to #139.
@ahyatt
Fix request tests
3bcd2b7
@ahyatt
Switch back to types as symbols, support properties correctly
7b2d8b7
@ahyatt
Merge branch 'main' into tool-use
a08525d
@ahyatt ahyatt had a problem deploying to Continuous Integration January 21, 2025 05:20 — with GitHub Actions Failure
@ahyatt ahyatt temporarily deployed to Continuous Integration January 21, 2025 05:20 — with GitHub Actions Inactive
@ahyatt ahyatt had a problem deploying to Continuous Integration January 21, 2025 05:20 — with GitHub Actions Failure
@ahyatt ahyatt had a problem deploying to Continuous Integration January 21, 2025 05:20 — with GitHub Actions Failure
@ahyatt ahyatt had a problem deploying to Continuous Integration January 21, 2025 05:20 — with GitHub Actions Error
@ahyatt ahyatt had a problem deploying to Continuous Integration January 21, 2025 05:20 — with GitHub Actions Error
@ahyatt ahyatt had a problem deploying to Continuous Integration January 21, 2025 05:20 — with GitHub Actions Error
@ahyatt ahyatt temporarily deployed to Continuous Integration January 21, 2025 05:23 — with GitHub Actions Inactive
@ahyatt ahyatt temporarily deployed to Continuous Integration January 21, 2025 05:23 — with GitHub Actions Inactive
@ahyatt ahyatt temporarily deployed to Continuous Integration January 21, 2025 05:23 — with GitHub Actions Inactive
@ahyatt ahyatt temporarily deployed to Continuous Integration January 21, 2025 05:23 — with GitHub Actions Inactive
@ahyatt ahyatt temporarily deployed to Continuous Integration January 21, 2025 05:23 — with GitHub Actions Inactive
@ahyatt ahyatt temporarily deployed to Continuous Integration January 21, 2025 05:23 — with GitHub Actions Inactive
@ahyatt ahyatt temporarily deployed to Continuous Integration January 21, 2025 05:23 — with GitHub Actions Inactive
@ahyatt ahyatt merged commit 600ea5e into main Jan 21, 2025
7 checks passed
@ahyatt ahyatt deleted the tool-use branch January 21, 2025 05:25
danielkrizian added a commit to danielkrizian/llm that referenced this pull request Feb 9, 2025
@danielkrizian
Rename llm-make-tool-function to llm-make-tool
3d510e5 
extends ahyatt#141 to utilities/elisp-to-tool.el
This was referenced Feb 9, 2025
Merged
Closed
ahyatt pushed a commit that referenced this pull request Feb 11, 2025
@danielkrizian
Rename llm-make-tool-function to llm-make-tool (#153)
697a4f1 
extends #141 to utilities/elisp-to-tool.el

Without this fix, 

```elisp
(require 'utilities/elisp-to-tool)
```
throws `Debugger entered--Lisp error: (void-function
llm-make-tool-function)`
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.
1 participant
@ahyatt