sensible-cinema 0.25.1 → 0.25.2

data/Rakefile

@@ -9,22 +9,22 @@ Jeweler::Tasks.new do |s|
     s.email = "rogerdpack@gmail.com"
     s.homepage = "http://github.com/rdp"
     s.authors = ["Roger Pack"]
-    s.add_dependency 'os', '>= 0.9.3'
-    s.add_dependency 'sane', '>= 0.22.0'
-    s.add_dependency 'rdp-win32screenshot', '>= 0.0.7.3'
+    s.add_dependency 'os', '>= 0.9.4'
+    s.add_dependency 'sane', '>= 0.24.2'
+    s.add_dependency 'rdp-win32screenshot', '= 0.0.9'
     s.add_dependency 'mini_magick', '>= 3.1' # for ocr...
     s.add_dependency 'whichr', '>= 0.3.6'
-    s.add_dependency 'jruby-win32ole'
-    s.add_dependency 'rdp-ruby-wmi' # windows
-    s.add_dependency 'plist' # mac
-    s.add_dependency 'ffi' # mouse, etc. needed at least for MRI
+    s.add_dependency 'rautomation', '> 0.6.3'
+    s.add_dependency 'rdp-ruby-wmi' # for windows
+    s.add_dependency 'plist' # for mac
+    s.add_dependency 'ffi' # mouse, etc. needed at least for windows MRI
     s.files.exclude '**/*.exe', '**/*.wav', '**/images/*'
     s.add_development_dependency 'hitimes' # now jruby compat!
     s.add_development_dependency 'rspec', '> 2'
     s.add_development_dependency 'jeweler'
     s.add_development_dependency 'rake'
     
-    # add as real dependencies for now, as gem install --development is still broken for jruby, basically installing transitive dependencies in error <sigh>
+    # add as real dependencies for now, as gem install --development is still broken for jruby, basically installing transitive dependencies in error <sigh> (actually might be fixed now though...so we may not need this)
 
     for gem in s.development_dependencies #['hitimes', 'rspec', 'jeweler', 'rake']
       # bundling rake won't be too expensive, right? and this allows for easier dev setup through gem install

data/TODO

@@ -1,6 +1,5 @@
 == roadmap [for just tasks, see below though] == 
 
-  release: cheap/mediumcheap inventionzy
   release: C EDL
   compare netflix/blu-ray timings et al
   
@@ -17,28 +16,29 @@
 == high prio like blocking bugs ==  
   
 == under next release specific items: ==
-   cheap minimize via (probably minimize foreground window/windows API) 
-   url -> url inference [hopefully this is cake...]
-   can handle multiple dvdid's et al
-   VLC "insert foreign missing scene" supah easy demo
-   overlay section with image (mplayer?)
-  add some "fake" "now pg" "now pg-12" "violence for age y" +- with edits at all even.
+  it can filter out specific profanities even, too :P
 
 == yes do ordered in conjunction with the above probably ==
-  release at some point...Oliver? facebook?
-  snapshots of snow white "normal", then deploy the various pics
+
+== yes do, un-ordered currently, from the various future release options ==
+  both mute and blank when blanking by default... (add the other type in there, esp. if a man answers)
+  cars EDL x2
   lightning strike from various outlets...
+  release at some point...Oliver? facebook? (one at a time...)
   research VOB
     timesplits?
-
-== yes do, un-ordered currently, from the various future release options ==
   todo.upconvert
-    except I'm really close [?] with netflix instant upconverter...hmm...
-  new open button "edit the one for current DVD"
-  the other button should open in the "editing" EDL dir.
-  overlapping mute failure when you hit the "watch edited realtime" should popup.
-  split out the "dependency" code from main.
-  try buy edit lists from all x,y,z [?] maybe ask for them for free first? ()
+    netflix quality upconverter
+    have people test it
+    me test it
+      1x, 2x Sintel
+      snapshots of Sintel "normal", then deploy the various comparison pics
+  netflix instant for OS X
+  add new open button "edit the one for current DVD"
+  the open button should open in the "editing" EDL dir.
+  an overlapping mute failure when you hit the "watch edited realtime" should popup.
+  split out the "dependency" code
+  try buy edit lists from all x,y,z [?] offer them money LOL
     http://www.familysafemovies.com
     cleanflix (earl? that guy [John webster http://www.amazon.com/Make-Edited-Movies-ebook/dp/B004EYT3BU]?)
     cougar video (that looks tough...)
@@ -52,72 +52,60 @@
     family flix
     http://theeditedmovieencyclopedia.blogspot.com/2008/08/clean-play-dvds-grand-opening-e-mail.html has grande list
     movie mask wayback machine
-  make it load faster! this is cuh-razy! or splash screen! yes!
-  netflix quality upconverter
-    want a prototype mplayer upconverter...I guess... 
-  add back in scale for OS X, see if it's expensive or not...I'd guess not though...
+  make it load faster! this is cuh-razy! or at least splash screen! yes!
   research XBMC
     @XBMC: your stuff is broken in at least these two ways (bug reports)
     also are they 30fps or not? and how do they handle timesplits?
     is there another format that will actually mute?
+    XBMC mode doesn't mind the 600 starting offset...
+      oh wait we weren't even using it right to begin with LOL
+      plus it needs to "always just skip instead of mute" for now I guess
+      report XBMC bugs, too.
   split into various separate windows themselves...
   consistency for flight of nav. srt? huh?
     consistency of others, also "DVD reported full-time versus ts time" for others...
-  XBMC mode doesn't mind the 600 starting offset...
-    oh wait we weren't even using it right to begin with LOL
-    plus it needs to "always just skip instead of mute" for now I guess
-    report XBMC bugs, too.
-  all unit tests
-  subs: if at beginning of a sub, add extra offset...hmm...why not?
-  say something if the EDL is actually "blank" :)
-  figure out if an offset of '1s' should also be accomodated for...
-    0.3 ... ? huh?
-  fix released version to work in mac
-  release capture filter, advertise it :)
-  report ffmpeg bug, mplayer various :P
-  note "you want to optimize your reverse blitting first... :)"
-    test speed of from DESKTOP versus HWND.
-    can use hwnd.  Somehow automated.
+  subtitles: if at beginning of a sub, add extra offset...hmm...why not?
+  say something if the EDL is actually "empty" :)
+  fix released version to downloadable work in mac
+  report ffmpeg bug, mplayer various (as bugs):P
   "--developer" mode
     export JSON command line, note it in the docu, show it in --developer mode, no new bat file for developer mode
   new button "auto-test" a subtitles file for timing :)
   new upconv button: auto-take some snapshots (just to straight mjpeg or jpeg or the what not)
-  publish a ripped raw big bunny
-    update development howto
   @jruby scan court jester is *so* slow...what the...[mac at least...]
-  urug again?
+  advertise urug again?
     after works in mac. Why not?
-  if 2 EDL's, still can choose a title track
   advertise virtual audio
+  advertise virtual desktop :)
+  mplayer directx patch check it...
   auto-add largest title to the generator EDL
     complain if times looks wrong...on playback? sum time
   tighten old splits... [remove, really, I guess...sure while I only have few...and also add better instructions...]
   "save to" -> same dir as "get from" yeah
-  open EDL can have a rememberable DIR
+  open EDL to edit it can have a rememberable DIR
   @instructions "this is great for comparing srt's, except...when it isn't, then you have to..."
   @instructions: full srt how-to document instead of tooltip [?] or combine the two! yes!
   volume_name => dvd_volume_name
-  C-based stuff cleanup loose ends
   why is the Sintel's EDL seemingly off?
     ascertain how accurate mplayer EDL is...I guess compared to...umm...something?
       fix discrepancy between EDL mplayer and ffmpeg...maybe a new button "accurate EDL playback"?
       maybe search at end of movies for certain things?
     double check--are my ffmpeg splits really really really the same as mplayer's EDL DVD splits? really?
   linearize: EDL with JSON on top, full timestamps.
-  setpriority for several :)
   bring to top after a conversion, and before prompts...
   can auto-use tsmuxer at all [?]
     mac just die with instructions to use tsmuxer first...
-  check if some buttons are now outdated...
+    actually don't do this, just make sure to warn them if they pass us an mkv...indeed.
+  check if some create buttons are now outdated...
   check with blu-ray...do they match timing-wise? sub-title wise?
-  donate tsmuxer, mplayer, http://ffmpeg.zeranoe.com/builds, mulder +- [?] smplayer menumeter http://instantitunes.wordpress.com/2010/02/26/use-tsmuxer-on-snow-leopard/
-  already donated: unlocker, mac fan thing, open office, many sourceforge tops, 7zip, VLC, cannot donate: ffmpeg
+  donate tsmuxer (sent email), http://ffmpeg.zeranoe.com/builds, mulder +- [?] smplayer menumeter http://instantitunes.wordpress.com/2010/02/26/use-tsmuxer-on-snow-leopard
   youtube/hulu timings
   split LLC's website+consumer
     get LLC before a "real" domain name
   employee instructions: watch @ realtime after profanity filter first pass :P  
-  @makemkv: timing feels broken weird-like!
+  @makemkv: timing feels broken weird-like! I'll donate you more money if you fix it <sniff>
     try out libdvdcss + handbrake on the one that was totally broken...is it better?
+    it should work like a champ shouldn't it?
   clear runnings
     have ben and chris test it out, have it work, then ask about video quality
     me try
@@ -145,9 +133,8 @@
   create a semi-accurate phantom edit EDL
     http://www.chefelf.com/forums/index.php?showtopic=6176
   inventionzy: blu-ray stuff from todo.inventionzy
-
   port to mac OS X
-    ppc dvdid? documentation button?
+    ppc dvdid? documentation button? notepad?
   the final "edited.avi" really "could/should" be playable in WMP...if possible
   add button "fast preview segment" (from dvd)
     add button "fast preview one" (from file) smplayer
@@ -161,7 +148,7 @@
     or maybe...umm...ask the dvdnav people why it might work with one and not the other? ask VLC? huh? what the...
     call out to VLC on DVDNAV_ERROR occurrence? what the... if can reproduce, submit bug report with trace et al.
 
-== other lists ==
+== other lists to do ==
 
   todo.upconvert
   todo.propaganda
@@ -169,21 +156,25 @@
   open_edl_list_org_todo.txt file
   todo.build_library
   todo.subtitle
+  todo.virtual_audio
 
 == DVD decide later to move up or down ==
 
-== DVD-realtime backlog once yes do, now not do, unordered ==
-  can I just use ffmpeg for an initial -> ts ? http://eotmsnotes.wordpress.com/2011/04/25/tsmuxer-notes/
-  investigate xbmc dvd edl how do you do that? [+- real dvd]
+== DVD+-DVD-realtime backlog (totally unordered, some very low prio, basically all never do) ==
+  already donated: unlocker, mac fan thing, open office, many sourceforge tops, 7zip, VLC, mplayer, cannot donate: ffmpeg
+  note "you want to optimize your reverse blitting first... :)"
+  if 2 EDL's, still can choose a title track
   fast preview: change it to be aggressive sometimes yes, other times no
   fast preview mode: watch for modifications to file, kill mplayer instances for them on change :)
   fast preview tends to "go too far" [like the end of profanity is still there when played via the "real way"]
-  mplayer EDL while grabbing, from grabbed :P
   note to them [in how_to_create]: can do the two types of preview at same time...concurrently...to save time...
-  can I make playback more smooth by upping mplayer cache? does smplayer pass it a parameter?
   investigate: can I use zoomplayer max?
 
-== DVD+-DVD-realtime backlog (totally unordered, some very low prio, basically all never do) ==
+  can I make playback more smooth by upping mplayer cache? does smplayer pass it a parameter? It does jerk a bit doesn't it, on slower?
+  mplayer EDL while grabbing, from grabbed :P
+  can I just use ffmpeg for an initial -> ts ? http://eotmsnotes.wordpress.com/2011/04/25/tsmuxer-notes/
+  investigate xbmc dvd edl how do you do that? [+- real dvd]
+  release capture filter, advertise it :)
   @ffdshow-tryouts: don't crash me!
   conference it can "pick out songs" (like a music video? mp3's?)
   check with WME plugins, add instructions, then note in the README "support for x..."
@@ -301,7 +292,6 @@
   forever strong DVD timestamps [and other?], compare timestamps with amazon VoD, too, also jonah amazon VoD [if desired?]
   take out the extra console window box output :) [scary, and closing it closes SC!]
   auto-feedback on crashes [?]
-  publish new version of rdp-win32screenshot, if it ever existed
   script avidemux, virtualdub, as a demo?
   accomodate failed deletes [files in use] better than dying with a backtrace
   vendorize ffmpeg, to avoid "scary" warning on install of imagemagick/scary active x

data/VERSION

@@ -1 +1 @@
-0.25.1
+0.25.2

data/bin/sensible-cinema

@@ -20,7 +20,7 @@ This file is part of Sensible Cinema.
 print 'Starting Sensible Cinema...'
 require File.expand_path(File.dirname(__FILE__) + "/../lib/add_any_bundled_gems_to_load_path.rb")
 begin
-  require 'sane' 
+  require 'sane'
 rescue LoadError
   p 'please run $ rake rebundle_copy_in_dependencies first, as we dislike needing locally installed rubygems'
   raise
@@ -45,12 +45,13 @@ end
 require_relative '../lib/swing_helpers'
 
 # attempt to load on demand...i.e. faster...
-for kls in [:MencoderWrapper, :EdlParser, :MplayerEdl, :PlayAudio, :SubtitleProfanityFinder, :ConvertThirtyFps, :RubyClip, :DriveInfo]
+for kls in [:MencoderWrapper, :MplayerEdl, :PlayAudio, :SubtitleProfanityFinder, :ConvertThirtyFps, :RubyClip, :DriveInfo]
   autoload kls, "./lib/#{kls.to_s.snake_case}"
 end
 
-# a few I'll need...
+# a few I'll always need eventually...
 require_relative '../lib/storage'
+require_relative '../lib/edl_parser'
 require 'tmpdir'
 require 'whichr'
 require 'os'
@@ -78,6 +79,7 @@ else
 end
 
 import 'javax.swing.ImageIcon'
+require_relative 'sensible-cinema-dependencies'
 
 module SensibleSwing
   VERSION = File.read(File.dirname(__FILE__) + "/../VERSION").strip
@@ -86,7 +88,7 @@ module SensibleSwing
   UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName()) # sigh
   
   class MainWindow < JFrame
-
+    
     def we_are_in_create_mode
      ARGV.index("--create-mode")
     end
@@ -345,7 +347,7 @@ module SensibleSwing
       
       @open_list = new_jbutton("Open/Edit a previously created EDL file", "If your DVD has a previously existing EDL for it, you can open it to edit it with this button.")
       @open_list.on_clicked {
-        filename = new_existing_file_selector_and_select_file( "Pick any file to open in editor", EDL_DIR)
+        filename = new_existing_file_selector_and_select_file( "Pick any file to open in editor", EdlParser::EDL_DIR)
         open_file_to_edit_it filename
       }
       
@@ -566,7 +568,7 @@ module SensibleSwing
     
     def choose_file_and_edl_and_create_sxs_or_play just_create_dot_edl_file_instead_of_play
       filename_mpg = new_existing_file_selector_and_select_file( "Pick moviefile (like moviename.mpg or video_ts/anything.ext)")
-      edl_filename = new_existing_file_selector_and_select_file( "Pick an EDL file to use with it", EDL_DIR)
+      edl_filename = new_existing_file_selector_and_select_file( "Pick an EDL file to use with it", EdlParser::EDL_DIR)
       assert_ownership_dialog
       if just_create_dot_edl_file_instead_of_play
         descriptors = EdlParser.parse_file edl_filename
@@ -656,7 +658,6 @@ module SensibleSwing
 
     def run_smplayer_blocking play_this, title_track_maybe_nil, extra_options, force_use_mplayer, show_subs
       unless File.exist?(File.expand_path(play_this))
