Update app.R

app.R

@@ -3011,13 +3011,67 @@ create_location_by_side_plot <- function(data, player_name, batter_side, pitch_c
   create_location_by_result_plot(data, player_name, batter_side, pitch_colors)
 }
 
-create_advanced_pitcher_pdf <- function(game_df, pitcher_name, output_file) {
+get_team_logo_path <- function(team_name, logo_dir = "logos") {
+  if (is.null(team_name) || is.na(team_name) || !nzchar(team_name)) return(NULL)
+  
+  # Normalize team name for file matching
+  team_clean <- tolower(gsub("[^a-zA-Z0-9]", "_", team_name))
+  team_nospace <- tolower(gsub("[^a-zA-Z0-9]", "", team_name))
+  
+  # Check multiple possible paths
+  possible_paths <- c(
+    file.path(logo_dir, paste0(team_clean, ".png")),
+    file.path(logo_dir, paste0(team_nospace, ".png")),
+    file.path(logo_dir, paste0(team_name, ".png")),
+    file.path(logo_dir, paste0(tolower(team_name), ".png")),
+    # Common abbreviations
+    file.path(logo_dir, "coastal_carolina.png"),
+    file.path(logo_dir, "ccu.png")
+  )
+  
+  for (path in possible_paths) {
+    if (file.exists(path)) {
+      return(path)
+    }
+  }
+  
+  return(NULL)
+}
+
+# Function to add logo to the report
+add_team_logo <- function(logo_path, x, y, width, height) {
+  if (is.null(logo_path) || !file.exists(logo_path)) {
+    return(invisible(NULL))
+  }
+  
+  tryCatch({
+    # Read the PNG image
+    img <- png::readPNG(logo_path)
+    
+    # Create a raster grob and draw it
+    grid::grid.raster(
+      img,
+      x = x,
+      y = y,
+      width = width,
+      height = height,
+      just = c("center", "center")
+    )
+  }, error = function(e) {
+    message("Could not load logo: ", e$message)
+    invisible(NULL)
+  })
+}
+
+
+create_advanced_pitcher_pdf <- function(game_df, pitcher_name, output_file, logo_dir = "logos") {
   if (length(dev.list()) > 0) try(dev.off(), silent = TRUE)
   
   pitch_colors <- c(
-    "Fastball" = "#FA8072", "FourSeamFastBall" =  "#FA8072", "Four-Seam" = "#FA8072", "Sinker" = "#fdae61",
-    "Slider"   = "#A020F0", "Sweeper"  = "magenta",  "Curveball" = "#2c7bb6",
-    "ChangeUp" = "#90EE90", "Splitter" = "#90EE32",  "Cutter"    = "red"
+    "Fastball" = "#FA8072", "FourSeamFastBall" = "#FA8072", "Four-Seam" = "#FA8072", 
+    "Sinker" = "#fdae61", "Slider" = "#A020F0", "Sweeper" = "magenta",  
+    "Curveball" = "#2c7bb6", "ChangeUp" = "#90EE90", "Splitter" = "#90EE32", 
+    "Cutter" = "red"
   )
   
   .text_on_fill <- function(hex) {
@@ -3029,7 +3083,6 @@ create_advanced_pitcher_pdf <- function(game_df, pitcher_name, output_file) {
     }, error = function(e) "black")
   }
   
-  # Safe cell accessor - handles missing columns gracefully
   get_cell_value <- function(df, colname, row_idx) {
     if (is.null(df) || !is.data.frame(df)) return(NA)
     if (is.null(colname) || !nzchar(colname)) return(NA)
@@ -3048,9 +3101,24 @@ create_advanced_pitcher_pdf <- function(game_df, pitcher_name, output_file) {
     return(output_file)
   }
   
+  # Get pitcher's team for logo
+  pitcher_team <- NULL
+  if ("PitcherTeam" %in% names(pitcher_df)) {
+    pitcher_team <- pitcher_df$PitcherTeam[1]
+  } else if ("Team" %in% names(pitcher_df)) {
+    pitcher_team <- pitcher_df$Team[1]
+  } else if ("HomeTeam" %in% names(pitcher_df)) {
+    # Try to determine team from context
+    pitcher_team <- pitcher_df$HomeTeam[1]
+  }
+  
+  # Get logo path
+
+  logo_path <- get_team_logo_path(pitcher_team, logo_dir)
+  
   game_day <- tryCatch(parse_game_day(pitcher_df), error = function(e) Sys.Date())
   
-  # Get summary stats with error handling
+  # Get summary stats
   summary_result <- tryCatch(
     create_advanced_pitcher_summary(pitcher_df, pitcher_name),
     error = function(e) {
@@ -3062,7 +3130,7 @@ create_advanced_pitcher_pdf <- function(game_df, pitcher_name, output_file) {
   summary_stats  <- summary_result$stats
   summary_colors <- summary_result$colors
   
-  # Get pitch characteristics with error handling
+  # Get pitch characteristics
   pitch_result <- tryCatch(
     create_advanced_pitch_characteristics(pitcher_df, pitcher_name),
     error = function(e) {
@@ -3076,7 +3144,7 @@ create_advanced_pitcher_pdf <- function(game_df, pitcher_name, output_file) {
   pitch_char <- pitch_result$stats
   pitch_colors_matrix <- pitch_result$colors
   
-  # Handle empty or NULL pitch_char
+  # Handle empty pitch_char
   if (is.null(pitch_char) || nrow(pitch_char) == 0) {
     pitch_char <- data.frame(
       Pitch = "-", Count = 0, `Usage%` = NA_real_,
@@ -3093,17 +3161,16 @@ create_advanced_pitcher_pdf <- function(game_df, pitcher_name, output_file) {
     pitch_colors_matrix <- matrix("#FFFFFF", nrow = 1, ncol = ncol(pitch_char))
   }
   
-  # Limit rows to max 9
+  # Limit rows
   max_rows_to_show <- min(nrow(pitch_char), 9)
   if (nrow(pitch_char) > max_rows_to_show) {
     pitch_char <- pitch_char[1:max_rows_to_show, , drop = FALSE]
   }
   
-  # Get final dimensions
   num_rows <- nrow(pitch_char)
   num_cols <- ncol(pitch_char)
   
-  # CRITICAL: Rebuild color matrix to EXACTLY match pitch_char dimensions
+  # Rebuild color matrix
   new_color_matrix <- matrix("#FFFFFF", nrow = num_rows, ncol = num_cols)
   if (!is.null(pitch_colors_matrix) && is.matrix(pitch_colors_matrix)) {
     rows_to_copy <- min(nrow(pitch_colors_matrix), num_rows)
@@ -3115,7 +3182,7 @@ create_advanced_pitcher_pdf <- function(game_df, pitcher_name, output_file) {
   }
   pitch_colors_matrix <- new_color_matrix
   
-  # Create plots with error handling
+  # Create plots
   movement_plot <- tryCatch(
     create_movement_plot(pitcher_df, pitcher_name, pitch_colors),
     error = function(e) ggplot2::ggplot() + ggplot2::theme_void() + ggplot2::ggtitle("Movement Plot Error")
@@ -3136,11 +3203,8 @@ create_advanced_pitcher_pdf <- function(game_df, pitcher_name, output_file) {
     create_count_usage_plot(pitcher_df, pitcher_name, pitch_colors),
     error = function(e) ggplot2::ggplot() + ggplot2::theme_void() + ggplot2::ggtitle("Count Usage Error")
   )
-   extension_height_plot <- tryCatch(
-    create_extension_height_plot(pitcher_df, pitcher_name, pitch_colors),
-    error = function(e) ggplot2::ggplot() + ggplot2::theme_void() + ggplot2::ggtitle("Extension/Height Error")
-  )
   
+  # ENLARGED release side plot
   relside_height_plot <- tryCatch(
     create_relside_height_plot(pitcher_df, pitcher_name, pitch_colors),
     error = function(e) ggplot2::ggplot() + ggplot2::theme_void() + ggplot2::ggtitle("RelSide/Height Error")
@@ -3175,14 +3239,28 @@ create_advanced_pitcher_pdf <- function(game_df, pitcher_name, output_file) {
   available_for_table <- y_top_char - (base_loc_top + table_margin)
   row_h_char <- min(max_row_h, max(min_row_h, available_for_table / max(1, rows_including_header)))
   
-  # Header
-  grid::pushViewport(grid::viewport(x = 0.5, y = header_y_top, width = 1, height = 0.04, just = c("center","top")))
+  # ===== HEADER WITH LOGO =====
+  grid::pushViewport(grid::viewport(x = 0.5, y = header_y_top, width = 1, height = 0.06, just = c("center","top")))
+  
+  # Add team logo on the left if available
+  if (!is.null(logo_path) && file.exists(logo_path)) {
+    add_team_logo(logo_path, x = 0.08, y = 0.5, width = 0.06, height = grid::unit(0.8, "npc"))
+  }
+  
+  # Title in center
   grid::grid.text(paste(pitcher_name, "- Advanced Pitcher Report"),
+                  x = 0.5, y = 0.5,
                   gp = grid::gpar(fontface = "bold", cex = 1.8, col = "#006F71"))
+  
+  # Add team logo on the right if available (mirror)
+  if (!is.null(logo_path) && file.exists(logo_path)) {
+    add_team_logo(logo_path, x = 0.92, y = 0.5, width = 0.06, height = grid::unit(0.8, "npc"))
+  }
+  
   grid::popViewport()
   
   # Summary section
-  grid::grid.text("Summary", x = 0.5, y = 0.94,
+  grid::grid.text("Summary", x = 0.5, y = 0.92,
                   gp = grid::gpar(fontface = "bold", cex = 1.1, col = "#006F71"))
   
   summary_headers <- names(summary_stats)
@@ -3191,7 +3269,7 @@ create_advanced_pitcher_pdf <- function(game_df, pitcher_name, output_file) {
   
   x_start <- 0.5 - sum(summary_widths)/2
   x_pos   <- c(x_start, x_start + cumsum(summary_widths[-length(summary_widths)]))
-  y_top   <- 0.925
+  y_top   <- 0.905
   row_h   <- 0.020
   
   for (i in seq_along(summary_headers)) {
@@ -3209,6 +3287,7 @@ create_advanced_pitcher_pdf <- function(game_df, pitcher_name, output_file) {
                     gp = grid::gpar(cex = 0.62))
   }
   
+  # Row 1: Movement plot (left) | Velocity distribution (right)
   grid::pushViewport(grid::viewport(x = 0.25, y = charts_y_top, width = 0.45, height = charts_height, just = c("center","top")))
   tryCatch(print(movement_plot, newpage = FALSE), error = function(e) NULL)
   grid::popViewport()
@@ -3217,16 +3296,16 @@ create_advanced_pitcher_pdf <- function(game_df, pitcher_name, output_file) {
   tryCatch(print(velo_plot, newpage = FALSE), error = function(e) NULL)
   grid::popViewport()
   
-  # Row 2: Count plot (left) | Release charts stacked (right)
-  # Count plot - moved to left side, narrower
-  grid::pushViewport(grid::viewport(x = 0.27, y = count_y_top, width = 0.50, height = 18, just = c("center","top")))
+  # Row 2: Count plot (left) | ENLARGED Release side plot (right)
+  # Count plot on left
+  grid::pushViewport(grid::viewport(x = 0.27, y = count_y_top, width = 0.50, height = count_height, just = c("center","top")))
   tryCatch(print(count_plot, newpage = FALSE), error = function(e) NULL)
   grid::popViewport()
   
-grid::pushViewport(grid::viewport(x = 0.77, y = count_y_top - count_height * 0.55, width = 0.42, height = count_height * 0.55, just = c("center","top")))
+  # ENLARGED Release side plot on right - increased width and height
+  grid::pushViewport(grid::viewport(x = 0.77, y = count_y_top, width = 0.44, height = count_height, just = c("center","top")))
   tryCatch(print(relside_height_plot, newpage = FALSE), error = function(e) NULL)
   grid::popViewport()
-
   
   # Pitch Characteristics table
   grid::grid.text("Pitch Characteristics", x = 0.5, y = y_top_char + 0.015,
@@ -3235,7 +3314,6 @@ grid::pushViewport(grid::viewport(x = 0.77, y = count_y_top - count_height * 0.5
   char_headers  <- names(pitch_char)
   num_char_cols <- length(char_headers)
   
-  # Calculate column widths safely
   if (num_char_cols > 1) {
     char_widths <- c(0.10, rep((1 - 0.10 - 0.06) / (num_char_cols - 1), num_char_cols - 1))
   } else {
@@ -3258,7 +3336,6 @@ grid::pushViewport(grid::viewport(x = 0.77, y = count_y_top - count_height * 0.5
                     gp = grid::gpar(col = "white", cex = 0.50, fontface = "bold"))
   }
   
-  # Find Pitch column index
   i_col_pitch  <- match("Pitch", char_headers)
   has_pitchcol <- !is.na(i_col_pitch) && i_col_pitch >= 1
   
@@ -3273,14 +3350,12 @@ grid::pushViewport(grid::viewport(x = 0.77, y = count_y_top - count_height * 0.5
       
       colname <- char_headers[i]
       
-      # Get background color safely - check bounds explicitly
       bg <- "#FFFFFF"
       if (r >= 1 && r <= nrow(pitch_colors_matrix) && i >= 1 && i <= ncol(pitch_colors_matrix)) {
         bg <- pitch_colors_matrix[r, i]
         if (is.na(bg) || !nzchar(bg)) bg <- "#FFFFFF"
       }
       
-      # Override for Pitch column with pitch type color
       if (has_pitchcol && identical(colname, "Pitch") && !is.na(pitch_name) && pitch_name %in% names(pitch_colors)) {
         bg <- pitch_colors[[pitch_name]]
       }
@@ -3288,7 +3363,6 @@ grid::pushViewport(grid::viewport(x = 0.77, y = count_y_top - count_height * 0.5
       grid::grid.rect(x = x_pos_char[i], y = y_row, width = char_widths[i]*0.985, height = row_h_char,
                       just = c("left","top"), gp = grid::gpar(fill = bg, col = "grey80", lwd = 0.3))
       
-      # Get value safely
       val <- get_cell_value(pitch_char, colname, r)
       
       display_val <- if (is.numeric(val)) {