Update app.R

app.R

@@ -1031,11 +1031,12 @@ create_bp_contact_map <- function(batter_name, bp_data) {
 }
 
 create_bp_contact_side <- function(batter_name, bp_data) {
+  # TrackMan contact coords: X = depth (toward pitcher), Y = height, Z = side
   contact_data <- bp_data %>%
     filter(Batter == batter_name, BIPind == 1, !is.na(ExitSpeed),
-           !is.na(ContactPositionY), !is.na(ContactPositionZ))
+           !is.na(ContactPositionX), !is.na(ContactPositionY))
   
-  # Home plate side illustration (tip toward catcher at Y = 0, front edge at 1.417 ft)
+  # Home plate side illustration (tip toward catcher at depth 0, front edge at 1.417 ft)
   plate_df <- data.frame(
     x = c(0, 0.708, 1.417, 1.417, 0.708),
     y = c(0.25, 0.35, 0.35, 0.15, 0.15)
@@ -1061,14 +1062,14 @@ create_bp_contact_side <- function(batter_name, bp_data) {
       ggplot() +
         geom_polygon(data = plate_df, aes(x, y), fill = NA, color = "gray70", linewidth = 0.5) +
         coord_fixed(xlim = c(-1, 4), ylim = c(0, 5), expand = FALSE) +
-        labs(x = "Contact Pos Y (in feet)", y = "Contact Pos Z (in feet)") +
+        labs(x = "Contact Depth (ft in front of plate)", y = "Contact Height (ft)") +
         ggtitle(paste("Contact Depth vs Height:", batter_name)) +
         annotate("text", x = 1.5, y = 2.5, label = "No contact data available", size = 5, color = "gray50") +
         base_theme
     )
   }
   
-  ggplot(contact_data, aes(x = ContactPositionY, y = ContactPositionZ)) +
+  ggplot(contact_data, aes(x = ContactPositionX, y = ContactPositionY)) +
     geom_polygon(data = plate_df, aes(x, y), fill = "white", color = "black",
                  linewidth = 0.6, inherit.aes = FALSE) +
     geom_point(aes(fill = ExitSpeed), size = 3, shape = 21, color = "black", stroke = 0.4, alpha = 0.85) +
@@ -1076,7 +1077,7 @@ create_bp_contact_side <- function(batter_name, bp_data) {
                          midpoint = mean(contact_data$ExitSpeed, na.rm = TRUE),
                          name = "Exit Velo") +
     coord_fixed(xlim = c(-1, 4), ylim = c(0, 5), expand = FALSE) +
-    labs(x = "Contact Pos Y (in feet)", y = "Contact Pos Z (in feet)") +
+    labs(x = "Contact Depth (ft in front of plate)", y = "Contact Height (ft)") +
     ggtitle(paste("Contact Depth vs Height:", batter_name)) +
     base_theme
 }
@@ -1117,6 +1118,76 @@ bp_leaderboard_colors <- function(x, target = NULL) {
   cols
 }
 
