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

Performance optimization of multidimensional array access #535

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jan-zajic
Copy link

Based on discussion in thread https://discourse.numenta.org/t/performance-optimization-of-htm-java/2652 i made optimization of get/set value/slice methods in htm.java.

@jan-zajic
Copy link
Author

@ibobak @cogmission can you have a look at this?

@jan-zajic
Copy link
Author

@cogmission i had signed Contributor License with my username but validation failed, is validar check something else than username?

@cogmission
Copy link
Collaborator

cogmission commented Feb 7, 2018

Hi @jan-zajic,

We have to wait for @rhyolight (Matt Taylor) to approve your License form.

@rhyolight can you have a look when you get a chance?

@rhyolight
Copy link
Member

Sorry I missed this. Will do next business day.

@rhyolight
Copy link
Member

I think this is ok to merge now.

@cogmission
Copy link
Collaborator

Hi @jan-zajic,

I apologize for the delay in getting back to this PR, are you still around? Let me know because I have some changes I would like to make before this PR is merged.

Cheers,
David

@jan-zajic
Copy link
Author

Hi @cogmission, i was a little busy but i'am back, what changes do you mean?

@jan-zajic
Copy link
Author

@cogmission, are you still around? I am. Or is htm.java dead (Latest commit on 14 Sep 2017)?

@cogmission
Copy link
Collaborator

@jan-zajic I'm still here... Been real busy w/my day job which also used to allot time for HTM.Java development (since they are also sponsors), but my role has expanded and left less time for HTM.Java recently. I will be getting back very soon though because I have projects that require it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants