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

Use OpenMP or SIMD to optimize observers #87

Closed
Bam4d opened this issue Dec 12, 2020 · 2 comments
Closed

Use OpenMP or SIMD to optimize observers #87

Bam4d opened this issue Dec 12, 2020 · 2 comments
Labels
help wanted Extra attention is needed

Comments

@Bam4d
Copy link
Owner

Bam4d commented Dec 12, 2020

The vector observer and observers using vulkan for rendering have certain loops which can and should be easily parallelized which would significantly speed up these observers.

RGBA to RGB optimize this:
https://github.com/Bam4d/Griddly/blob/develop/src/Griddly/Core/Observers/Vulkan/VulkanDevice.cpp#L541

Optimize these loops here:
Should probably iterate over the objects and convert the x,y coordinates to make this significantly faster on it's
https://github.com/Bam4d/Griddly/blob/develop/src/Griddly/Core/Observers/VectorObserver.cpp#L58

@Bam4d Bam4d added the help wanted Extra attention is needed label Dec 12, 2020
@Bam4d
Copy link
Owner Author

Bam4d commented Jan 16, 2021

Some optimizations have taken place in #95, but no SIMD/OPENMP

@Bam4d
Copy link
Owner Author

Bam4d commented Jan 18, 2021

No need to do OpenMP and SIMD anymore because other optimizations are much better!!! #100

@Bam4d Bam4d closed this as completed Jan 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant