Update app.R

app.R

@@ -231,10 +231,13 @@ server <- function(input, output, session){
       have <- intersect(names(df), num_cols)
       df[have] <- lapply(df[have], function(x) suppressWarnings(as.numeric(x)))
 
-      df <- df %>% distinct()
+df <- df %>% distinct()
+
+if (!".uid" %in% names(df)) df$.uid <- seq_len(nrow(df))
+
+processed_data(df)
+plot_data(df)
 
-      processed_data(df)
-      plot_data(df)
 
       if ("Pitcher" %in% names(df)) {
         choices <- sort(unique(df$Pitcher[!is.na(df$Pitcher)]))
@@ -272,14 +275,12 @@ server <- function(input, output, session){
     datatable(processed_data(), options=list(scrollX=TRUE, pageLength=10), filter="top")
   })
 
-  # pitcher-specific reactive with stable row_key (per pitcher group)
-  pitcher_df <- reactive({
-    req(plot_data(), input$pitcher_select)
-    plot_data() %>%
-      filter(Pitcher == input$pitcher_select) %>%
-      filter(!is.na(HorzBreak), !is.na(InducedVertBreak), !is.na(RelSpeed)) %>%
-      mutate(row_key = row_number())
-  })
+pitcher_df <- reactive({
+  req(plot_data(), input$pitcher_select)
+  plot_data() %>%
+    filter(Pitcher == input$pitcher_select) %>%
+    filter(!is.na(HorzBreak), !is.na(InducedVertBreak), !is.na(RelSpeed))
+})
 
 output$movement_plot <- renderPlotly({
   d <- pitcher_df()
@@ -297,8 +298,8 @@ output$movement_plot <- renderPlotly({
     ),
     hoverinfo = "text",
     key = ~row_key,
-    color = ~factor(TaggedPitchType),      # <-- data-driven
-    colors = pitch_colors,                 # <-- your palette
+    color = ~factor(TaggedPitchType),      
+    colors = pitch_colors,                 
     marker = list(size = 10)
   ) |>
     layout(
@@ -406,13 +407,26 @@ output$location_plot <- renderPlotly({
     )
   })
 
-  # ---- Apply single-change from modal ----
-  observeEvent(input$update_pitch, {
-    info <- selected_pitch(); req(info)
-    new_type <- input$modal_new_pitch_type; req(new_type)
-    cur <- plot_data(); req("Pitcher" %in% names(cur))
-   cur <- plot_data(); req("Pitcher" %in% names(cur))
-  cur$TaggedPitchType <- as.character(cur$TaggedPitchType)  
+observeEvent(input$update_pitch, {
+  info <- selected_pitch(); req(info)
+  new_type <- input$modal_new_pitch_type; req(new_type)
+
+  cur <- plot_data(); req("Pitcher" %in% names(cur))
+  cur$TaggedPitchType <- as.character(cur$TaggedPitchType)   # ensure character
+
+  uid <- info$uid
+  hit_idx <- which(cur$.uid == uid)
+
+  if (length(hit_idx) == 1) {
+    cur$TaggedPitchType[hit_idx] <- new_type
+    plot_data(cur); processed_data(cur)
+    removeModal()
+    showNotification("Updated pitch tag.", type="message", duration=3)
+    selected_pitch(NULL)
+  } else {
+    showNotification("Could not map selection back to dataset.", type="error")
+  }
+}) 
 
     # map row_key back to absolute index for this pitcher slice
     idx_pitcher <- which(cur$Pitcher == info$pitcher)
@@ -451,25 +465,28 @@ output$location_plot <- renderPlotly({
     paste(nrow(sel), "points selected:", paste(names(cnt), "(", cnt, ")", collapse=", "))
   })
 
-  # bulk change
-  observeEvent(input$apply_bulk_change, {
-    req(input$selection_mode == "drag")
-    keys <- selected_keys(); req(length(keys) > 0)
-    new_type <- input$bulk_pitch_type; req(new_type)
-
-    cur <- plot_data(); req("Pitcher" %in% names(cur))
-    idx_pitcher <- which(cur$Pitcher == input$pitcher_select)
-    valid_keys <- keys[keys > 0 & keys <= length(idx_pitcher)]
-    abs_rows <- idx_pitcher[valid_keys]
-    cur$TaggedPitchType[abs_rows] <- new_type
+observeEvent(input$apply_bulk_change, {
+  req(input$selection_mode == "drag")
+  keys <- selected_keys(); req(length(keys) > 0)
+  new_type <- input$bulk_pitch_type; req(new_type)
 
-    plot_data(cur); processed_data(cur)
-    selected_keys(integer(0))
-    showNotification(paste("Updated", length(abs_rows), "pitches to", new_type),
-                     type="message", duration=3)
   cur <- plot_data(); req("Pitcher" %in% names(cur))
-  cur$TaggedPitchType <- as.character(cur$TaggedPitchType)
-  })
+  cur$TaggedPitchType <- as.character(cur$TaggedPitchType)   # ensure character
+
+  # keys from plotly are .uid values
+  uids <- as.integer(keys)
+  hit_idx <- which(cur$.uid %in% uids & cur$Pitcher == input$pitcher_select)
+
+  if (length(hit_idx) == 0) {
+    showNotification("No matching rows found for bulk edit.", type="warning"); return()
+  }
+
+  cur$TaggedPitchType[hit_idx] <- new_type
+  plot_data(cur); processed_data(cur)
+  selected_keys(integer(0))
+  showNotification(paste("Updated", length(hit_idx), "pitches to", new_type),
+                   type="message", duration=3)
+})
 
   # click info small text
   output$click_info <- renderText({