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.

Sign up for GitHub

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

Jump to bottom

Hidden LookML fields are included in Gemini-Pro Context #47

Closed

pjabl opened this issue Jun 13, 2024 · 0 comments · Fixed by #49

Labels

Contributor

pjabl commented Jun 13, 2024

explore-assistant-extension bug
Looker generateExploreUrl includes all the fields (measures and dimensions) returned by useLookerFields:
https://github.com/looker-open-source/looker-explore-assistant/blob/main/explore-assistant-extension/src/hooks/useLookerFields.ts
I believe we should exclude hidden fields, as they're not available within explore, and LLM should use those fields for building output to end user input.

How to reproduce the problem:

Mark some field that is included in the explore as hidden
Open Explore Assistant Extension in Looker
open browser console (ctrl + shift + I)
Enter a question / prompt in the Assistant window
U'll see the whole generated context sent to gemini-pro in the console output
Copy that context output and search for the field that is hidden , you should be able to find it

github-actions bot added need triage p3 labels

pjabl mentioned this issue

fix(extension): remove fields marked as hidden from measures and dimensions #49

Merged

LukaFontanilla closed this as completed in #49

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment