Skip to content

fix: elaborate on configuring search index#53

Open
qugu wants to merge 1 commit intogramps-project:mainfrom
qugu:patch-3
Open

fix: elaborate on configuring search index#53
qugu wants to merge 1 commit intogramps-project:mainfrom
qugu:patch-3

Conversation

@qugu
Copy link
Contributor

@qugu qugu commented Nov 6, 2025

Follow-up on: https://gramps.discourse.group/t/ai-chat-with-openai-returns-an-error-on-successfull-call/8735/4

It might be better to call "enable" -> "configure" because that's what happens and then actually trigger the rebuild in the web UI

Follow-up on: https://gramps.discourse.group/t/ai-chat-with-openai-returns-an-error-on-successfull-call/8735/4

It might be better to call "enable" -> "configure" because that's what happens and then actually trigger the rebuild in the web UI
@DavidMStraub
Copy link
Member

Hmm... but if the model is not configured, semantic search is really disabled in the sense that the search page will not show the full-text/semantic toggle button. So I'm not sure this is clearer. But you're right, we enable it by configuring it (this is what was originally meant with "enabling ... can be as simple as setting the configuration option". Perhaps a combination?

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

Successfully merging this pull request may close these issues.

2 participants

Comments