lava-nc / lava-dl

Deep Learning library for Lava

Home Page:https://lava-nc.org

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Binning Summation

pbonazzi opened this issue · comments

Can you clarify this binning mode summation?
Shouldn't we use torch.index_add_ ? Otherwise there is no difference between binningmode SUM and OR

https://github.com/lava-nc/lava-dl/blob/185edfe9fb035e13c24eec063a4971ccb0956d3a/src/lava/lib/dl/slayer/io.py#L234C1-L241C38

Hi @pbonazzi I am guessing you are meaning to say SUM mode and OR mode. Yes the current implementation seems to be a bug. Would you like to make a correction PR?

Yes, I can do it over the weekend.