-        #_dbg
         raise play_this + ' non existing?' # till these go away in mac :)
       end
 
@@ -782,7 +783,7 @@ module SensibleSwing
 # "mplayer_dvd_splits" => ["59:59", "1:04:59"], # or [] if there are none.  Additive currently.  12345.6 ok. In mplayer, the DVD timestamp "resets" to zero for some reason, so you need to specify when if you want to use mplayer DVD realtime playback, or use mencoder -edl to split your file.  See http://goo.gl/yMfqX
         EOL
       # TODO auto-ify above, move docs to a file in documentation.
-      filename = EDL_DIR + "/edls_being_edited/" + english_name.gsub(' ', '_') + '.txt'
+      filename = EdlParser::EDL_DIR + "/edls_being_edited/" + english_name.gsub(' ', '_') + '.txt'
       filename.downcase!
       File.write(filename, input) unless File.exist?(filename) # lodo let them choose name (?)
       open_file_to_edit_it filename
@@ -801,14 +802,12 @@ module SensibleSwing
           begin
             p = proc{ ole = WMI::Win32_Process.find(:first,  :conditions => {'Name' => exe_name}); sleep 1 unless ole; ole }
             piddy = p.call || p.call || p.call # we actually do need this to loop...guess we're too quick
-            # but the first time through this still inexplicably fails all 3... LODO
-            if piddy
-              # piddy.SetPriority low_prio # this call can seg fault at times...yikes...JRUBY-5422
+            # but the first time through this still inexplicably fails all 3...odd
+            piddys = WMI::Win32_Process.find(:all,  :conditions => {'Name' => exe_name})
+            for piddy in piddys
+              # piddy.SetPriority low_prio # this call can seg fault at times...JRUBY-5422
               pid = piddy.ProcessId # this doesn't seg fault, tho
-              system_original("vendor\\setpriority -lowest #{pid}") # use the PID on the command line
-            else
-              # XXXX first one always fails [?] huh?
-              p 'unable to find to set priority ' + exe_name
+              system_original("vendor\\setpriority -lowest #{pid}") # uses PID for the command line
             end
           rescue Exception => e
             p 'warning, got exception trying to set priority [jruby prob? ...]', e
@@ -831,12 +830,6 @@ module SensibleSwing
       undef system
     end
     
-    def download full_url, to_here
-      require 'open-uri'
-      writeOut = open(to_here, "wb")
-      writeOut.write(open(full_url).read)
-      writeOut.close
-    end
    
     def play_dvd_smplayer_unedited use_mplayer_instead, show_instructions, show_subs
       drive_or_file, dvd_volume_name, dvd_id, edl_path_maybe_nil, descriptors_maybe_nil = choose_dvd_or_file_and_edl_for_it false
@@ -854,6 +847,7 @@ module SensibleSwing
     if OS.doze? # avoid spaces :)
       EdlTempFile = EightThree.convert_path_to_8_3(Dir.tmpdir + '/mplayer.temp.edl')
     else
+      raise if Dir.tmpdir =~ / / # that would be odd...
       EdlTempFile = Dir.tmpdir + '/mplayer.temp.edl'
     end
     
@@ -993,8 +987,8 @@ module SensibleSwing
       Dir.mkdir('./vendor/cache') unless File.directory? 'vendor/cache' # development may not have it created yet... [?]
       unless File.exist? 'vendor/cache/7za.exe'
         Dir.chdir('vendor/cache') do
-          print 'downloading unzipper (400K) ...'
-          download("http://downloads.sourceforge.net/project/sevenzip/7-Zip/9.20/7za920.zip", "7za920.zip")
+          print 'downloading unzipper (7zip--400K) ...'
+          MainWindow.download("http://downloads.sourceforge.net/project/sevenzip/7-Zip/9.20/7za920.zip", "7za920.zip")
           system_blocking("../unzip.exe -o 7za920.zip") # -o means "overwrite" without prompting
         end
       end
