Closed elvinhajizada closed 12 months ago
All check have passes
Integration tests are added
Just glanced over it, will review in more detail after some questions.
- Why did you call it prototype_lif? It seems like a 3F learning LIF neuron. Also, is this usable in general? If yes, we should not hide it inside clp folder.
- What does nsm mean? The process is called Readout, I could not figure out what nsm stands for.
- I saw print statements in the PyNoveltyDetectorModel. I assume they are left overs from the development?
@PhilippPlank
Issue Number: #706
Objective of pull request: As a user, I want to be able to use the CLP algorithm in Lava, to learn continually from a stream of data in Loihi chip.
Pull request checklist
Your PR fulfills the following requirements:
flakeheaven lint src/lava tests/
) and (bandit -r src/lava/.
) pass locallypytest
) passes locallyPull request type
Please check your PR type:What is the current behavior?
What is the new behavior?
This PR is the first step in the full implementation of CLP. With this version of CLP, three base processes are implemented in the CPU backend and have been tested in terms of the following behaviors:
In addition, several integration tests are written to test if LearningDense -> PrototypeLIF -> NoveltyDetection intengration works properly
Does this introduce a breaking change?
Supplemental information