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

Update Quickstart (UI and python) #1145

Merged
merged 5 commits into from
Jan 30, 2024
Merged

Update Quickstart (UI and python) #1145

merged 5 commits into from
Jan 30, 2024

Conversation

brilee
Copy link
Contributor

@brilee brilee commented Jan 30, 2024

I noticed that .count() wasn't a method on Dataset, so I promoted it out of the duckdb implementation. I also reduced GTE batch size because the progress bar was updating only once every 30 seconds for me, which was just slow enough to make me think it had bugged out.

I have not rerecorded videos yet but hopefully some early feedback will prevent me from recording too many unusable
takes :)

The Quickstart deemphasizes PII and adds clustering/deleting sections. The Python quickstart demos some more powerful filters, like grabbing all clusters with a name at once.

@dsmilkov dsmilkov enabled auto-merge (squash) January 30, 2024 13:55
docs/getting_started/quickstart.md Show resolved Hide resolved
docs/getting_started/quickstart.md Show resolved Hide resolved
docs/getting_started/quickstart.md Show resolved Hide resolved
docs/getting_started/quickstart.md Show resolved Hide resolved
- Running a [signal](../signals/signals.md) (e.g. PII detection, language detection, text
statistics, etc.)
- Running a [concept](../concepts/concepts.md) (e.g. profanity, sentiment, etc. or a custom concept
that you create)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

docs/getting_started/quickstart_python.md Show resolved Hide resolved
docs/getting_started/quickstart_python.md Show resolved Hide resolved
docs/getting_started/quickstart_python.md Show resolved Hide resolved
docs/getting_started/quickstart_python.md Show resolved Hide resolved
lilac/data/dataset.py Show resolved Hide resolved
@dsmilkov dsmilkov merged commit e77d1bb into main Jan 30, 2024
4 checks passed
@dsmilkov dsmilkov deleted the quickstart branch January 30, 2024 14:31
brilee added a commit that referenced this pull request Jan 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants