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.

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

Youtube notebook bug #40

Closed
eddyxu opened this issue Apr 23, 2023 · 0 comments
Closed

Youtube notebook bug #40

eddyxu opened this issue Apr 23, 2023 · 0 comments
Labels
bug Something isn't working

Comments

@eddyxu
Copy link
Contributor

eddyxu commented Apr 23, 2023

From adamb on Discord

i found a minor bug in the current demo notebook:
YouTubeVideo(top_match["url"].split("/")[-1], start=top_match["start"])

should be
YouTubeVideo(top_match["url"].split("/")[-1], start=int(top_match["start"]))
@eddyxu eddyxu added the bug Something isn't working label Apr 23, 2023
changhiskhan added a commit that referenced this issue Apr 25, 2023
@changhiskhan
Merge pull request #45 from lancedb/changhiskhan/notebook-fix
44ea687 
Minor notebook fix. Closes #40
raghavdixit99 pushed a commit to raghavdixit99/lancedb that referenced this issue Apr 5, 2024
@changhiskhan
Minor notebook fix. Closes lancedb#40
64d90c2
raghavdixit99 pushed a commit to raghavdixit99/lancedb that referenced this issue Apr 5, 2024
@changhiskhan
Merge pull request lancedb#45 from lancedb/changhiskhan/notebook-fix
0f5f6b0 
Minor notebook fix. Closes lancedb#40
alexkohler pushed a commit to alexkohler/lancedb that referenced this issue Apr 20, 2024
@eddyxu
Merge two RecordBatchs or StructArrays (lancedb#40)
baea4e8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@eddyxu