[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
How Has This Been Tested?
I tested with a LlamaIndex + ChainLit + GPT 3.5 application running locally. As shown below, the LLM can now call the function without complaining ValueError: fn_schema is None.
before
after
[ ] Added new unit/integration tests
[ ] Added new notebook (that tests end-to-end)
[x] I stared at the code and made sure it makes sense
Suggested Checklist:
[ ] I have added a library.json file if a new loader/tool was added
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] I ran make format; make lint to appease the lint gods
Description
All 3 functions were missing schema. This PR adds them.
Fixes https://github.com/run-llama/llama_index/issues/9732
Type of Change
Please delete options that are not relevant.
How Has This Been Tested?
I tested with a LlamaIndex + ChainLit + GPT 3.5 application running locally. As shown below, the LLM can now call the function without complaining
ValueError: fn_schema is None
.Suggested Checklist:
make format; make lint
to appease the lint gods