search

ioam / topographica

A general-purpose neural simulator focusing on topographic maps.
topographica.org
BSD 3-Clause "New" or "Revised" License
53 stars 32 forks source link

coord_mapper seems to confuse sparse matrix creation in sparse implementation #641

Closed mjabri closed 8 years ago

mjabri commented 8 years ago

Issue discussed here https://github.com/ioam/topographica/issues/640 and can be reproduced using gcal_sparse.ty.

mjabri commented 8 years ago

Ok, I think i found the problem and validated my fixes with gcal_sparse.ty with and without coord_mapper. So now all initial weights with and w/o coord_mapper and on sparse and non-sparse all all identical, and results (activities and weights) after 10000 iter are same. I will merge all my changes (incl. getting gpu weights back to host) once I am on latest release and will raise a pull request.

philippjfr commented 8 years ago

Okay, sounds great. I still haven't been able to get the GUI working, which isn't helped by the fact that I recently switched to OSX and can't get Tk running without segfaulting at all. Will have to boot into linux and track the issue down. I'm looking forward to your fixes as I'm anxious to start using the GPU implementation myself for some of my larger models.

mjabri commented 8 years ago

Great. Thanks. Looking forward to getting to latest version.