search

thechiselgroup / biomixer

BioMixer
http://bio-mixer.appspot.com/
16 stars 13 forks source link

Long Viz Load #498

Closed everbeek closed 9 years ago

everbeek commented 9 years ago

Viz takes a while to load now. I bet it is the new checkbox update code. I probably need to make it less fancy, and super direct instead.

everbeek commented 9 years ago

It is only actually slow when the console is open, breakpoints exist, and are being skipped. This is a bad indication for performance, so I will still try to fix it.

It does an updateUI() call on each node addition. Finding other hot spots.

Ok, after looking at this some more, it doesn't seem to so bad. I discovered that the breakpoints were related to it when I happened to remove them all...so I don't know where they were. The slow loading may have been only a matter of where those were, as opposed to the changes I made to checkbox updating. I am closing this as unreplicable for now. If it's an issue in the future, it can be investigated then.