-
Notifications
You must be signed in to change notification settings - Fork 10.2k
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
Streaming conversion with no torch #176
Conversation
q = queue.Queue(maxsize=2) | ||
|
||
def writer(): | ||
while True: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
while True
? Does this function ever return? I don't know if the function exists but maybe something like while !q.atEnd()
Please correct me if I'm wrong. I haven't worked with Python since a year or so.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed!
The python dependencies in .devops/full.Dockerfile should also be updated, will conflict with my PR #293. |
This looks like a very useful addition. Lets give it a priority and merge after resolving the conflicts |
@ggerganov Any update on this? Because I really do not want to install pytorch on my system (because of memory). |
This is probably too outdated so closing for now |
Drop torch, do not load whole file into memory, process files in parallel and use separate threads for r/w