hudon / spike

Brain Simulator Parallelization
http://nengo.ca/
1 stars 1 forks source link

EnsembleProcess instance has no attribute 'add_learned_termination' #59

Closed RobertElder closed 10 years ago

RobertElder commented 10 years ago

On branch test-learning

ERROR: Output from '/usr/bin/python2 /home/robert/spike/test/nengo_tests/test_learning.py /home/robert/spike/test/../src' and '/usr/bin/python2 /home/robert/spike/test/nengo_tests/test_learning.py /home/robert/spike/test/../examples/new-theano' does not match. make: *\ [test] Error 1

Run this command to see error:

robert@robert-ubuntu:~/spike$ /usr/bin/python2 /home/robert/spike/test/nengo_tests/test_learning.py /home/robert/spike/test/../src Made populations: 2.16073989868 Traceback (most recent call last): File "/home/robert/spike/test/nengo_tests/test_learning.py", line 23, in net.learn(pre='A', post='B', error='error1') File "/home/robert/spike/test/../src/nef_theano/network.py", line 351, in learn return post.add_learned_termination(name=pre_name, pre=pre, error=error, AttributeError: EnsembleProcess instance has no attribute 'add_learned_termination'

RobertElder commented 10 years ago

branch test-learning-array has the same issue.

gretac commented 10 years ago

So this bug is really there because we are not supporting learning right now. I have not included it in the port so far. Thing is, I think it is better for us to make the distribution a higher priority than this right now (mainly because I do not yet know how to port this easily). Let's discuss in our meeting this weekend whether we all agree to move this out of the current milestone and take care of this later. That would also mean not considering test-learning and test-learning-array as part of the tests required for this milestone.

RobertElder commented 10 years ago

Sounds good.