From 4c68085fb4c31b9595ea657d1fdd92acc02c1127 Mon Sep 17 00:00:00 2001 From: "Dawn M. Foster" Date: Wed, 13 Mar 2024 12:15:23 +0000 Subject: [PATCH] fix leap year issue where I was assuming a year was 365 days despite knowing better :) --- health_by_repo.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/health_by_repo.py b/health_by_repo.py index 49ac67b..2bd8ec0 100644 --- a/health_by_repo.py +++ b/health_by_repo.py @@ -64,6 +64,8 @@ import argparse import sys import pandas as pd +import datetime +import calendar from utils.augur_connect import augur_db_connect from utils.date_calcs import get_dates from utils.repo_info import get_repo_info, fork_archive, get_org_repos @@ -93,7 +95,7 @@ print('Parameters: Years =', years, 'Business Days', bus_days) # Get the dates for the analysis using the years argument if provided -days = 365 * years +days = 365 + calendar.isleap(year=datetime.datetime.now().year) start_date, end_date = get_dates(days) # Create the connection to the Augur database