@@ -1005,7 +999,7 @@ module SensibleSwing
       Dir.chdir('vendor/cache') do
         file_name = url.split('/')[-1]
         print "downloading #{english_name} ..."
-        download(url, file_name)
+        MainWindow.download(url, file_name)
         system_blocking("7za e #{file_name} -y -o#{to_this}")
         puts 'done ' + english_name
         # creates vendor/cache/mencoder/mencoder.exe...
@@ -1076,27 +1070,6 @@ module SensibleSwing
       end
     end
 
-    def single_edit_list_matches_dvd dvd_id
-      matching =  Dir[EDL_DIR + '/**/*.txt'].select{|file|
-        begin
-          parse_edl(file)["disk_unique_id"] == dvd_id
-        rescue SyntaxError => e
-          # ignore poorly formed edit lists for the auto choose phase...
-          p 'warning, unable to parse a file:' + file + " " + e.to_s
-          false
-        end
-      }
-      if matching.length == 1
-        file = matching[0]
-        p "selecting the one only matching EDL #{file} for this DVD ID: #{dvd_id}"
-        file
-      else
-        nil
-      end
-    end
-
-    EDL_DIR = File.expand_path(__DIR__  + "/../zamples/edit_decision_lists/dvds").to_filename
-
     def repeat_last_copy_dvd_to_hard_drive
       generate_and_run_bat_file *LocalStorage['last_params']
     end
@@ -1157,9 +1130,9 @@ module SensibleSwing
       drive_or_file, dvd_volume_name, dvd_id = choose_dvd_drive_or_file false
       
       unless @_edit_list_path # cache file selection...
-        edit_list_path = single_edit_list_matches_dvd(dvd_id)
+        edit_list_path = EdlParser.single_edit_list_matches_dvd(dvd_id)
         if !edit_list_path && force_choose_edl_file_if_no_easy_match
-          edit_list_path = new_existing_file_selector_and_select_file("Please pick a DVD Edit List File (none or more than one were found that seem to match #{dvd_volume_name})--may need to create one for it", EDL_DIR)
+          edit_list_path = new_existing_file_selector_and_select_file("Please pick a DVD Edit List File (none or more than one were found that seem to match #{dvd_volume_name})--may need to create one for it", EdlParser::EDL_DIR)
           raise 'cancelled choosing an EDL' unless edit_list_path
         end
         @_edit_list_path = edit_list_path
@@ -1224,6 +1197,7 @@ module SensibleSwing
       
       descriptors = parse_edl(edit_list_path)
       if watch_unedited
+        # reset them
         descriptors['mutes'] = descriptors['blank_outs'] = []
       end
       
@@ -1254,7 +1228,13 @@ module SensibleSwing
         LocalStorage['end_time'] = end_time
       end
       dvd_friendly_name = descriptors['name']
-      raise 'no dvd name in EDL?' unless dvd_friendly_name
+      unless dvd_friendly_name
+        _dbg
+              drive_or_file, dvd_volume_name, dvd_id, edit_list_path, descriptors = choose_dvd_or_file_and_edl_for_it
+
+        descriptors = parse_edl(edit_list_path)
+        raise 'no dvd name in EDL?' 
+      end
       
       dvd_title_track = get_title_track(descriptors)
       if dvd_id == NonDvd

data/bin/sensible-cinema-cli

@@ -20,9 +20,7 @@ This file is part of Sensible Cinema.
 require File.dirname(__FILE__) + "/../lib/add_any_bundled_gems_to_load_path.rb"
 require 'sane'
 
-Thread.abort_on_exception=true # gah
-
-for file in ['overlayer', 'keyboard_input', 'screen_tracker', 'mouse', 'file_chooser', 'ocr', 'vlc_programmer']
+for file in ['overlayer', 'keyboard_input', 'screen_tracker', 'mouse', 'file_chooser', 'ocr', 'vlc_programmer', 'edl_parser', 'auto_window_finder']
   require_relative '../lib/' + file
 end
 
@@ -31,7 +29,7 @@ def go_sc(args)
   $VERBOSE = 1 if args.delete('-v')
   $DEBUG = 1 if args.delete('-d')
   if args.delete('--clear-cache')
