Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

the interactions from chatbot with Sensor case are not saved for the analytics #15

Open
anneliret opened this issue Mar 4, 2024 · 10 comments
Labels
bug Something isn't working

Comments

@anneliret
Copy link

I've just tried again today (4th March) with my account (annel@trial.com) on the Sensor anomaly detection case (Bosch). I went thourgh the entire interaction flow with Auditor persona. At the end, I get a message like "chatbot disconnected please try restart". So I thought my session was recorded in iSee.

but when I go to the analytics, there is no additional record on the 4th March for my session.
I've really worried that all the trial test of people with the sensor case were lost because of some issue at the end of the chatbot.

@anneliret anneliret added the bug Something isn't working label Mar 4, 2024
@bfleisch
Copy link

bfleisch commented Mar 5, 2024

Logs from the API container:

2024-03-05T10:34:48.908561104Z PayloadTooLargeError: request entity too large
2024-03-05T10:34:48.908598302Z at readStream (/app/node_modules/raw-body/index.js:156:17)
2024-03-05T10:34:48.908605230Z at getRawBody (/app/node_modules/raw-body/index.js:109:12)
2024-03-05T10:34:48.908610611Z at read (/app/node_modules/body-parser/lib/read.js:79:3)
2024-03-05T10:34:48.908615820Z at jsonParser (/app/node_modules/body-parser/lib/types/json.js:135:5)
2024-03-05T10:34:48.908620800Z at Layer.handle [as handle_request] (/app/node_modules/express/lib/router/layer.js:95:5)
2024-03-05T10:34:48.908625953Z at trim_prefix (/app/node_modules/express/lib/router/index.js:328:13)
2024-03-05T10:34:48.908631911Z at /app/node_modules/express/lib/router/index.js:286:9
2024-03-05T10:34:48.908637009Z at Function.process_params (/app/node_modules/express/lib/router/index.js:346:12)
2024-03-05T10:34:48.908642278Z at next (/app/node_modules/express/lib/router/index.js:280:10)
2024-03-05T10:34:48.908647391Z at /app/node_modules/express-fileupload/lib/index.js:35:14

@anjanaw
Copy link
Member

anjanaw commented Mar 5, 2024

Thanks @bfleisch . This can be because the images are too big. This came up when executing NN explainer for this use case as well.
I'm open to suggestions on how to solve this.

@anneliret
Copy link
Author

anneliret commented Mar 5, 2024 via email

@anjanaw
Copy link
Member

anjanaw commented Mar 5, 2024

The issue does not seem to come from the storage, its the API request body being too large.
Also this request body includes all the explanations as well in base64. A long term solution would be to look for a way to increase the request size.

Saving the interaction without images is a temporary fix, which i can do just for this use case (hardcoded on the chatbot). It will have to reverted after the user studies. If you are happy to do that, let me know.

@anneliret
Copy link
Author

anneliret commented Mar 5, 2024 via email

@anjanaw
Copy link
Member

anjanaw commented Mar 5, 2024

No. No need to regenerate the link. I will update you once the changes are deployed.

@anjanaw
Copy link
Member

anjanaw commented Mar 6, 2024

@anneliret please test and see if the issue still exists. I have tested once, and my responses were saved correctly.

@anneliret
Copy link
Author

anneliret commented Mar 6, 2024 via email

@anjanaw
Copy link
Member

anjanaw commented Mar 7, 2024

See if it is fixed now.
Analytics/Postman will not be working from these conversation logs. I will add a workaround for that soon.
But you can still check if the conversation is saved from the cockpit usecases view where runs incremented.

@anjanaw
Copy link
Member

anjanaw commented Mar 7, 2024

Analytics are also updated to ignore missing data. Please test and close. @anneliret

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants