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

BugFix for ContextDatastore #43

Closed
MagicShel opened this issue Mar 6, 2023 · 1 comment
Closed

BugFix for ContextDatastore #43

MagicShel opened this issue Mar 6, 2023 · 1 comment
Assignees

Comments

@MagicShel
Copy link
Collaborator

MagicShel commented Mar 6, 2023

reactor.core.Exceptions$ErrorCallbackNotImplemented: java.lang.NullPointerException: messageEventData not set
Caused by: java.lang.NullPointerException: messageEventData not set
	at es.thalesalv.gptbot.adapters.data.ContextDatastore.lambda$1(ContextDatastore.java:36)
	at java.base/java.util.Optional.orElseThrow(Optional.java:403)
	at es.thalesalv.gptbot.adapters.data.ContextDatastore.getMessageEventData(ContextDatastore.java:36)
	at es.thalesalv.gptbot.application.service.ChatGptModelService.generate(ChatGptModelService.java:37)
@thaalesalves
Copy link
Collaborator

Like we discussed, we can pass that as method parameters to avoid using the thread context, since we're going reactive and that's throwing problems. We can use this as a temporary solution until we refactor the persona system. Then this thread context won't even be necessary anymore. I'll implement the fix and we can expand on the idea of this persona system so this is fixed once and for all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants