mirror of
https://github.com/ChuckBuilds/LEDMatrix.git
synced 2026-04-12 05:42:59 +00:00
refactor: reduce debug logging output in calendar and display managers
This commit is contained in:
@@ -33,13 +33,9 @@ class CalendarManager:
|
|||||||
self.events = []
|
self.events = []
|
||||||
self.service = None
|
self.service = None
|
||||||
|
|
||||||
# Log font information during initialization
|
# Log font information during initialization only
|
||||||
logger.debug(f"Display Manager fonts:")
|
if self.enabled:
|
||||||
logger.debug(f" Small font: {self.display_manager.small_font}")
|
logger.info(f"Calendar configuration: enabled={self.enabled}, update_interval={self.update_interval}, max_events={self.max_events}, calendars={self.calendars}")
|
||||||
logger.debug(f" Calendar font: {self.display_manager.calendar_font}")
|
|
||||||
logger.debug(f" Font types - Small: {type(self.display_manager.small_font)}, Calendar: {type(self.display_manager.calendar_font)}")
|
|
||||||
|
|
||||||
logger.info(f"Calendar configuration: enabled={self.enabled}, update_interval={self.update_interval}, max_events={self.max_events}, calendars={self.calendars}")
|
|
||||||
|
|
||||||
# Get timezone from config
|
# Get timezone from config
|
||||||
self.config_manager = ConfigManager()
|
self.config_manager = ConfigManager()
|
||||||
@@ -122,7 +118,7 @@ class CalendarManager:
|
|||||||
try:
|
try:
|
||||||
# Get date, time, and summary
|
# Get date, time, and summary
|
||||||
date_text = self._format_event_date(event)
|
date_text = self._format_event_date(event)
|
||||||
time_text = self._format_event_time(event).replace(" ", "") # Remove space between time and AM/PM
|
time_text = self._format_event_time(event)
|
||||||
datetime_text = f"{date_text} {time_text}"
|
datetime_text = f"{date_text} {time_text}"
|
||||||
summary = event.get('summary', 'No Title')
|
summary = event.get('summary', 'No Title')
|
||||||
|
|
||||||
@@ -242,15 +238,12 @@ class CalendarManager:
|
|||||||
if 'T' in start:
|
if 'T' in start:
|
||||||
# The datetime string already includes timezone info (-05:00)
|
# The datetime string already includes timezone info (-05:00)
|
||||||
dt = datetime.fromisoformat(start)
|
dt = datetime.fromisoformat(start)
|
||||||
logger.debug(f"Parsed datetime from event: {dt}")
|
|
||||||
else:
|
else:
|
||||||
dt = datetime.strptime(start, '%Y-%m-%d')
|
dt = datetime.strptime(start, '%Y-%m-%d')
|
||||||
# Make date object timezone-aware (assume UTC if no tz info)
|
# Make date object timezone-aware (assume UTC if no tz info)
|
||||||
dt = pytz.utc.localize(dt)
|
dt = pytz.utc.localize(dt)
|
||||||
logger.debug(f"Parsed date from event: {dt}")
|
|
||||||
|
|
||||||
# No need to convert timezone since it's already in the correct one
|
# No need to convert timezone since it's already in the correct one
|
||||||
logger.debug(f"Using event timezone: {dt}")
|
|
||||||
return dt.strftime("%a %-m/%-d") # e.g., "Mon 4/21"
|
return dt.strftime("%a %-m/%-d") # e.g., "Mon 4/21"
|
||||||
except ValueError as e:
|
except ValueError as e:
|
||||||
logging.error(f"Could not parse date string: {start} - {e}")
|
logging.error(f"Could not parse date string: {start} - {e}")
|
||||||
@@ -265,10 +258,8 @@ class CalendarManager:
|
|||||||
try:
|
try:
|
||||||
# The datetime string already includes timezone info (-05:00)
|
# The datetime string already includes timezone info (-05:00)
|
||||||
dt = datetime.fromisoformat(start)
|
dt = datetime.fromisoformat(start)
|
||||||
logger.debug(f"Parsed time from event: {dt}")
|
|
||||||
# No need to convert timezone since it's already in the correct one
|
# No need to convert timezone since it's already in the correct one
|
||||||
logger.debug(f"Using event timezone: {dt}")
|
return dt.strftime("%I:%M%p")
|
||||||
return dt.strftime("%I:%M %p")
|
|
||||||
except ValueError as e:
|
except ValueError as e:
|
||||||
logging.error(f"Could not parse time string: {start} - {e}")
|
logging.error(f"Could not parse time string: {start} - {e}")
|
||||||
return "Invalid Time"
|
return "Invalid Time"
|
||||||
|
|||||||
@@ -243,14 +243,8 @@ class DisplayManager:
|
|||||||
# Select font based on parameters
|
# Select font based on parameters
|
||||||
if font:
|
if font:
|
||||||
current_font = font
|
current_font = font
|
||||||
if not hasattr(self, '_last_font') or self._last_font != font:
|
|
||||||
logger.debug(f"Using custom font: {font}")
|
|
||||||
self._last_font = font
|
|
||||||
else:
|
else:
|
||||||
current_font = self.small_font if small_font else self.regular_font
|
current_font = self.small_font if small_font else self.regular_font
|
||||||
if not hasattr(self, '_last_font') or self._last_font != current_font:
|
|
||||||
logger.debug(f"Using {'small' if small_font else 'regular'} font: {current_font}")
|
|
||||||
self._last_font = current_font
|
|
||||||
|
|
||||||
# Calculate x position if not provided (center text)
|
# Calculate x position if not provided (center text)
|
||||||
if x is None:
|
if x is None:
|
||||||
|
|||||||
Reference in New Issue
Block a user