Skip to content

small fixes#725

Merged
changhiskhan merged 4 commits intolance-format:mainfrom
gsajko:main
Mar 28, 2023
Merged

small fixes#725
changhiskhan merged 4 commits intolance-format:mainfrom
gsajko:main

Conversation

@gsajko
Copy link
Copy Markdown
Contributor

@gsajko gsajko commented Mar 23, 2023

added dependencies, and place to provide open ai keys

Copy link
Copy Markdown
Contributor

@changhiskhan changhiskhan left a comment

Choose a reason for hiding this comment

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

Thanks for the PR. I'm happy to share the dataset btw

Comment thread notebooks/youtube_transcript_search.ipynb Outdated
@gsajko
Copy link
Copy Markdown
Contributor Author

gsajko commented Mar 23, 2023

are they big? I was thinking about including it into repo, so other people trying tutorial wouldn't need to use their openai credits. I think it costs around ~15$ to create embeddings. Cheers

@changhiskhan
Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Contributor

@changhiskhan changhiskhan left a comment

Choose a reason for hiding this comment

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

thanks!

@changhiskhan changhiskhan merged commit 6b1bf94 into lance-format:main Mar 28, 2023
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