-
Notifications
You must be signed in to change notification settings - Fork 70
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
Considerably worse performance on Windows #172
Comments
Can you verify that they are using the same version? |
I don't think this was a great test, my internet seemed like it was struggling today. I'll run these again at some point but I don't need to be cluttering up your issue tracker. Does humanify do multiple passes, pick up where it left off or cache anything at the moment? M1 Pro
Windows 10
|
I think maybe I just misunderstood the way it's divvying up the files or there's an element of randomness to the ordering of it. |
@brianjenkins94 To my knowledge I don't think it has any cache/etc method implemented currently. |
I also noticed that this: https://github.com/jehna/humanify/blob/main/src/plugins/webcrack.ts#L9-L20 will re-convert (and bundle?) any other files that happen to be in the Workaround: async function unminify(filename, outputDir, plugins = []) {
ensureFileExists(filename);
const bundledCode = await fs.readFile(filename, "utf-8");
+ try { await fs.rm("intermediate", { "recursive": true }); } catch (error) {}
+ await fs.mkdir("intermediate");
- const extractedFiles = await webcrack(bundledCode, outputDir);
+ const extractedFiles = await webcrack(bundledCode, "intermediate"); |
This is just a casual observation that I noticed when running humanify on my Windows machine vs. my Mac, both using OpenAI as the backend.
I'll see if I can take some benchmarks, because Windows looked like it was going to take days and Mac looks like it's only going to take a couple of hours.
The text was updated successfully, but these errors were encountered: