Alex Yang

Results 510 comments of Alex Yang

It has been fixed in the latest version

![image](https://github.com/run-llama/LlamaIndexTS/assets/14026360/b5dc23bb-ed4d-4e83-a87d-af2b34686681)

> I know the project now includes `Settings` etc but that would not work at all for our use case, hence the above. Please don't deprecate `serviceContextFromDefaults` it is very...

I see, I think we didn't pass some context properly, I will check that later

if so, would u please add a test case for this? I want to make repo stable

This is a bug that Ollama does not always return a valid JSON string

There is one solution for this case: 1. retry many times to LLM asking for a "fixed" result 2. crash to user 3. try to parse a invalid json

Sorry for the late response, I was on vacation. This is LGTM, let me do some small change

I remove `node-canvas` deps, seems like we don't need this for non-image