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

Native Arrays #51

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Native Arrays #51

wants to merge 9 commits into from

Conversation

danini-the-panini
Copy link
Owner

@danini-the-panini danini-the-panini commented Mar 25, 2017

First attempt at performance improvement. Still need to benchmark this, but I've replaced all Arrays with Native (Fiddle malloc'd) arrays where possible.

In some cases, native arrays actually make things slower. I need to find out where these places are and try and find the best compromise.

E.g, I tried to use native arrays for Matrix elements, but that slowed things down because they're modified and read in Ruby-land a lot, and with pack/unpack that's a lot of unnecessary string/array allocations.

I haven't really noticed anything running faster yet. I might need to see where the real performance hits are happening. Theoretically, passing big data arrays (think, all those vertices) from Ruby to C will be more efficient now (I think? I hope!).

Daniel Smith added 4 commits March 25, 2017 21:04
@danini-the-panini danini-the-panini modified the milestones: 1.0.0, 0.2.1 Apr 1, 2017
Base automatically changed from master to main February 4, 2021 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant