lava-nc / lava

A Software Framework for Neuromorphic Computing
https://lava-nc.org
Other
529 stars 136 forks source link

Suggestion: Have LearningConnection as a normal Class and not as a Process #776

Open SveaMeyer13 opened 10 months ago

SveaMeyer13 commented 10 months ago

Issue: LearningDense and LearningSparse contain code duplicates as they primarily inherit from LearningConnectionProcess but also need the functionality of the base class of Sparse and Dense, respectively. We cannot call init() of both super classes since both of them in turn inherit from and initialize AbstractProcess, which would lead to overwritten proc_params.

Suggested Solution: Have LearningConnection (instead of LearningConnectionProcess) as a normal Class that does not inherit from AbstractProcess. Have LearningDense/LearningSparse inherit from Sparse (primarily) and from LearningConnection.