+create_bp_leaderboard_pdf <- function(lb, date_label, output_file) {
+  if (length(dev.list()) > 0) try(dev.off(), silent = TRUE)
+  
+  stat_cols <- c("BBE", "Avg EV", "Avg LA", "Max EV", "SC%", "10-30%", "HH%", "Barrel%")
+  color_cols <- setdiff(stat_cols, "BBE")  # BBE = volume, not quality
+  
+  cell_fill <- matrix("white", nrow = nrow(lb), ncol = length(stat_cols),
+                      dimnames = list(NULL, stat_cols))
+  for (cn in color_cols) {
+    cell_fill[, cn] <- if (cn == "Avg LA") bp_leaderboard_colors(lb[[cn]], target = 20)
+    else bp_leaderboard_colors(lb[[cn]])
+  }
+  
+  pdf(output_file, width = 11, height = 8.5)
+  on.exit(try(dev.off(), silent = TRUE), add = TRUE)
+  
+  name_w <- 0.20
+  col_w <- 0.082
+  row_h <- 0.028
+  x0 <- 0.5 - (name_w + length(stat_cols) * col_w) / 2
+  rows_per_page <- 25
+  n_pages <- max(1, ceiling(nrow(lb) / rows_per_page))
+  
+  draw_cell <- function(x, y, w, fill, label, bold = FALSE, txt_col = "black") {
+    grid::grid.rect(x = x, y = y, width = w * 0.99, height = row_h,
+                    just = c("left", "top"),
+                    gp = grid::gpar(fill = fill, col = "black", lwd = 0.4))
+    grid::grid.text(label, x = x + w * 0.5, y = y - row_h / 2,
+                    gp = grid::gpar(cex = 0.85, col = txt_col,
+                                    fontface = if (bold) "bold" else "plain"))
+  }
+  
+  for (pg in seq_len(n_pages)) {
+    grid::grid.newpage()
+    grid::grid.text("Daily BP Leaderboard", y = 0.955,
+                    gp = grid::gpar(fontface = "bold", cex = 1.5, col = "#006F71"))
+    if (!is.null(date_label)) {
+      grid::grid.text(date_label, y = 0.915, gp = grid::gpar(cex = 1.1, col = "grey30"))
+    }
+    if (n_pages > 1) {
+      grid::grid.text(paste0("Page ", pg, " of ", n_pages), x = 0.95, y = 0.03,
+                      gp = grid::gpar(cex = 0.7, col = "grey50"))
+    }
+    
+    y_top <- 0.86
+    
+    # Header row
+    draw_cell(x0, y_top, name_w, "#006F71", "Player", bold = TRUE, txt_col = "white")
+    for (j in seq_along(stat_cols)) {
+      draw_cell(x0 + name_w + (j - 1) * col_w, y_top, col_w, "#006F71",
+                stat_cols[j], bold = TRUE, txt_col = "white")
+    }
+    
+    idx <- (((pg - 1) * rows_per_page) + 1):min(pg * rows_per_page, nrow(lb))
+    for (r in seq_along(idx)) {
+      i <- idx[r]
+      y_r <- y_top - r * row_h
+      draw_cell(x0, y_r, name_w, "white", lb$Batter[i], bold = TRUE)
+      for (j in seq_along(stat_cols)) {
+        val <- lb[[stat_cols[j]]][i]
+        draw_cell(x0 + name_w + (j - 1) * col_w, y_r, col_w,
+                  cell_fill[i, stat_cols[j]],
+                  ifelse(is.finite(val), as.character(val), "-"))
+      }
+    }
+  }
+  
+  invisible(output_file)
+}
+
 create_bp_pdf <- function(bp_data, batter_name, output_file) {
   if (length(dev.list()) > 0) try(dev.off(), silent = TRUE)
   
@@ -7087,6 +7158,36 @@ server <- function(input, output, session) {
     contentType = "application/zip"
   )
   
+  output$download_bp_leaderboard <- downloadHandler(
+    filename = function() {
+      sel <- input$bp_leaderboard_date
+      date_str <- if (!is.null(sel) && sel != "All Dates") {
+        gsub("-", "", sel)
+      } else if (is.null(sel)) {
+        df <- data_bp(); req(df)
+        tryCatch(format(parse_game_day(df), "%Y%m%d"),
+                 error = function(e) format(Sys.Date(), "%Y%m%d"))
+      } else "All_Dates"
+      paste0("BP_Leaderboard_", date_str, ".pdf")
+    },
+    content = function(file) {
+      lb <- bp_leaderboard_data()
+      req(nrow(lb) > 0)
+      sel <- input$bp_leaderboard_date
+      date_label <- if (!is.null(sel) && sel != "All Dates") {
+        format(as.Date(sel), "%B %e, %Y")
+      } else if (is.null(sel)) {
+        df <- data_bp()
+        tryCatch(format(parse_game_day(df), "%B %e, %Y"), error = function(e) NULL)
+      } else "All Dates"
+      withProgress(message='Generating Leaderboard PDF', value=0.5, {
+        create_bp_leaderboard_pdf(lb, date_label, file)
+      })
+      showNotification("Leaderboard PDF ready!", type="message", duration=3)
+    },
+    contentType = "application/pdf"
+  )
+  
   output$download_all_coastal_pitchers <- downloadHandler(
     filename = function() {
       df <- data_pitcher(); req(df)
@@ -7387,6 +7488,8 @@ server <- function(input, output, session) {
                       choices = c(date_choices, "All Dates"),
                       selected = date_choices[1], width = "220px"),
         DT::dataTableOutput("bp_leaderboard"),
+        div(style = "margin-top:8px;",
+            downloadButton("download_bp_leaderboard", "Download Leaderboard PDF", class = "btn-secondary")),
         br(),
         h4("BP Spray Chart", style = "color: #006F71;"),
         plotOutput("preview_bp_spray", height = "400px"),
@@ -7441,7 +7544,8 @@ server <- function(input, output, session) {
   
   output$bp_leaderboard <- DT::renderDataTable({
     lb <- bp_leaderboard_data()
-    validate(need(nrow(lb) > 0, "No BP data for this date"))
+    # jsonlite (loaded after shiny) masks shiny::validate, so namespace explicitly
+    shiny::validate(shiny::need(nrow(lb) > 0, "No BP data for this date"))
     
     # Red = bad, white = group average, green = good (Avg LA scored by closeness to 20 deg)
     # BBE stays uncolored - volume, not quality