nengo / nengo-spa

Implementation of the Semantic Pointer Architecture for Nengo
https://www.nengo.ai/nengo-spa/
Other
20 stars 5 forks source link

Naming inconsistencies #199

Closed jgosmann closed 6 years ago

jgosmann commented 6 years ago

Not all modules are named like the main class they contain and different abbreviated forms are used (CircularConvolution, cconv, etc). All of this should follow a coherent structure.

jgosmann commented 6 years ago

Here is my concrete proposal for renames (based on #198):

This makes all module names to match the main class in those modules (except where modules are a collection of multiple things of equal importance such as nengo_spa.math or nengo_spa.networks.selection). Most of these module names are never referenced by the user as all these classes are usually availlible from the parent module (e.g. nengo.networks) if not the root nengo_spa module. Furthermore, HrrAlgebra is shorter than CircularConvoluitionAlgebra and maybe expresses more that it is elementwise addition and circular convolution that together define the algebra.

@Seanny123 @tcstewar

Seanny123 commented 6 years ago

Legit.