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.

Sign up for GitHub

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

Jump to bottom

Knn dev #374

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

Knn dev #374

wants to merge 38 commits into from

Conversation

cogmission
Copy link
Collaborator

Fixes #132

Work In Progress... (i.e. Merge not yet requested)

  • Create new Parameters for application to KNNClassifier construction
  • Create new Constants class to begin getting rid of open-ended strings in HTM.java
  • Create new static Builder for fluent construction of the many KNNClassifier parameters
  • Enhance the BeanUtil to accept classes with no mutator fields (for Parameters.apply() method)
  • Extends Parameters test to include test of new BeanUtil and new Enum properties
  • Add new array handling methods to ArrayUtils to equal methods called in KNNClassifier
  • Complete port of KNNClassifier unit test
  • Complete port of all KNNClassifier methods

@cogmission
modified build and pom files, Added new Parameter
3e295eb
@cogmission
Added new KNNClassifier class and test
219a023
@cogmission
Added new Enums and Constants
c00eff7
@cogmission
Modified BeanUtil to work with private fields, added test to ensure p…
add191e 
…arameter application through new introspective code
@cogmission
Added assertions which validate enum compatibility
ec5da11
@cogmission
Added test class (BeanUtilTest), and added test methods
27e3ff1
@cogmission
Added test for builder
7238233
@cogmission
Added new utility methods: rotateLeft(), rotateRight(), ravel(), unra…
e2d4f21 
…vel(), reshape() to ArrayUtils
@cogmission
Added new NN Matrix implementation
9e8d2d1
@cogmission
Merge remote-tracking branch 'upstream/master' into knn_dev
00ee815
@cogmission
more work on knn
12a1f99
@cogmission
more work on classifier
8db7a17
@cogmission
added more code and merged in ArrayUtil additions
9223fcd
@cogmission
changed input datatype to double added new ArrayUtils method for change
2002c4c
@cogmission
added test files to resources dir
6c295da
@cogmission
Add test for new PCA data generation
3ada066
@cogmission
Remove reference to test resources from source
6e533e1
@cogmission
add tests for new ArrayUtils functions
aeb60ed
@cogmission
Add new TupleCollector and change test to use Streams
c7d8f65
@cogmission
Added matrix transpose method
d9d1d79
@cogmission
added original test code back in
7d62579
@cogmission
remove working code
a3ecca1
@cogmission
added test for another branch to dot product
edf9065
@cogmission
Renamed an ArrayUtil method, added a subtract method to support knn dev
0b58b3d
@cogmission
Added matrix dot with 2d to 1d, added new transpose, removed unused libs
4782e40
@cogmission
Add completed sparsifyVector test
2f502d5
@cogmission
Added more ArrayUtils tests (setRangeTo()), and finished logic branch…
de02de1 
… in KNNClassifier
@cogmission
Added la4j library
5ac8ec8
@cogmission
Added new linear matrix lib
7db47f6
@cogmission
add changes to .classpath
37accba
@cogmission cogmission closed this Sep 30, 2016
@cogmission cogmission reopened this Sep 30, 2016
@coveralls
Copy link

coveralls commented Sep 30, 2016 •
edited

Coverage Status

Coverage decreased (-37.7%) to 44.796% when pulling 7e374e8 on cogmission:knn_dev into f7b2ea2 on numenta:master.

@rhyolight
Copy link
Member

@cogmission Is this stale? Should you close it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@cogmission @coveralls @rhyolight