search

dgschwend / netscope

Neural network visualizer and analyzer
http://dgschwend.github.io/netscope
164 stars 72 forks source link

Concat error since last update #12

Closed rejay015 closed 7 years ago

rejay015 commented 7 years ago

Hi dgschwend,

Glad to contact you again. I've found a concat parameter error of this netscope project, since the project was updated 7days ago. You can find it at "fire2/concat", the output channel should be 128 but was demonstrated as 192.

See the link or picture below. http://dgschwend.github.io/netscope/#/preset/zynqnet

concat_error_at_fire2_concat

dgschwend commented 7 years ago

Hi rejay015 Sorry for the delay, I'm currently on vacation. I introduced a bug in analyzer.coffee in the last commit. Before the line d.chIn += p.analysis.chOut for p in n.parents you need to add one line saying d.chIn =0 That should fix it. I'm currently on my phone, so cannot recompile, but will fix this next week. Thanks for spotting + reporting! David

rejay015 commented 7 years ago

Have a nice vacation, I guess it's a long journey, out of Europe ^_^

dgschwend commented 7 years ago

fixed in 234e8f7d1d37ba1f8dc9b487c7e78cdb6931acc1