AvarchLLC / EIPsInsight

EIPsInsight Webpage ver 2.0
https://eipui.vercel.app
5 stars 6 forks source link

Add trend line to `/Analytics` #56

Open SamWilsn opened 1 week ago

SamWilsn commented 1 week ago

I'd like to request a feature:

bitmap

Where the magenta line represents the number of open pull requests/issues, so we can see how we're doing overall with reviewing EIPs.

viwinkumarpadala commented 1 day ago

Hey @SamWilsn ! 👋

Please check out our website and let us know if it is clear or if we need to update anything.

Thanks again for making the request! 🙏 We really appreciate your contribution. Please continue to share feedback (or)make more requests like this and help us make the website even better! 🚀.

SamWilsn commented 1 day ago

So the magenta line is the total number of open issues + open prs?

viwinkumarpadala commented 1 day ago

So, Open PRs represent the number of PRs that were still open during that month and the Created PRs are those that have been newly opened in that month. (we are showing both the data in our plots)

SamWilsn commented 1 day ago

Er, sorry, I brainfarted there. I should have said: does the magenta line represent the number of open items (PRs on the PRs section, and issues on the issues section)?


Honestly, I think I misread the graph the first time I looked at it :sweat_smile:

The legend says "Closed PRs" and "Merged PRs". I interpreted that to mean "the number of pull requests opened/closed/merged during that month."

What I think you're actually showing is the absolute number of each of those items. Is that correct?


The checkbox doesn't toggle the line on and off.

SamWilsn commented 1 day ago

I'm kinda just noodling here, but what do you think of a graph like this instead:

eips-insights-graph

Key features:

viwinkumarpadala commented 23 hours ago

Hey @SamWilsn,

poojaranjan commented 14 hours ago

I liked the chart suggested by @SamWilsn. Let's see if this can be implemented.