Update tape_sim3_UI.lua

Author: ZaurS2001

Audio Effects/ReaPyTapeSim3/tape_sim3_UI.lua

@@ -18,21 +18,17 @@ if not reaper.ImGui_CreateContext then
     return
 end
 
-local ctx = reaper.ImGui_CreateContext('ReaPyTapeSim3')
+local ctx = reaper.ImGui_CreateContext('Analog Tape Simulator')
 
 -- Variables
 local age = 50.0
 local tape_type_idx = 1 -- 1 = A, 2 = B
-
 local format_list = {"WAV", "MP3"}
 local format_idx = 1 -- 1 = WAV, 2 = MP3
-
 local sr_list = {"44100", "48000", "88200", "96000"}
 local sr_idx = 2 -- default 48000
-
 local bd_list = {"16", "24", "32"}
 local bd_idx = 2 -- default 24
-
 local br_list = {"128k", "192k", "256k", "320k"}
 local br_idx = 4 -- default 320k
 
@@ -45,19 +41,15 @@ local processing_frames = 0
 -- Locate python script
 local script_path = debug.getinfo(1, "S").source:match("@?(.*[\\/])")
 if not script_path then script_path = "" end
-local py_script = script_path .. "tape_sim3.py"
+local py_script = script_path .. "tape_sim.py"
 
 -- DSP Processing Execute
 function ExecuteProcessing()
     local ext = format_idx == 1 and "wav" or "mp3"
     local out_file = input_file .. "_taped." .. ext
 
-    -- Delete any old render sitting there to ensure we check for a fresh file later
-    if reaper.file_exists(out_file) then
-        os.remove(out_file)
-    end
+    if reaper.file_exists(out_file) then os.remove(out_file) end
 
-    -- Format CLI command
     local cmd = string.format('python "%s" -i "%s" -o "%s" -a %f -t %s --out-format %s --samplerate %s',
         py_script, input_file, out_file, age, tape_type_idx == 1 and "A" or "B", ext, sr_list[sr_idx])
 
@@ -67,90 +59,86 @@ function ExecuteProcessing()
         cmd = cmd .. string.format(' --bitrate %s', br_list[br_idx])
     end
 
-    -- Run processing and capture all stdout/stderr to log errors
     local handle = io.popen(cmd .. ' 2>&1')
     local result = ""
     if handle then
         result = handle:read("*a")
         handle:close()
     end
 
-    -- ERROR CHECKING: Did Python actually create the file?
     if not reaper.file_exists(out_file) then
-        reaper.ShowConsoleMsg("--- TAPE SIMULATOR PYTHON ERROR ---\n\n")
-        reaper.ShowConsoleMsg("Command run:\n" .. cmd .. "\n\n")
-        reaper.ShowConsoleMsg("Output/Error Log:\n" .. (result or "None") .. "\n")
-        reaper.ShowMessageBox("Processing failed!\n\nThe audio file was not created. This usually means Python, FFmpeg, or a required package (numpy, scipy, soundfile) is missing or errored.\n\nCheck the REAPER Console for the exact crash log.", "Processing Error", 0)
-        
+        reaper.ShowConsoleMsg("--- TAPE SIM ERROR ---\n" .. (result or "") .. "\n")
+        reaper.ShowMessageBox("Processing failed! Check Console for logs.", "Error", 0)
         processing = false
         return
     end
 
-    -- Handle output to Reaper depending on source
     if target_item and reaper.ValidatePtr(target_item, "MediaItem*") then
-        -- Apply strictly to timeline clip (Take addition)
         local take = reaper.AddTakeToMediaItem(target_item)
         local src = reaper.PCM_Source_CreateFromFile(out_file)
         reaper.SetMediaItemTake_Source(take, src)
         reaper.SetActiveTake(take)
         reaper.UpdateItemInProject(target_item)
     else
-        -- Create completely new track (For Drag/Drop or Browsed Files)
         reaper.InsertTrackAtIndex(0, true)
         local track = reaper.GetTrack(0, 0)
         reaper.GetSetMediaTrackInfo_String(track, "P_NAME", "Tape Sim Output", true)
         local item = reaper.AddMediaItemToTrack(track)
         reaper.SetMediaItemInfo_Value(item, "D_POSITION", 0)
-        
         local take = reaper.AddTakeToMediaItem(item)
         local src = reaper.PCM_Source_CreateFromFile(out_file)
         reaper.SetMediaItemTake_Source(take, src)