-    OCR.clear_cache! 
+    OCR.clear_cache!
     puts 'cleared cache'
   end
   
@@ -41,95 +39,119 @@ def go_sc(args)
   
     puts <<-END
   
-    syntax: [player_description.yml] [delete_list.txt|test] [-t -v --clear-cache] (or just nothing at all--it will prompt for all things needed)
+    syntax: [player_description.yml] [delete_list.txt|test] [-t, -v, --clear-cache] (or just nothing at all--it will prompt for all things it needs)
     
-    If you specify "test" for the edit list, it will pause 4s, take a snapshot of the player, then exit.
+    If you specify "test" for the edit list, it will pause 2s, take a single snapshot of the selected player, then exit.
     Useful for debugging your screen capture of the current player.
     You can also specify -v or -t if you want to enable more verbose (chatty) output.
     
     END
       
-    for file in Dir[__dir__ + '../zamples/mute*/*']
+    for file in Dir[__DIR__ + '../zamples/mute*/*']
        puts "\n", "Example file:", file + "\n\n", File.read(file)
     end
     
     exit 1
-    
-  else
-   
-    # allow for command line filenames
+  end
   
-    player_description = args.shift.to_s
-    if !File.exist?(player_description)    
-      puts 'Please Select Computer Player'
-      player_description = FileChooser.choose_file("     SELECT COMPUTER PLAYER", __dir__ + "/../zamples/players")
+  players_root_dir = __DIR__ + "/../zamples/players"
+  # allow for command line filenames
+  player_description = args.shift
+  unless player_description
+    player_description = AutoWindowFinder.search_for_player_and_url_match(players_root_dir)
+    if player_description
+      p 'auto selected player ' + player_description 
+    else
+      player_description = ''
     end
+  end
   
-    edit_decision_list	 = args.shift.to_s
-  
-    if edit_decision_list	 == 'test'
-      overlay = nil
-      p 'got test...just doing screen dump'
-      $VERBOSE=true # adds some extra output
-    else
-      if !File.exist? edit_decision_list	
-        puts 'Select Edit Decision List'
-        edit_decision_list	 = FileChooser.choose_file("     SELECT EDIT DECISION LIST", __dir__  + "/../zamples/edit_decision_lists")
-      end
-      
-      if !edit_decision_list	
-        puts "error: have to specify a scene descriptions file\n    or specify \"test\" on the command line if you just want to snapshot your player"
-        exit 1
-      end
-      
-      puts 'Selected scene descriptions file ' + File.basename(edit_decision_list) + "\n\t(full path: #{edit_decision_list})"
-      Blanker.startup
-      # todo start it late as it has an annoying startup blip
-      overlay = OverLayer.new(edit_decision_list	)    
-    end  
-    
-    if File.exist? player_description.to_s
-      puts 'Selected player ' + File.basename(player_description) + "\n\t(full path: #{player_description})"
-      # this one doesn't use any updates, so just pass in file contents, not filename
-      screen_tracker = ScreenTracker.new_from_yaml File.binread(player_description), overlay
-      does_not_need_mouse_jerk = YAML.load_file(player_description)["does_not_need_mouse_movement"]
-      unless does_not_need_mouse_jerk
-        p 'yes using mouse jitter' if $VERBOSE or $DEBUG
-        Mouse.jitter_forever_in_own_thread # when this ends you know a snapshot was taken...
+  if !File.exist?(player_description)    
+    puts 'Please Select Computer Player'
+    player_description = FileChooser.choose_file("     SELECT COMPUTER PLAYER", players_root_dir)
+    raise unless player_description
+  end
+
+  edit_decision_list = args.shift.to_s
+
+  if edit_decision_list == 'test'
+    overlay = nil
+    p 'got test...just doing screen dump'
+    $VERBOSE=true # adds some extra output
+  elsif File.exist? edit_decision_list
+    # accept it from the command line
+  else
+    # assume they want an online player, right? LODO player can tell us...
+    auto_found = AutoWindowFinder.search_for_single_url_match
+    if auto_found
+      p 'auto-discovered open window for EDL, using it ' + auto_found
+      settings = EdlParser.parse_file auto_found
+      if settings["from_url"]
+         require 'tempfile'
+         to = Tempfile.new 'abc'
+         require_relative 'sensible-cinema-dependencies'
+         SensibleSwing::MainWindow.download(settings["from_url"], to.path)
+         edit_decision_list = to.path
       else
