Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Metrics V2 #246

Merged
merged 129 commits into from
Oct 22, 2024
Merged

Metrics V2 #246

merged 129 commits into from
Oct 22, 2024

Conversation

IsaacMilarky
Copy link
Collaborator

@IsaacMilarky IsaacMilarky commented Oct 4, 2024

Metrics V2

Problem

Metrics has not reached all of the goals for the V2 version of the code.

Solution

Reach all of the remaining goals for V2

V2 Goals:

  • COCOMO front-end
  • Libyears front-end l
  • Age of Project Front-end
  • Predominant Languages Front-end
  • Libyears backend

V2 Stretch Goals:

  • Average Commit Counts per PR - Front-end
  • Average Issue Resolution Time - Front-end

Result

Summary:

  • Added Libyears to backend
  • Added predominant languages chart
  • Increased global timeout from 20 to 120
  • Added project age toggle
  • Added Libyear visualization
  • Added DRYness visualization
  • Added top committers bar graph
  • Added COCOMO cost estimate bar graph
  • Added COCOMO time estimate bar graph
  • Added COCOMO contributor estimate bar graph
  • Added COCOMO language pie chart
  • Refactor format methods for pygals
  • Added fix for rate limit issues

Test Plan

(Write your test plan here. If you changed any code, please provide us with
clear instructions on how you verified your changes work.)

Signed-off-by: sachin-panayil <[email protected]>
Signed-off-by: sachin-panayil <[email protected]>
This was linked to issues Oct 17, 2024
Add Logic to Handle if Key Encounters GitHub API Rate Limit
Added Frontend and Backend for Average Issue Resolution Time Graph
decause-gov
decause-gov previously approved these changes Oct 18, 2024
Copy link
Contributor

@decause-gov decause-gov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM +1 🚢 2️⃣

sachin-panayil
sachin-panayil previously approved these changes Oct 22, 2024
Copy link
Contributor

@sachin-panayil sachin-panayil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did testing on this and everything LGTM! i did put out a PR for some minor bug fixes but after that gets merged in, i believe its ready to ship 🚢 💯

@sachin-panayil sachin-panayil dismissed stale reviews from decause-gov and themself via df4aeb9 October 22, 2024 15:59
Copy link
Contributor

@sachin-panayil sachin-panayil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🚢 💯

@IsaacMilarky IsaacMilarky merged commit f842193 into main Oct 22, 2024
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Front-end: COCOMO, Pt. 2 Project Rotation: Repo Libyears
4 participants