-        
         local length = reaper.GetMediaSourceLength(src)
         reaper.SetMediaItemInfo_Value(item, "D_LENGTH", length)
         reaper.UpdateItemInProject(item)
     end
 
-    reaper.Main_OnCommand(40047, 0) -- Peaks: Build any missing peaks
+    reaper.Main_OnCommand(40047, 0) 
     reaper.UpdateArrange()
 
     input_file = ""
     target_item = nil
 end
 
 function DrawUI()
-    local visible, open = reaper.ImGui_Begin(ctx, 'ReaPyTapeSim3', true, reaper.ImGui_WindowFlags_AlwaysAutoResize())
+    local visible, open = reaper.ImGui_Begin(ctx, 'Tape Simulator Offline Processor', true, reaper.ImGui_WindowFlags_AlwaysAutoResize())
     if not visible then return open end
 
     -- 1. DROP / BROWSE AREA
+    -- We push color, draw button, handle dragdrop, THEN pop color.
     reaper.ImGui_PushStyleColor(ctx, reaper.ImGui_Col_Button(), 0x333333FF)
 
-    local btn_text = input_file == "" and "Click to Browse File\n\n(or Drag & Drop File Here)" or "File Loaded:\n" .. input_file:match("([^\\/]+)$")
+    local btn_text = input_file == "" and "Click to Browse\nOR\n[ DRAG AUDIO FILE HERE ]" or "File Loaded:\n" .. input_file:match("([^\\/]+)$")
 
-    -- Opens File Explorer on click
-    if reaper.ImGui_Button(ctx, btn_text, 350, 70) then
+    -- Main Button
+    if reaper.ImGui_Button(ctx, btn_text, 350, 80) then
         local retval, filename = reaper.GetUserFileNameForRead("", "Select Audio File", "")
         if retval then
             input_file = filename
-            target_item = nil -- FORCE New track creation
+            target_item = nil
         end
     end
-    reaper.ImGui_PopStyleColor(ctx)
 
-    -- Handle Native Drag & Drop from OS / Media Explorer
+    -- Drag & Drop Handler (Must be immediately after the target item)
     if reaper.ImGui_BeginDragDropTarget(ctx) then
         local rv, payload = reaper.ImGui_AcceptDragDropPayload(ctx, 'DND_FILES')
         if rv then
-            for file in payload:gmatch("(.-)%z") do
+            -- DND_FILES returns paths separated by \0
+            local file_found = false
+            for file in payload:gmatch("[^%z]+") do 
+                -- Accept the first file found
                 input_file = file
-                target_item = nil -- FORCE New track creation
-                break
+                target_item = nil
+                file_found = true
+                break 
             end
         end
         reaper.ImGui_EndDragDropTarget(ctx)
     end
 
-    -- Dedicated button for Reaper Selected Items (Updates Item in Place)
+    reaper.ImGui_PopStyleColor(ctx) -- Pop color AFTER drag logic
+
+    -- Helper for Timeline Items
     if reaper.ImGui_Button(ctx, "Grab Selected Item from REAPER Timeline", 350, 25) then
         local item = reaper.GetSelectedMediaItem(0, 0)
         if item then
@@ -160,9 +148,11 @@ function DrawUI()
                 local path = reaper.GetMediaSourceFileName(src, "")
                 if path ~= "" then
                     input_file = path
-                    target_item = item -- Registers specific clip
+                    target_item = item
                 end
             end
+        else
+            reaper.ShowMessageBox("No item selected in timeline!", "Selection Error", 0)
         end
     end
 
@@ -217,10 +207,8 @@ function DrawUI()
 
     -- 4. PROCESSING LOGIC
     if processing then
-        reaper.ImGui_Text(ctx, "Processing... Please wait. REAPER may freeze briefly.")
+        reaper.ImGui_Text(ctx, "Processing... Please wait.")
         processing_frames = processing_frames + 1
-        
-        -- Delay execution so UI physically renders the text first
         if processing_frames > 2 then
             ExecuteProcessing()
             processing = false
@@ -239,19 +227,15 @@ function DrawUI()
         end
     end
 
-    if process_triggered and not processing then
-        processing = true 
-    end
+    if process_triggered and not processing then processing = true end
 
     reaper.ImGui_End(ctx)
     return open
 end
 
 function loop()
     local open = DrawUI()
-    if open then
-        reaper.defer(loop)
-    end
+    if open then reaper.defer(loop) end
 end
 
-reaper.defer(loop)
+reaper.defer(loop)