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

Improved model creation #224

Open
wants to merge 21 commits into
base: main
Choose a base branch
from
Open

Conversation

sfc-gh-kschmaus
Copy link
Collaborator

@sfc-gh-kschmaus sfc-gh-kschmaus commented Jan 16, 2025

This PR modifies semantic model creation. The main change is enabling support for cortex search service integration. Some small changes are made to AI description generation. This removes the user's selection of the number of sample values. Sample values are removed for non string-columns. For string columns with 8 or less distinct sample values, all are included as sample values. For string columns with more than 8 distinct values, no sample values are used, and the user can setup cortex search service integration for those columns.

@sfc-gh-kschmaus sfc-gh-kschmaus changed the base branch from customer-eval to main January 16, 2025 23:12
@sfc-gh-kschmaus sfc-gh-kschmaus changed the base branch from main to customer-eval January 16, 2025 23:13
Base automatically changed from customer-eval to main January 17, 2025 19:43
@sfc-gh-kschmaus sfc-gh-kschmaus force-pushed the improved-model-creation branch from 4c0984c to 87e8b07 Compare January 21, 2025 18:53
@sfc-gh-kschmaus sfc-gh-kschmaus force-pushed the improved-model-creation branch 3 times, most recently from a4c5e29 to 7d28cd2 Compare January 21, 2025 21:31
@sfc-gh-kschmaus sfc-gh-kschmaus force-pushed the improved-model-creation branch from 7d28cd2 to 9989ead Compare January 21, 2025 21:52
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