search

oss-aspen / 8Knot

Dash app in development to serve open source community visualizations using GitHub data from Augur. Hosted app: https://eightknot.osci.io
MIT License
49 stars 60 forks source link

PR and issue assignment optimization #610

Closed cdolfi closed 8 months ago

cdolfi commented 9 months ago

Might need more updates to optimize more but this makes a HUGE difference

JamesKunstle commented 9 months ago

@cdolfi This limits the range of dates for which the graph is rendered, correct? So, we're visualizing the data the same way, but we're visualizing less data at once by default with this change.

cdolfi commented 9 months ago

@JamesKunstle Yes precisely, the initial rendering takes so much less time than before

JamesKunstle commented 9 months ago

@cdolfi In testing, this is much better than before. I think we still ought to aggregate small values of assignments, and let the user know that un-aggregating will have a browser performance penalty. For the Kubernetes org, I get the following:

Screenshot 2024-01-08 at 4 27 06 PM

Which is very slow to interact with.

cdolfi commented 9 months ago

@JamesKunstle That makes sense to me, can you detail what you have in mind in the issue linked? Think itd be good to do in a separate PR (and would not close said issue w this PR)

JamesKunstle commented 9 months ago

Yeah I definitely can

JamesKunstle commented 9 months ago

613 opened this issue, and #614 this one

JamesKunstle commented 9 months ago

you'll have to rebase the branch manually for it to be merged.