LoadState error about llamasharp

[martindevans]

This isn't part of llamasharp I'm very familiar with, but it does look like it's just meant to be valid JSON. Here's the relevant code:

InteractiveExecutorState state = (InteractiveExecutorState)GetStateData();
using(FileStream fs = new FileStream(filename, FileMode.OpenOrCreate, FileAccess.Write))
{
    JsonSerializer.Serialize(fs, state);
}

I think it's because it's using OpenOrCreate, so there's still junk in the file if the new content is smaller than the old content.

Try deleting the file before saving.

saddam213 beat me to it 😆

[mphacker]

Deleting the file before saving works well.

[martindevans]

Could you share a copy of the JSON file?

[mphacker]

Here is an image of it in VS code. Is the SaveState supposed to just write out well formed JSON? It looks like it is writing out a bunch of odd data after the JSON.

If you need the full file I can try to paste it here.

[saddam213]

We need to change FileMode.OpenOrCreate to FileMode.Create so it overwrites the entite file

[martindevans]

I just pushed up a PR that should fix this, could you confirm if using the master branch fixes it?