Update app.R

app.R

@@ -498,46 +498,47 @@ server <- function(input, output, session) {
   })
   
   # Processing summary
-output$process_summary <- renderText({
-  if (is.null(input$file)) {
-    return("No file uploaded yet.")
-  }
-  
-  if (is.null(processed_data())) {
-    return("Processing...")
-  }
-  
-  df <- processed_data()
-  original_df <- read.csv(input$file$datapath, nrows = 1)
-  selected_cols_to_remove <- input$columns_to_remove %||% character(0)
-  removed_cols <- intersect(selected_cols_to_remove, names(original_df))
-  
-  # Build the removed columns text
-  removed_cols_text <- if (length(removed_cols) > 0) {
-    cols_display <- if (length(removed_cols) > 5) {
-      paste(paste(head(removed_cols, 5), collapse = ", "), "...")
+  output$process_summary <- renderText({
+    if (is.null(input$file)) {
+      return("No file uploaded yet.")
+    }
+    
+    if (is.null(processed_data())) {
+      return("Processing...")
+    }
+    
+    df <- processed_data()
+    original_df <- read.csv(input$file$datapath, nrows = 1)
+    selected_cols_to_remove <- input$columns_to_remove %||% character(0)
+    removed_cols <- intersect(selected_cols_to_remove, names(original_df))
+    
+    # Build the removed columns text
+    removed_cols_text <- if (length(removed_cols) > 0) {
+      cols_display <- if (length(removed_cols) > 5) {
+        paste(paste(head(removed_cols, 5), collapse = ", "), "...")
+      } else {
+        paste(removed_cols, collapse = ", ")
+      }
+      paste("✓ Removed columns:", length(removed_cols), "\n  -", cols_display)
     } else {
-      paste(removed_cols, collapse = ", ")
+      "✓ Removed columns: 0"
     }
-    paste("✓ Removed columns:", length(removed_cols), "\n  -", cols_display)
-  } else {
-    "✓ Removed columns: 0"
-  }
-  
-  summary_text <- paste(
-    "✓ File processed successfully!",
-    paste("✓ Original columns:", ncol(original_df)),
-    paste("✓ Final columns:", ncol(df)),
-    paste("✓ Target columns: 167"),
-    paste("✓ Rows processed:", nrow(df)),
-    removed_cols_text,
-    "✓ Duplicates removed",
-    paste("✓ Ready for further processing"),
-    sep = "\n"
-  )
+    
+    summary_text <- paste(
+      "✓ File processed successfully!",
+      paste("✓ Original columns:", ncol(original_df)),
+      paste("✓ Final columns:", ncol(df)),
+      paste("✓ Target columns: 167"),
+      paste("✓ Rows processed:", nrow(df)),
+      removed_cols_text,
+      "✓ Duplicates removed",
+      paste("✓ Ready for further processing"),
+      sep = "\n"
+    )
+    
+    return(summary_text)
+  })
   
-  return(summary_text)
-})
   # Preview table
   output$preview <- DT::renderDataTable({
     req(processed_data())
@@ -635,6 +636,10 @@ output$process_summary <- renderText({
     if (min(distances) <= 2) {
       clicked_pitch <- pitcher_data[closest_idx, ]
       
+      # DEBUG: Print to console
+      print("Pitch selected!")
+      print(paste("Type:", clicked_pitch$TaggedPitchType))
+      
       # Store the original row index in the full dataset
       full_data <- plot_data() %>% filter(Pitcher == input$pitcher_select)
       original_row <- which(full_data$HorzBreak == clicked_pitch$HorzBreak & 
@@ -654,15 +659,6 @@ output$process_summary <- renderText({
       
       # Show modal
       toggleModal(session, "pitchEditModal", toggle = "open")
-
-       
-  if (min(distances) <= 2) {
-    clicked_pitch <- pitcher_data[closest_idx, ]
-    
-    # DEBUG: Print to console
-    print("Pitch selected!")
-    print(clicked_pitch$TaggedPitchType)
-    
     }
   })
   
@@ -781,97 +777,99 @@ output$process_summary <- renderText({
     }
   })
   
-output$movement_stats <- DT::renderDataTable({
-  req(plot_data(), input$pitcher_select)
-  
-  data <- plot_data()
-  
-  movement_stats <- data %>%
-    filter(Pitcher == input$pitcher_select) %>%
-    filter(!is.na(HorzBreak), !is.na(InducedVertBreak), !is.na(TaggedPitchType)) %>%
-    mutate(
-      pitch_group = case_when(
-        TaggedPitchType %in% c("Fastball", "FourSeamFastBall", "FourSeamFastB", "Four-Seam", "4-Seam") ~ "Fastball",
-        TaggedPitchType %in% c("OneSeamFastBall", "TwoSeamFastBall", "Sinker", "Two-Seam", "One-Seam") ~ "Sinker", 
-        TaggedPitchType %in% c("ChangeUp", "Changeup") ~ "Changeup",
-        TRUE ~ TaggedPitchType
-      ),
-      # Create necessary indicator variables if they don't exist
-      in_zone = ifelse("StrikeZoneIndicator" %in% names(.), StrikeZoneIndicator, 
-                      ifelse(!is.na(PlateLocSide) & !is.na(PlateLocHeight) & 
-                               PlateLocSide >= -0.95 & PlateLocSide <= 0.95 &
-                               PlateLocHeight >= 1.6 & PlateLocHeight <= 3.5, 1, 0)),
-      is_whiff = ifelse("WhiffIndicator" %in% names(.), WhiffIndicator,
-                       ifelse(!is.na(PitchCall) & PitchCall == "StrikeSwinging", 1, 0)),
-      chase = ifelse("Chaseindicator" %in% names(.), Chaseindicator,
-                    ifelse(!is.na(PitchCall) & !is.na(PlateLocSide) & !is.na(PlateLocHeight) &
-                             PitchCall %in% c("StrikeSwinging", "FoulBallNotFieldable", "FoulBall", "InPlay") &
-                             (PlateLocSide < -0.95 | PlateLocSide > 0.95 | PlateLocHeight < 1.6 | PlateLocHeight > 3.5), 1, 0))
-    )
-  
-  # Calculate total pitches for usage percentage
-  total_pitches <- nrow(movement_stats)
-  
-  summary_stats <- movement_stats %>%
-    group_by(`Pitch Type` = pitch_group) %>%
-    summarise(
-      Count = n(),
-      `Usage%` = sprintf("%.1f%%", (n() / total_pitches) * 100),
-      `Ext.` = ifelse("Extension" %in% names(movement_stats), 
-                      sprintf("%.1f", mean(Extension, na.rm = TRUE)), 
-                      "—"),
-      `Avg Velo` = sprintf("%.1f mph", mean(RelSpeed, na.rm = TRUE)),
-      `90th Velo` = sprintf("%.1f mph", quantile(RelSpeed, 0.9, na.rm = TRUE)),
-      `Max Velo` = sprintf("%.1f mph", max(RelSpeed, na.rm = TRUE)),
-      `Avg IVB` = sprintf("%.1f in", mean(InducedVertBreak, na.rm = TRUE)),
-      `Avg HB` = sprintf("%.1f in", mean(HorzBreak, na.rm = TRUE)),
-      `Avg Spin` = ifelse("SpinRate" %in% names(movement_stats),
-                         sprintf("%.0f rpm", mean(SpinRate, na.rm = TRUE)),
-                         "—"),
-      `Rel Height` = ifelse("RelHeight" %in% names(movement_stats),
-                           sprintf("%.1f", mean(RelHeight, na.rm = TRUE)),
-                           "—"),
-      `Zone%` = sprintf("%.1f%%", round(mean(in_zone, na.rm = TRUE) * 100, 1)),
-      `Whiff%` = sprintf("%.1f%%", round(mean(is_whiff, na.rm = TRUE) * 100, 1)),
-      `Chase%` = sprintf("%.1f%%", round(mean(chase, na.rm = TRUE) * 100, 1)),
-      .groups = "drop"
-    ) %>%
-    arrange(desc(Count))
-  
-  DT::datatable(summary_stats, 
-                options = list(pageLength = 15, dom = 't', scrollX = TRUE),
-                rownames = FALSE) %>%
-    DT::formatStyle(columns = names(summary_stats), fontSize = '12px')
-})
-  
-output$selected_pitch_info <- renderText({
-  pitch_info <- selected_pitch()
-  if (!is.null(pitch_info)) {
-    pitch_data <- pitch_info$data
-    
-    # Build info text with only available fields
-    info_lines <- c(
-      paste("Pitcher:", pitch_info$pitcher),
-      paste("Current Type:", pitch_data$TaggedPitchType),
-      paste("Velocity:", round(pitch_data$RelSpeed, 1), "mph"),
-      paste("Horizontal Break:", round(pitch_data$HorzBreak, 1), "inches"),
-      paste("Induced Vertical Break:", round(pitch_data$InducedVertBreak, 1), "inches")
-    )
+  # Movement stats table
+  output$movement_stats <- DT::renderDataTable({
+    req(plot_data(), input$pitcher_select)
     
-    # Add optional fields only if they exist and have values
-    if ("SpinRate" %in% names(pitch_data) && !is.na(pitch_data$SpinRate)) {
-      info_lines <- c(info_lines, paste("Spin Rate:", round(pitch_data$SpinRate, 0), "rpm"))
-    }
+    data <- plot_data()
     
-    if ("Date" %in% names(pitch_data) && !is.na(pitch_data$Date)) {
-      info_lines <- c(info_lines, paste("Date:", pitch_data$Date))
-    }
+    movement_stats <- data %>%
+      filter(Pitcher == input$pitcher_select) %>%
+      filter(!is.na(HorzBreak), !is.na(InducedVertBreak), !is.na(TaggedPitchType)) %>%
+      mutate(
+        pitch_group = case_when(
+          TaggedPitchType %in% c("Fastball", "FourSeamFastBall", "FourSeamFastB", "Four-Seam", "4-Seam") ~ "Fastball",
+          TaggedPitchType %in% c("OneSeamFastBall", "TwoSeamFastBall", "Sinker", "Two-Seam", "One-Seam") ~ "Sinker", 
+          TaggedPitchType %in% c("ChangeUp", "Changeup") ~ "Changeup",
+          TRUE ~ TaggedPitchType
+        ),
+        # Create necessary indicator variables if they don't exist
+        in_zone = ifelse("StrikeZoneIndicator" %in% names(.), StrikeZoneIndicator, 
+                        ifelse(!is.na(PlateLocSide) & !is.na(PlateLocHeight) & 
+                                 PlateLocSide >= -0.95 & PlateLocSide <= 0.95 &
+                                 PlateLocHeight >= 1.6 & PlateLocHeight <= 3.5, 1, 0)),
+        is_whiff = ifelse("WhiffIndicator" %in% names(.), WhiffIndicator,
+                         ifelse(!is.na(PitchCall) & PitchCall == "StrikeSwinging", 1, 0)),
+        chase = ifelse("Chaseindicator" %in% names(.), Chaseindicator,
+                      ifelse(!is.na(PitchCall) & !is.na(PlateLocSide) & !is.na(PlateLocHeight) &
+                               PitchCall %in% c("StrikeSwinging", "FoulBallNotFieldable", "FoulBall", "InPlay") &
+                               (PlateLocSide < -0.95 | PlateLocSide > 0.95 | PlateLocHeight < 1.6 | PlateLocHeight > 3.5), 1, 0))
+      )
     
-    return(paste(info_lines, collapse = "\n"))
-  } else {
-    return("No pitch selected")
-  }
-})
+    # Calculate total pitches for usage percentage
+    total_pitches <- nrow(movement_stats)
+    
+    summary_stats <- movement_stats %>%
+      group_by(`Pitch Type` = pitch_group) %>%
+      summarise(
+        Count = n(),
+        `Usage%` = sprintf("%.1f%%", (n() / total_pitches) * 100),
+        `Ext.` = ifelse("Extension" %in% names(movement_stats), 
+                        sprintf("%.1f", mean(Extension, na.rm = TRUE)), 
+                        "—"),
+        `Avg Velo` = sprintf("%.1f mph", mean(RelSpeed, na.rm = TRUE)),
+        `90th Velo` = sprintf("%.1f mph", quantile(RelSpeed, 0.9, na.rm = TRUE)),
+        `Max Velo` = sprintf("%.1f mph", max(RelSpeed, na.rm = TRUE)),
+        `Avg IVB` = sprintf("%.1f in", mean(InducedVertBreak, na.rm = TRUE)),
+        `Avg HB` = sprintf("%.1f in", mean(HorzBreak, na.rm = TRUE)),
+        `Avg Spin` = ifelse("SpinRate" %in% names(movement_stats),
+                           sprintf("%.0f rpm", mean(SpinRate, na.rm = TRUE)),
+                           "—"),
+        `Rel Height` = ifelse("RelHeight" %in% names(movement_stats),
+                             sprintf("%.1f", mean(RelHeight, na.rm = TRUE)),
+                             "—"),
+        `Zone%` = sprintf("%.1f%%", round(mean(in_zone, na.rm = TRUE) * 100, 1)),
+        `Whiff%` = sprintf("%.1f%%", round(mean(is_whiff, na.rm = TRUE) * 100, 1)),
+        `Chase%` = sprintf("%.1f%%", round(mean(chase, na.rm = TRUE) * 100, 1)),
+        .groups = "drop"
+      ) %>%
+      arrange(desc(Count))
+    
+    DT::datatable(summary_stats, 
+                  options = list(pageLength = 15, dom = 't', scrollX = TRUE),
+                  rownames = FALSE) %>%
+      DT::formatStyle(columns = names(summary_stats), fontSize = '12px')
+  })
+  
+  # Selected pitch info in modal
+  output$selected_pitch_info <- renderText({
+    pitch_info <- selected_pitch()
+    if (!is.null(pitch_info)) {
+      pitch_data <- pitch_info$data
+      
+      # Build info text with only available fields
+      info_lines <- c(
+        paste("Pitcher:", pitch_info$pitcher),
+        paste("Current Type:", pitch_data$TaggedPitchType),
+        paste("Velocity:", round(pitch_data$RelSpeed, 1), "mph"),
+        paste("Horizontal Break:", round(pitch_data$HorzBreak, 1), "inches"),
+        paste("Induced Vertical Break:", round(pitch_data$InducedVertBreak, 1), "inches")
+      )
+      
+      # Add optional fields only if they exist and have values
+      if ("SpinRate" %in% names(pitch_data) && !is.na(pitch_data$SpinRate)) {
+        info_lines <- c(info_lines, paste("Spin Rate:", round(pitch_data$SpinRate, 0), "rpm"))
+      }
+      
+      if ("Date" %in% names(pitch_data) && !is.na(pitch_data$Date)) {
+        info_lines <- c(info_lines, paste("Date:", pitch_data$Date))
+      }
+      
+      return(paste(info_lines, collapse = "\n"))
+    } else {
+      return("No pitch selected")
+    }
+  })
   
   # Update pitch type
   observeEvent(input$update_pitch, {