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.

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

Closed
wants to merge 6 commits into from

Conversation

diimdeep
Copy link

Drop torch, do not load whole file into memory, process files in parallel and use separate threads for r/w

q = queue.Queue(maxsize=2)

def writer():
while True:

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.

Copy link
Author

Choose a reason for hiding this comment

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

fixed!

@gjmulder gjmulder added enhancement New feature or request performance Speed related topics labels Mar 15, 2023
@sw
Copy link
Contributor

sw commented Mar 19, 2023

The python dependencies in .devops/full.Dockerfile should also be updated, will conflict with my PR #293.

@ggerganov
Copy link
Owner

This looks like a very useful addition. Lets give it a priority and merge after resolving the conflicts

@tim-gromeyer
Copy link

@ggerganov Any update on this? Because I really do not want to install pytorch on my system (because of memory).

@Green-Sky Green-Sky added the high priority Very important issue label Mar 26, 2023
@sw sw mentioned this pull request Mar 27, 2023
16 tasks
@ggerganov
Copy link
Owner

This is probably too outdated so closing for now

@ggerganov ggerganov closed this Mar 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request high priority Very important issue performance Speed related topics
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants