IDSIA / brainstorm

Fast, flexible and fun neural networks.
Other
1.3k stars 152 forks source link

Change the signature of `copy_to` #51

Closed Qwlouse closed 8 years ago

Qwlouse commented 8 years ago

Currently the signature is copy_to(dest, src), following the numpy.copy_to signature. But the ordering is confusing. I vote for changing it to copy_to(src, dest) to make it more intuitive, and because I don't think it is that important to be consistent with this numpy method (which is rarely used anyways). If you think that would still be confusing we could rename the method to more clearly deviate from the numpy one.