-        p 'not using mouse jitter' if $VERBOSE or $DEBUG
+        edit_decision_list = auto_found
       end
-      
-      # exit early if we just wanted a screen dump...a little kludgey...
-      unless overlay
-        puts 'warning--only doing screen dump in t-minus 2s...'      
-        sleep 2
-        puts 'snap!'
-        screen_tracker.dump_bmps
-        exit 1
-      end
-      screen_tracker.process_forever_in_thread
     else
-      puts 'warning--not using any screen tracking...'
+      puts 'Select Edit Decision List to use'
+      edit_decision_list = FileChooser.choose_file("     SELECT EDIT DECISION LIST", __DIR__  + "/../zamples/edit_decision_lists")
+    end
+    
+    if !edit_decision_list	
+      puts "error: have to specify a scene descriptions file\n    or specify \"test\" on the command line if you just want to snapshot your player"
+      exit 1
     end
     
-    OCR.unserialize_cache_from_disk # do this every time so we don't delete it if they don't have one...
+    puts 'Selected scene descriptions file ' + File.basename(edit_decision_list) + "\n\t(full path: #{edit_decision_list})"
+    Blanker.startup
+    # todo start it late as it has an annoying startup blip
+    overlay = OverLayer.new(edit_decision_list	)    
+  end  
   
-    puts "Opening the curtains... (please play in your other video player now)"
-    overlay.start_thread true
-    key_input = KeyboardInput.new overlay
-    key_input.start_thread # status thread
-    at_exit {
-      Blanker.shutdown # lodo move this and the 'q' key to within overlayer
-      OCR.serialize_cache_to_disk
-    }
-    begin
-      key_input.handle_keystrokes_forever # blocking...
-    ensure
-      puts "syntax from command line: \"#{player_description}\" \"#{edit_decision_list}\""
+  if File.exist? player_description.to_s
+    puts 'Selected player ' + File.basename(player_description) + "\n\t(full path: #{player_description})"
+    # this one doesn't use any updates, so just pass in file contents, not filename
+    screen_tracker = ScreenTracker.new_from_yaml File.binread(player_description), overlay
+    does_not_need_mouse_jerk = YAML.load_file(player_description)["does_not_need_mouse_movement"]
+    unless does_not_need_mouse_jerk
+      p 'yes using mouse jitter' if $VERBOSE or $DEBUG
+      Mouse.jitter_forever_in_own_thread # when this ends you know a snapshot was taken...
+    else
+      p 'not using mouse jitter' if $VERBOSE or $DEBUG
     end
     
+    # exit early if we just wanted a screen dump...a little kludgey...
+    unless overlay
+      puts 'warning--only doing screen dump in t-minus 2s...'      
+      sleep 2
+      puts 'snap!'
+      screen_tracker.dump_bmps
+      exit 1
+    end
+    screen_tracker.process_forever_in_thread
+  else
+    puts 'warning--not using any screen tracking...'
   end
+  
+  OCR.unserialize_cache_from_disk # do this every time so we don't delete it if they don't have one...
+
+  puts "Opening the curtains... (please play in your other video player now)"
+  overlay.start_thread true
+  key_input = KeyboardInput.new overlay
+  key_input.start_thread # status thread
+  at_exit {
+    Blanker.shutdown # lodo move this and the 'q' key to within overlayer
+    OCR.serialize_cache_to_disk
+  }
+  begin
+    key_input.handle_keystrokes_forever # blocking...
+  ensure
+    puts "syntax from command line: \"#{player_description}\" \"#{edit_decision_list}\""
+  end
+    
 end
 
 if __FILE__ == $0