vbatts/llama.cpp
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

SimpleChat:MCUI:CornerCases:Skip new chat, show only if current

Browse source 
Skip NewChat if user cancels or if one waiting for response from
the ai model.

Dont show a chat with newly got ai model response, if current chat
session has changed, some how. Chat session shouldnt be allowed to
change, if there is a pending response, but still as a additional
sanity check.
This commit is contained in:
HanishKVC 2024-05-21 21:55:05 +05:30
parent d3405ea072
commit c2e8d62707
1 changed files with 13 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

16
examples/server/public_simplechat/simplechat.js
View file

@ -359,7 +359,11 @@ class MultiChatUI {
}
}
chat.add(Roles.Assistant, assistantMsg);
chat.show(this.elDivChat);
if (chatId == this.curChatId) {
chat.show(this.elDivChat);
} else {
console.debug(`DBUG:MCUI:HandleUserSubmit:ChatId has changed:[${chatId}] [${this.curChatId}]`);
}
// Purposefully clear at end rather than begin of this function
// so that one can switch from chat to completion mode and sequece
// in a completion mode with multiple user-assistant chat data
@ -382,11 +386,17 @@ class MultiChatUI {
}
elDiv.replaceChildren();
// Btn for creating new chat session
let btnNew = el_create_button("NeW cHaT", (ev)=> {
let btnNew = el_create_button("New CHAT", (ev)=> {
if (this.elInUser.disabled) {
console.error(`ERRR:MCUI:NewChatClick:Current session [${this.curChatId}] awaiting response, ignoring request...`);
alert("ERRR:MCUI\nWait for response to pending query, before starting new chat session");
return;
}
let chatId = `Chat${Object.keys(this.simpleChats).length}`;
let chatIdGot = prompt("SimpleChat:MCUI:\nEnter id for new chat session", chatId);
if (!chatIdGot) {
chatIdGot = chatId;
console.error("ERRR:MCUI:NewChat:Skipping based on user request...");
return;
}
this.new_chat_session(chatIdGot, true);
this.create_session_btn(elDiv, chatIdGot);