-
Notifications
You must be signed in to change notification settings - Fork 1
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
Multi thread import #19
base: master
Are you sure you want to change the base?
Conversation
@cdmpants could you tests these changes some when you have the time |
Just going to mention that |
Definitely! I did a fair bit of research on that, and from what I found, I/O is one of the very few things that doesn't care as much as other stuff and I tested this by importing few models and some crashed, but they also crashed before these changes(last time I tested importing most of these models was before we moved to 3.1). So I'm guessing my laptop is scuffed since I run Arch Linux btw, or 3.1 introduce since issues with importing. I'll need someone else to verify this though, so I'm just waiting lol |
since it's not needed there anymore
This should make importing faster, especially on large models, as it should take only as long as the longest import of a single LOD to execute