systemapic / systemapic.js

Systemapic Javascript API
https://systemapic.com
0 stars 3 forks source link

optimize bigslider #191

Closed knutole closed 8 years ago

knutole commented 8 years ago

Forslag til optimaliseringer

1. Draw gray area only once

Ser ut som hele det grå området blir redrawn for hver frame? Se denne console loggen:

        // Rebuild graph data
        this.years[year].forEach(function (d, i) {

            if ( d.Doy < day ) {
                console.log('rebuild graph data');  // <- 
                d.date = this._dateFromNo(i)
                this.graphData.push(d);
            }

        }.bind(this));

screen shot 2016-03-31 at 12 52 45 am

Antall loops per frame: slider-redraws

Forslag: draw gray area only once. Og minimere så mye som mulig operasjoner - spesielt looper og lesing av mye data. På animation, så bør kun den rød trekanten flytte på seg og den rød streken tegnes.

2. Fetching of allYears data

Jeg prøvde å gjøre noe lurt der, med å kun hente data én gang, men funka ikke. Jeg skal fikse dette. Skal også gjøre om på at bigSlider blir callet fra layer, det var en dårlig idé.

jorgenevil commented 8 years ago

Rebuild graph data endrer ikke på det grå området. Det er kun this.graphData som blir endret, og det inneholder kun årets data (maks 365 dager).