Log in
Log into community
Find answers from the community
s
F
Y
a
P
3,278
View all posts
Related posts
Did this answer your question?
π
π
π
Powered by
Hall
Inactive
Updated 8 months ago
0
Follow
@Logan M , I hope you are looking into
@Logan M , I hope you are looking into
0
Follow
A
Anurag Agrawal
8 months ago
Β·
@Logan M , I hope you are looking into this? π¬
L
A
14 comments
Share
Open in Discord
L
Logan M
8 months ago
Feel free to make a PR for it π
A
Anurag Agrawal
8 months ago
Not again!
A
Anurag Agrawal
8 months ago
Just kidding, on it. Thanks for checking π
L
Logan M
8 months ago
I think llama-hub had some duplicate readers that were also in llama-index, and when they merged, one version just got picked
A
Anurag Agrawal
8 months ago
Yeah, that's what I thought! Np, i'll submit this PR soon
A
Anurag Agrawal
8 months ago
Submitted
https://github.com/run-llama/llama_index/pull/11913
A
Anurag Agrawal
8 months ago
@Logan M , this is what I see after my PR got merged:
What does this mean?
Attachment
L
Logan M
8 months ago
Oh hmm, the auto publisher seems to be having some issue
A
Anurag Agrawal
8 months ago
Okay I tried to test my changes by installing from main but it didn't work. So, I'll have to wait until this is fixed to do my testing?
L
Logan M
8 months ago
yea seems like something is borked with our pypi setup, oof
L
Logan M
8 months ago
debugging with the team lol
A
Anurag Agrawal
8 months ago
Thanks @Logan M , please let me know when I can test my changes π
L
Logan M
8 months ago
Should be published π
A
Anurag Agrawal
8 months ago
Yes, it worked! Thanks @Logan M !!
Add a reply
Sign up and join the conversation on Discord
Join on Discord