Skip to content
This repository has been archived by the owner on Oct 31, 2020. It is now read-only.

Making a connection between ensembles of different dimensions should give an error if you don't specify the transform (or pre/post indexes) #390

Open
tcstewar opened this issue Apr 2, 2013 · 0 comments
Assignees

Comments

@tcstewar
Copy link
Contributor

tcstewar commented Apr 2, 2013

Right now if you do this:

net.make('A', 100, 3)
net.make('B', 100, 5)
net.connect('A', 'B')

the system default to this transform matrix:

[[1.0 0.0 0.0]
 [0.0 1.0 0.0]
 [0.0 0.0 1.0]
 [1.0 0.0 0.0]
 [0.0 1.0 0.0]]

This is horribly wrong and has led to confusing bugs.

My suggestion is that this should give an error, forcing you to specify either transform= or some combination of pre_index and post_index.

Another option might be to use the following as the default matrix in this case, but I think giving an error is the clearer option

[[1.0 0.0 0.0]
 [0.0 1.0 0.0]
 [0.0 0.0 1.0]
 [0.0 0.0 0.0]
 [0.0 0.0 0.0]]
@ghost ghost assigned tcstewar Apr 2, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

No branches or pull requests

1 participant