shaun smith

Results 69 comments of shaun smith

I did think on this one quite hard, but I think mitigating are: - Relatively low take-up of Prompts/Sampling Features reduce the risk. Those using the features will likely be...

Well, it's put here as a draft to provoke the conversation - and get input from the Maintainers. I'm happy to put the work in to a solution of any...

On this one, I am planning on writing a discussion thread showing examples of this, and potential workarounds with sample code.

Yes, I understand it is a breaking change, and was proposed as such. Given the changes for StructuredOutput for the next protocol revision I'm not sure that this is worse...

> This is no longer the case. OpenAI and Claude both allow arbitrary ordering, and I think Gemini does too. Came here to say same (I explicitly test fast-agent handling...

#1577 partially solves this, new SEP to be considered for PromptMessage types.

OK, happy to submit a PR for this. Changes are to CopyToClipBoardBtn.svelte to set icon height consistently, and use prop passing for the style classes. **BEFORE FIX**: ![image](https://github.com/user-attachments/assets/19709f8c-857b-4ef3-adbb-ad589af80c45) ![image](https://github.com/user-attachments/assets/5ac47d6d-55a3-4dca-8476-5bd81fe30542) **AFTER...

This has been troubling me too. My quick investigation is leading me here: https://github.com/huggingface/chat-ui/blob/d7b02af46d2c3e6f5b954d97ebbefdfbd0b3b612/src/lib/utils/messageUpdates.ts#L227-L248 Looks like the buffer on the browser side fills up, and the delay calculations don't work...

This commit added a PUBLIC_SMOOTH_UPDATES flag: https://github.com/huggingface/chat-ui/commit/a59bc5e1436b80d6867ff1b7c76103a2cf78517b Set it to true to enable the existing behaviour.