-
Notifications
You must be signed in to change notification settings - Fork 0
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
Our own Tools struct #5
Conversation
…th all Qwen2.5 models
|
||
let expectedTool1 = WeatherTool.getCurrentWeather(.init( | ||
location: "Paris, France", | ||
unit: .celsius |
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.
@myobie Interestingly, Qwen 2.5 7B usually seems to return Celsius for this question. Granted, it makes sense because France uses Celsius. The smaller version of this model usually returns Fahrenheit, though, maybe because it's not smart enough to think about what measurement people in the city being asked about uses. Of course, that's just a guess.
"role": "system", | ||
"content": "You are weather fetching assistant. Your only purpose is to fetch weather data.", | ||
], | ||
["role": "system", "content": "The user prefers F°."], |
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.
@myobie I swapped Fahrenheit for Celsius here just to see if 7B listens to us. It seems to most of the time.
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.
Nice.
This should make it easier to specify tools.