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

Consider using separate thread for loading asset/project #46

Closed
JamesTKhan opened this issue May 24, 2022 · 1 comment · Fixed by #92
Closed

Consider using separate thread for loading asset/project #46

JamesTKhan opened this issue May 24, 2022 · 1 comment · Fixed by #92
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@JamesTKhan
Copy link
Owner

When a larger project is loading all you see is a black screen until it has finished loading. Consider loading on a separate thread and displaying a loading message when opening a project. I suspect the main culprit is asset loading for the delay.

@JamesTKhan JamesTKhan changed the title Consider use separate thread for loading asset/project Consider using separate thread for loading asset/project May 24, 2022
@JamesTKhan JamesTKhan added enhancement New feature or request good first issue Good for newcomers and removed good first issue Good for newcomers labels May 25, 2022
@JamesTKhan
Copy link
Owner Author

JamesTKhan commented Jul 20, 2022

This is on the back burner. A major issue with implementing this is running GL calls on a separate thread. The asset loading process makes GL calls for somethings like model instances I believe. So the whole project/asset loading process would need reworking.

A solution may be adding two load methods. A pre-load method that handles all loading that can be done on another thread. Then the normal load method which will do all the GL calls on main thread.

@JamesTKhan JamesTKhan self-assigned this Jul 26, 2022
@JamesTKhan JamesTKhan added this to the 0.4.0 Release milestone Jul 26, 2022
@JamesTKhan JamesTKhan linked a pull request Jul 26, 2022 that will close this issue
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant