Skip to content

✨ Set temperature_unit to fahrenheit for weather example#69

Merged
pdettori merged 3 commits intokagenti:mainfrom
evaline-ju:weather-update
Oct 3, 2025
Merged

✨ Set temperature_unit to fahrenheit for weather example#69
pdettori merged 3 commits intokagenti:mainfrom
evaline-ju:weather-update

Conversation

@evaline-ju
Copy link
Copy Markdown
Contributor

@evaline-ju evaline-ju commented Oct 3, 2025

Summary

Removed unused imports and updated temperature unit for the weather tool

For the default model example (llama3.2:3b-instruct-fp16), there tended to be 'hallucinations' e.g. Screenshot 2025-10-02 at 2 03 02 PM showing that the weather appeared to be often "freezing". On further inspection, the number returned tended to be corrected (in Celsius) but misinterpreted by the agent. This sets the tool to explicitly use Fahrenheit so at least by default when the weather is presented in Fahrenheit, it is more correct. Another strategy is to change the agent itself to default to Celsius but would require different prompting.

With the update:
Screenshot 2025-10-02 at 7 31 48 PM

Screenshot 2025-10-02 at 7 32 00 PM Screenshot 2025-10-02 at 7 32 38 PM Screenshot 2025-10-02 at 7 32 33 PM

Signed-off-by: Evaline Ju <69598118+evaline-ju@users.noreply.github.com>
Signed-off-by: Evaline Ju <69598118+evaline-ju@users.noreply.github.com>
Signed-off-by: Evaline Ju <69598118+evaline-ju@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@pdettori pdettori left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@pdettori pdettori merged commit 21bc50d into kagenti:main Oct 3, 2025
2 checks passed
@evaline-ju evaline-ju deleted the weather-update branch October 3, 2025 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants