Page unresponsive when loading report of many repos with huge date gap

[eugenepeh]

As per title.

For e.g. If we set up the dashboard demo to show the analysis for period of a year, the page will become unresponsive for a long while (even though most of the actual contributions are within 3 months) which makes it near unusable.

[emer7]

I'll look into this

[emer7]

Sorry for the late response

The culprit is from this line of code.
For larger date gap, there will be more commits being pushed to user.commits

v_summary.js:344

        const { date } = commit;
        if (date >= sinceDate && date <= untilDate) {
          user.commits.push(commit);
        }

Just a quick thought, I am thinking of asynchronous implementation.

[eugenepeh]

hmm, can we do away with empty commits as a whole?

[emer7]

hmm, can we do away with empty commits as a whole?

If I understood correctly, merging the empty commit become 1 big empty commit?

If yes, I think it could be done, it will need change to how report is generated (empty commit is lumped together. I need to look into how it is done) and also change v_summary.js#getSlicePos

[eugenepeh]

If yes, I think it could be done, it will need change to how report is generated (empty commit is lumped together. I need to look into how it is done) and also change v_summary.js#getSlicePos

Hmm actually what I thought of is that let the commits stores nothing other than non-empty commits.
You can try it out, research on this issue on how can we be more memory/computational efficient and present your findings in your PR.