indexed_gzip icon indicating copy to clipboard operation
indexed_gzip copied to clipboard

For unseekable files, can't build full index by reading the entire file

Open epicfaace opened this issue 3 years ago • 8 comments

For unseekable files, can't build full index by reading the entire file. I have to run build_full_index, but I can't just instead read the entire file to build the index (which ideally should be equivalent and should work as well). See https://github.com/pauldmccarthy/indexed_gzip/pull/106 for a reproduction.

epicfaace avatar Oct 19 '22 17:10 epicfaace

Hi @epicfaace, sorry - I'm on holiday until early November, so won't be able to look into this until then. Is it by any chance related to #74?

pauldmccarthy avatar Oct 20 '22 06:10 pauldmccarthy

No worries! I don't know if it's related to #74, but perhaps it is. I'll take a look.

cc @wwwjn -- I think this is the issue we're stuck on

epicfaace avatar Oct 20 '22 19:10 epicfaace

Hi @pauldmccarthy, do you have some bandwidth to look into this issue or its there any progress? Thanks a lot!

wwwjn avatar Nov 02 '22 17:11 wwwjn

He's on holiday 🙂 but we should probably try to do as much as we can to help in the meantime -- maybe debugging, trying to find a minimal reproducible example, seeing if we can also figure out a fix

epicfaace avatar Nov 03 '22 02:11 epicfaace

Hi @pauldmccarthy , wanted to see if you might have gotten a chance to look into this issue!

epicfaace avatar Nov 30 '22 22:11 epicfaace

@epicfaace Sorry, I've had a busy couple of months. I might find time to look at this over the Christmas break, but feel free to dive into it if either of you have time,.

pauldmccarthy avatar Dec 05 '22 09:12 pauldmccarthy

@pauldmccarthy Wish you have a great holidays! Is there any update on this issue? I try to take a look about this issue but can not find a solution to this...

wwwjn avatar Jan 04 '23 15:01 wwwjn

Hi @wwwjn, sorry, I really don't have time to look at non-critical issues at the moment. But I'm happy to review PRs if you have the time to do the work.

pauldmccarthy avatar Jan 23 '23 09:01 pauldmccarthy