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

Bug commit analysis #8

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 3 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
36 changes: 14 additions & 22 deletions src/count.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,9 @@
import requests
import time
import os
import os
import sys
from typing import Optional
import fire

# Constants

Expand All @@ -22,22 +24,22 @@ def safe_request(url, headers, max_retries=MAX_RETRIES, timeout=10):
sleep_time = max(reset_time - int(time.time()), 1)
print(f"Rate limit exceeded. Waiting for {sleep_time} seconds.")
time.sleep(sleep_time)
# Failed to fetch data after retries
else:
print(f"Request failed with status code {response.status_code}. Attempt {attempt + 1} of {max_retries}.")
time.sleep(2 ** attempt)
except requests.exceptions.RequestException as e:
print(f"Request exception: {e}. Attempt {attempt + 1} of {max_retries}.")
# Exponential backoff
time.sleep(2 ** attempt)
# Failed to fetch data after retries
return None

def read_repos(file_path):
with open(file_path, 'r') as file:
repos = [line.strip() for line in file.readlines()]
return repos

def get_commits_count(repo):
def get_commits_count(repo) -> Optional[int]:
start_date = "2023-01-01T00:00:00Z"
end_date = "2024-12-31T23:59:59Z"
commits_url = f"{GITHUB_API}/repos/{repo}/commits?since={start_date}&until={end_date}&per_page=1"
Expand All @@ -47,15 +49,14 @@ def get_commits_count(repo):
last_page_url = response.links['last']['url']
last_page_number = int(last_page_url.split('=')[-1])
return last_page_number
elif response.json():
elif response.json():
# Check if there's at least one commit in the specified range
return 1
# No commits in the specified date range
return 0
return "Error"
return None


def get_bugs_count(repo):
def get_bugs_count(repo) -> Optional[int]:
bug_count = 0
page = 1
while True:
Expand All @@ -66,30 +67,21 @@ def get_bugs_count(repo):
if not issues:
break
for issue in issues:
# Get the date part of the timestamp
created_at = issue['created_at'][:10]
if "2023-01-01" <= created_at <= "2024-12-31":
bug_count += 1
page += 1
else:
return "Error"
return None
return bug_count



def write_output(repos_info, output_file):
with open(output_file, 'w') as file:
for repo, commits_count, bugs_count in repos_info:
file.write(f"{repo.replace('/', ',')} , {commits_count} , {bugs_count}\n")

def main():
if len(sys.argv) < 2:
print("Usage: python script.py input_file_path")
sys.exit(1)

file_path = sys.argv[1]
output_file = 'output4.csv'
repos = read_repos(file_path)
def main(input_file_path, output_file_path):
repos = read_repos(input_file_path)
repos_info = []

for repo in repos:
Expand All @@ -98,8 +90,8 @@ def main():
bugs_count = get_bugs_count(repo)
repos_info.append((repo, commits_count, bugs_count))

write_output(repos_info, output_file)
print("Finished. Output written to", output_file)
write_output(repos_info, output_file_path)
print("Finished. Output written to", output_file_path)

if __name__ == "__main__":
main()
fire.Fire(main)
17 changes: 0 additions & 17 deletions src/format.py

This file was deleted.

28 changes: 28 additions & 0 deletions src/plot.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@
import pandas as pd
import matplotlib.pyplot as plt
import seaborn as sns
import fire

def generate_charts(df_path, bug_chart_path, commit_chart_path):
df = pd.read_csv(df_path)
top_20_by_bug_count = df.nlargest(20, 'BugCount')
Copy link
Owner

Choose a reason for hiding this comment

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

could you make histograms of all instead of only top 20?

top_20_by_commit_count = df.nlargest(20, 'CommitCount')

plt.figure(figsize=(10, 8))
sns.barplot(x='BugCount', y='RepoName', data=top_20_by_bug_count, palette='viridis')
plt.title('Top 20 Repositories by Bug Count')
plt.xlabel('Bug Count')
plt.ylabel('Repository')
plt.tight_layout()
plt.savefig(bug_chart_path)

plt.figure(figsize=(10, 8))
sns.barplot(x='CommitCount', y='RepoName', data=top_20_by_commit_count, palette='viridis')
plt.title('Top 20 Repositories by Commit Count')
Copy link
Owner

Choose a reason for hiding this comment

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

don't put titles on plots, it is the job of the caption

plt.xlabel('Commit Count')
plt.ylabel('Repository')
plt.tight_layout()
plt.savefig(commit_chart_path)
Copy link
Owner

Choose a reason for hiding this comment

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

please save to PDF format as they are vector graphic files


if __name__ == '__main__':
fire.Fire(generate_charts)