open-sauced / app

🍕 Insights into your entire open source ecosystem.
https://pizza.new
Apache License 2.0
381 stars 202 forks source link

feat: ROSS chart in repo pages #3586

Closed zeucapua closed 1 week ago

zeucapua commented 2 weeks ago

Description

Implements ROSS (Rate of Self Selection) Dev Stat for repo pages, showing the spread between new, recurring, and internal contributors. This new chart replaces the ContributorChart while taking the total and average per day values calculated from the contributor histogram data.

Related Tickets & Documents

Closes #3548

Mobile & Desktop Screenshots/Recordings

image

Steps to QA

Tier (staff will fill in)

[optional] What gif best describes this PR or how it makes you feel?

rapper-bars-gif-by-vh1

netlify[bot] commented 2 weeks ago

Deploy Preview for design-insights ready!

Name Link
Latest commit 3fff12e4b5b03be8574c136c75c356dac08d46f2
Latest deploy log https://app.netlify.com/sites/design-insights/deploys/667461ff7256170008494f1b
Deploy Preview https://deploy-preview-3586--design-insights.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] commented 2 weeks ago

Deploy Preview for oss-insights ready!

Name Link
Latest commit 3fff12e4b5b03be8574c136c75c356dac08d46f2
Latest deploy log https://app.netlify.com/sites/oss-insights/deploys/667461fffded32000816297d
Deploy Preview https://deploy-preview-3586--oss-insights.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

brandonroberts commented 1 week ago

Looks great! The weeks are in reverse order

image
zeucapua commented 1 week ago

More of an aesthetic, but the graph x axis seems real short (looks like it was before so maybe a moot point). but the bars are super chonky compared to the other graphs and what it was before.

Before ROSS index

CleanShot 2024-06-20 at 11 49 55

With ROSS index

CleanShot 2024-06-20 at 11 48 31

I think this is due to the difference in delta between the two charts. Before it could go from 1 to 20. Now the delta is smaller so the chart is probably rendering it like that.

bdougie commented 1 week ago

Is it possible to not show items, like "internal" when they are 0

image

zeucapua commented 1 week ago

Is it possible to not show items, like "internal" when they are 0

image

Now fixed! image