Skip to content

fix: toolbox_agent bugs of date format #794

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

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

fix: toolbox_agent bugs of date format #794

wants to merge 8 commits into from

Conversation

qidu
Copy link
Contributor

@qidu qidu commented May 20, 2025

fix: Modify SQL in samples/toolbox_agent/tools.yaml to process date format right.

Date format

  • When we search hotels with some LLM, the date format shows as bellow:
**Check-in Date:** April 23, 2024

And the users maybe also update checkin or checkout date following this format.

  • Modifying SQL in tools.yaml to be compatible with sqlite3:
update checkin date to 'April 24,2024' and checkout date to 'April 26,2024' of Best Western Bern

Or the checkin or checkout date my be updated to 2024 or 0

When search hotels with some LLM, the date format shows as bellow
```
**Check-in Date:** April 23, 2024
```
So i update the SQL in tools.yaml to be compatible with following instruction: 
```sql
update checkin date to 'April 24,2024' and checkout date to 'April 26,2024' of Best Western Bern
```
Copy link

google-cla bot commented May 20, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@seanzhou1023 seanzhou1023 self-assigned this May 21, 2025
@seanzhou1023
Copy link
Collaborator

thank you for the fix. but you need to go through CLA as a contributor.

@seanzhou1023 seanzhou1023 added the ready to pull for importing back to Google label May 21, 2025
@qidu
Copy link
Contributor Author

qidu commented May 21, 2025

thank you for the fix. but you need to go through CLA as a contributor.

done

Agreement Name Date Signed Manage
Google Individual CLA qidu   Edit Contact Information

@qidu qidu changed the title Update tools.yaml Update SQL in samples/toolbox_agent/tools.yaml for date format processing May 21, 2025
@qidu qidu changed the title Update SQL in samples/toolbox_agent/tools.yaml for date format processing FIX toolbox_agent bugs of date format and tools_dict crash May 21, 2025
@qidu qidu requested a review from seanzhou1023 May 21, 2025 12:13
@qidu qidu changed the title FIX toolbox_agent bugs of date format and tools_dict crash fix: toolbox_agent bugs of date format and tools_dict crash May 21, 2025
@seanzhou1023 seanzhou1023 added ready to pull for importing back to Google and removed ready to pull for importing back to Google labels May 21, 2025
@seanzhou1023 seanzhou1023 self-requested a review May 21, 2025 20:07
@qidu qidu changed the title fix: toolbox_agent bugs of date format and tools_dict crash fix: toolbox_agent bugs of date format May 22, 2025
@seanzhou1023 seanzhou1023 added tools Issues related to tools ready to pull for importing back to Google and removed ready to pull for importing back to Google tools Issues related to tools labels May 23, 2025
copybara-service bot pushed a commit that referenced this pull request May 23, 2025
--
004eb36 by sudu <teric@outlook.com>:

Update tools.yaml

When search hotels with some LLM, the date format shows as bellow
```
**Check-in Date:** April 23, 2024
```
So i update the SQL in tools.yaml to be compatible with following instruction:
```sql
update checkin date to 'April 24,2024' and checkout date to 'April 26,2024' of Best Western Bern
```
--
1300688 by qidu <qidu@outlook.com>:

Fix the tools of `ToolboxToolset` initialization bugs for
`samples/toolbox_agent`

COPYBARA_INTEGRATE_REVIEW=#794 from qidu:main 5b3199d
PiperOrigin-RevId: 762259287
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to pull for importing back to Google
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy