SimonBuxx / LogiJS

Discover and create logic circuits
https://logijs.com
GNU General Public License v3.0
42 stars 6 forks source link

BUG: outputsInv and inputsInv in the .json's have too many entries #13

Closed SimonBuxx closed 6 years ago

SimonBuxx commented 6 years ago

The inverted inputs/outputs arrays have far too many entries. This seems to be a new problem, but isn't effecting the functionality.

SimonBuxx commented 6 years ago

unbenannt As an example, the 2-Bit counter has one input and two outputs but three inputs and six outputs are listed in the vector as not inverted.

gatchrat-zz commented 6 years ago

image works correctly for me. tested in chrome and edge

SimonBuxx commented 6 years ago

unbenannt Try loading the file "cpu" and immediatly save it and look under customs. All wrong for me in edge.

gatchrat-zz commented 6 years ago

https://github.com/gatchrat/LogiJS/commit/ff75566aa7203f6deb173a2eadc2c04a2375d6e1 fixed. Old save files even get fixed when loading and saving them again.

SimonBuxx commented 6 years ago

Thank you!

This issue is now done with my last commit.