sensible-cinema 0.26.2 → 0.26.3

data/TODO

@@ -1,135 +1,126 @@
 == roadmap [for just tasks, see below though] == 
 
   remember this stuff is all "oh gosh I have to do that too?" crud time, don't even want it.
+    and "won't even be useful to people in the end so why bother?"
   netflix
     upconv netflix
     mac'ify OCR/Mouse
   upconvert DVD
-  srt expand, release
-  tweaks/docu
   C EDL
   
   === non ordered/eventually (also add line items for them) ===
     create a skype recorder
-    compare netflix/blu-ray timings et al
     netflix/skype "disable screen saver while running" program [?]
     linearize [?] hmm...
     re-update todo/all docu/user feedback etc.
-    create openedls.org/api-ify
     LLC
       if...back off to just the ffmpeg (plus an "[A,B]" viewer) only...but for now stick with just DVD playback [?] until people want more...
     no propaganda...
     no auto subtitle downloads...
     fix VLC dvdnav itself +- mplayer DVD EDL
+    fix mplayer broken DVD EDL
   
 == high prio like blocking bugs ==  
   
 == next things I was working on ==
   unit tests work
   screen recording sintel
-  more netflix instant timing
-    bluray jonah total time vs. other
-  
-== yes do ordered for release  ==
+  netflix instant CGI how good is their HD?
+  Ben's DVD (hook)
+
+== yes do ordered for release/other projects ja  ==
   test out DVD upscale
-    no super scale res?
+    real "screen record" of upconverting sintel, see if it shows a difference
+      see if I need to fix my capturer...
+    no super scale res necessary?
       
-  add srt instructions to docu, release
   fake "tenseness" metric
   netflix instant descriptor for OS X/work at all
-  post release notes to the google group
-  real "screen record" of upconverting sintel, see if it shows a difference
-    then see if I need to fix my receiver...
+  netflix upconvert
+  more netflix instant timing
+    do another srt comparison with netflix...see if it matches less 1s
+  C player feedback message
+  nextsprocket...
+  releases that include Debug, python too :)
 
 == yes do, un-ordered currently, from the various future release options ==
+  todo.upconvert
+    netflix quality upconverter, have a go at it
+      finalize screen capturer
+    me test it
+      1x, 2x Sintel
+      snapshots of Sintel "normal", then deploy the various comparison pics
+    DVD test it (+- with others testing it for me too)
+
+  "open EDL" should save dir
+  ri_for updatez
+  subtitle -> url everywhere
+  english name (movie name) mass everywhere
+  handle gates of he.....
+  ruby_prof stuff
   warn other wordpresses
   legal2
   test mplayer -vo gl at home
-  extra printf's for scanning for matching
-  Ben's DVD (hook)
+  add new batch file for play with netflix OCR
+  remove those extra printf's for scanning for matching
   Ben's other DVD's
-  from an EDL, it can "just play" for netflix (auto-open browser etc.)
   template-ify for netflix too "create for netflix"
     add netflix instant instructions,
-      then add a blog post comment after releasing it as template
-  netflix HD versus none...
-  prototype new style EDL
   "note" competitor upconverters
-  add a "record start stop" + record x time for the virtual audio and other
-  rename as sanitizer
-    note "we are in the creation phase and need your help"
+  add a "record start stop" for screen capture
   that extra .5s DVD reports for file size...
-  TODO test DVD upconv it at 720*x resolution too...
-  investigate the srt morpher...
+  test DVD upconv it at 720*x resolution too...
   EDL gottman
-  bounty hqdn3d for ffmpeg
-  Greg's DVD's
-  srt/parsing buttons new window [?]
-  demo of conference "music video from file"
+  EDL Greg's DVD's
+  opening/creating buttons -> new separate sub-window
+  add a demo of conference "music video from local file"
   change upconvert options needs its own "close" button
-  real docu on how to use the srt scanner
+    and status field?
   lightning strike from various outlets...
-  accomodate splits vs. blanks and work
-  todo.upconvert
-    netflix quality upconverter, have a go at it
-      finalize screen capturer
-    me test it
-      1x, 2x Sintel
-      snapshots of Sintel "normal", then deploy the various comparison pics
-    DVD test it (+- with others testing it for me too)
-  add new open button "edit the EDL for current DVD"
+  accomodate splits vs. blanks and work everywhere
+  linearize: full EDL with JSON on top, full timestamps. or something I like...
+  add new open button "edit EDL for current DVD"
   add "play at 1.5x speed" mplayer button, I guess
-  play in mplayer unedited button should display OSD by default
-  bounty ffmpeg stuff (motion detect, directshow)
-  "fix" mplayer EDL pausing their code
+  short list of todo's for screen capture
+  "fix" mplayer EDL pausing (within the mplayer code itself)
+    bounty it for now/test with latest ffmpeg et al, maybe on mac with source [?
   the open button should open in the "editing" EDL dir.
-  warn against makemkv (and check vs. handbrake, and offer a bounty)
   add the other type in there, esp. if a man answers
     add to the template, various can use it
   an overlapping mute failure when you hit the "watch edited realtime" should popup.
-  witch lady
   split out the "dependency" codes
   consistency for flight of nav. srt? huh?
     consistency of others, also "DVD reported full-time versus ts time" for others...
-  subtitles: if at beginning of a sub, add extra offset...hmm...why not?
+  subtitles: if at beginning of a sub, add extra initial offset...hmm...why not?
+    and end too why not?
   fix released version to downloadable work in mac [7zip won't cut it]
   "--developer" mode
     has "export JSON command line, reload code" , note it in the docu
-  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 [?])
   advertise urug again?
     after works in mac for capture+overlay :)
-  create skype recorder windows
-  mplayer directx patch check it...
+  create skype/google+ hangouts recorder windows
+  mplayer directx fix patch check it...
   auto-add largest title to the generator EDL
     complain if times look wrong...on playback? sum time
+  mv jruby-swing-helpers to lib dir
   splits... [add better instructions on how to be precise...]
   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
+  volume_name => dvd_volume_label
   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.
-  also bring to top after a conversion, and before prompts/file prompts...
-  can auto-use tsmuxer at all [?]
-    mac just die with instructions to use tsmuxer first...
-    actually don't do this, just make sure to warn them if they pass us an mkv...indeed.
   check with blu-rays...do they match timing-wise? sub-title wise? do subs match something else?
   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! 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
     me try
     have ben and chris test it out [realtime?]
-  try out disney digital copy
   C player:
     use DVD navigator (check)
     just use mplayer EDL syntax, plus a title track (or just detect track change, if long enough, assume... :P) :)
@@ -152,21 +143,21 @@
     once I can jump around, advertise as a phantom editor :)
       maybe even before?
   ask for more beta testers for use, ask for more beta testers for creation, too
-  create a semi-accurate phantom edit EDL [?] just with splits
+  create a semi-accurate phantom edit EDL [?] with just splits
     http://www.chefelf.com/forums/index.php?showtopic=6176
   add button "fast preview segment" (mplayer from dvd)
   investigate WMC "call out to smplayer + EDL" or the like...
     add to docu
-  refresh bug try to reproduce
   can edit books, too.
-    +- list/publish :) [like a manual list et al]
+    +- list/publish mine so far :)
   create the following EDL: 10 commandments DVD to skip the intro music LOL
   inventionzy: blu-ray stuff from todo.inventionzy
   sons of provo "music video" EDL
-  port to mac OS X
-    PPC dvdid
-    documentation button? notepad?
-  ffmpeg compile it mingw, see if it runs dshow windows
+  ffmpeg compile it in mingw, see if it runs dshow windows
+  clarify input for srt (remove those other ones, that's lame-o)
+    it needs to remember the timings "start timing" and text, "end timing" and text
+    also double check unit tests, and if I do one in the middle does it still work?
+    and why was there an extra uncaught profanity in wrinkle in time?
   
 == other lists to not do (duplicate the todo here if want to actually do it though, basically no do) ==
 
@@ -180,15 +171,44 @@
   screen_capture TODO
 
 == DVD decide later to move up or down ==
-  add option "just skip don't mute"
-  
+ 
 == DVD+-DVD-realtime backlog (totally unordered, some very low prio, basically all never do) ==
+  "srt" double checker "ok now at exactly this time this should be here."
+   java webstart for being able to playback netflix instant edited "automagically"
+    also it can uses key presses/buttons to log keystrokes to know when edits should occur/associate them with timestamps
+      it can publish them easily/share them easily with friends. Oh boy.
+  can auto-use tsmuxer at all
+  port to mac OS X
+    documentation button?
+    PPC dvdid work?
+  .srt input auto-scan from a "full two entries from the .srt file" (easier)
+  rename as sanitizer, or something else SEO friendlier or something like that
+  from an EDL, it can "just play" for netflix (auto-open browser etc.)
+  add button "re-order srt"
+  @makemkv: 
+    try out libdvdcss + handbrake on the one that was totally broken...is it better?
+    can I reproduce it this days? double check OSOH, repost to forum
+  advertise http://betterlogic.com/roger/2011/06/mkv-mencoder-too-many-audio-packets-in-the-buffer/
+  add option "just skip don't mute always instead of muting"
+  bounty/ask hqdn3d for ffmpeg
+    bounty /ask mplayer OS X screen capture input can I use it in ffmpeg?
+    ask dll, say, for pausing [?]
+      ask if I can make a pausing filter, to start paused...hmm... [?]
+    bounty ffmpeg server for doze
+    bounty motion-detect, audio
+  try out disney digital copy
+  EDL's (if not lewd) (only after can do propaganda, really)
+    monty python
+    LOTR
+    forever strong
+    MOTAB music videos :P
+  refresh bug try to reproduce
+  if I don't use jruby complete is it faster?
   tweak web stuff/propaganda for upconverter filenames et al
   propaganda DVD upconverter
-  advertise virtual audio
-  advertise screen capture
+  propaganda virtual audio
+  propaganda screen capture
   report ffplay -t 6 doesn't stop bug
-    report ffplay -vol 1 doesn't work
   research XBMC
     break out into its own separate window?
     @XBMC: your stuff is broken in at least these two ways (bug reports)
@@ -201,8 +221,7 @@
   research VOB dirs
     still needs timesplits? what about in XBMC? what about XBMC with DVD's
   @avisynth: 6 fps from 2? huh? also why are some progs [ffmpeg] seemingly resetting the graphedit pins? huh? also tell that one already logged bug how I work around it...
-  virtual audio TS's line up
-  sout-keep for VLC EDL'ish?
+  sout-keep for VLC EDL'ish or playlist?
   local cache volume_name -> dvd id's, in case that can speed things up
   check if some create buttons are now outdated...
   make it load faster! this is cuh-razy! or at least splash screen!
@@ -233,7 +252,7 @@
   "save to" -> same dir as "get from" yeah
   figure out a way to get to the roku [ps3ms?], +- netflix edited to roku LOL
   prompt something if the EDL is actually "empty" :)
-  already donated: rubular, mplayer, vlc, menumeter, http://instantitunes.wordpress.com/2010/02/26/use-tsmuxer-on-snow-leopard, unlocker, mac fan thing, open office, many sourceforge tops, 7zip, VLC, mplayer, cannot donate: ffmpeg, http://ffmpeg.zeranoe.com/builds, tsmuxer (sent email), asked mulder, smplayer ($5), http://lame.buanzo.com.ar, sox, smcFanControl (eidac)
+  already donated: sox, rubular, mplayer, vlc, menumeter, http://instantitunes.wordpress.com/2010/02/26/use-tsmuxer-on-snow-leopard, unlocker, mac fan thing, open office, many sourceforge tops, 7zip, VLC, mplayer, cannot donate: ffmpeg, http://ffmpeg.zeranoe.com/builds, tsmuxer (sent email), asked mulder, smplayer ($5), http://lame.buanzo.com.ar, sox, smcFanControl (eidac)
   note "you want to optimize your reverse blitting first... :)"
   if 2 EDL's, still can choose a title trackfor automatically playing it right.
   fast preview: change it to be aggressive sometimes yes, other times no

data/VERSION

@@ -1 +1 @@
-0.26.2
+0.26.3

data/bin/sensible-cinema-cli

@@ -20,10 +20,12 @@ This file is part of Sensible Cinema.
 require File.dirname(__FILE__) + "/../lib/add_any_bundled_gems_to_load_path.rb"
 require 'sane'
 
-for file in ['overlayer', 'keyboard_input', 'screen_tracker', 'mouse', 'ocr', 'vlc_programmer', 'edl_parser', 'auto_window_finder', 'swing_helpers']
+for file in ['overlayer', 'keyboard_input', 'screen_tracker', 'ocr', 'vlc_programmer', 'edl_parser', 'auto_window_finder', 'jruby-swing-helpers/swing_helpers']
   require_relative '../lib/' + file
 end
 
+# shared
+require_relative '../lib/jruby-swing-helpers/mouse'
 
 def choose_file title, dir
   SwingHelpers::FileDialog.new_previously_existing_file_selector_and_go title, dir

data/change_log_with_feature_list.txt

@@ -1,6 +1,28 @@
+== 0.26.3 ==
+
+Make it so that it can modify timestamps from .srt files programmatically in order to ensure that time signatures match
+whichever player you're using now.
+
+Includes a means of coordinating across different time signatures based on start time and length.
+
+Had a git commit with an example pure JSON EDL, and several other mixed/mplayer compatible styles.
+
+NB that sensible cinema can use a directshow capture device and pin it to the location of a given window to be able to 
+stream/upconvert.
+
+NB that we have an optimized routine for taking screen captures so as to not
+use up as much cpu by sampling/doing them less frequently et all.
+
+Also includes work toward a directshow DVD player employing its API's
+(in separate project) for playback/skipping/muting/pausing et al.
+and basically progammatic control of a playing back DVD player.
+
+example EDL with a tenseness metric by level
+example EDL with a tenseness metric by age commits
+
 == 0.26.2 ==
 
-Fix license acceptance code, quick redeploy
+Fix license acceptance code (quick redeploy).
 
 == 0.26.1 ==
 

data/developer_how_to_contribute_to_the_project.txt

@@ -33,6 +33,8 @@ $ jruby bin/sensible-cinema --help # tells you all available options
   
 Feedback/contribution welcome :)
 
+Ex: want JSON'ified output somehow? Just ask.
+
 == mac os x developers ==
 
 You'll have to install a few dependencies.  It will tell you which at runtime, 

data/lib/edl_parser.rb

@@ -237,7 +237,8 @@ class EdlParser
     combined.compact
   end
   
-  # its reverse: translate_time_to_human_readable
+  # this one is 1:01:02.0 => 36692.0
+  # its reverse is: translate_time_to_human_readable
   def self.translate_string_to_seconds s
     # might actually already be a float, or int, depending on the yaml
     # int for 8 => 9 and also for 1:09 => 1:10
@@ -254,6 +255,8 @@ class EdlParser
      p 'failed!', s
      raise e
     end
+    raise unless seconds =~ /^\d+(|[,.]\d+)$/
+    seconds.gsub!(',', '.')
     total += seconds.to_f
     minutes = s.split(":")[-2] || "0"
     total += 60 * minutes.to_i

data/lib/eight_three.rb

@@ -16,8 +16,8 @@ DWORD WINAPI GetShortPathName(
   attach_function :path_to_8_3, :GetShortPathNameA, [:pointer, :pointer, :uint], :uint
   def self.convert_path_to_8_3 path
     out = FFI::MemoryPointer.new 256 # bytes
-    path_to_8_3(path, out, out.length)
-    out = out.get_string
+    path_to_8_3(path, out, out.size)
+    out = out.get_string 0
     if out.size == 0
       raise 'unable to convert--probably file does not exist yet ' + path
     else

data/lib/gui/sensible-cinema-base.rb

@@ -31,10 +31,12 @@ class String
   end
 end
 
+# shared
 require_relative '../jruby-swing-helpers/swing_helpers'
+require_relative '../jruby-swing-helpers/drive_info'
 
 # attempt to load on demand...i.e. faster...
-for kls in [:MencoderWrapper, :MplayerEdl, :PlayAudio, :SubtitleProfanityFinder, :ConvertThirtyFps, :RubyClip, :DriveInfo]
+for kls in [:MencoderWrapper, :MplayerEdl, :PlayAudio, :SubtitleProfanityFinder, :ConvertThirtyFps, :RubyClip]
   autoload kls, "./lib/#{kls.to_s.snake_case}"
 end
 
@@ -544,28 +546,32 @@ module SensibleSwing
       end
     end
     
-    def open_file_to_edit_it filename, start_minimized=false
+    def open_file_to_edit_it filename, options = {} # :start_minimized
       if OS.windows?
-        if start_minimized
+        if options[:start_minimized]
           system_non_blocking "start /min notepad \"#{filename}\""
         else
           system_non_blocking "notepad \"#{filename}\""
         end
       else
+        # ignore minimized :P
         system_non_blocking "open -a TextEdit \"#{filename}\""
       end
     end
     
     def new_nonexisting_filechooser_and_go title = nil, default_dir = nil, default_file = nil
+      bring_to_front
       JFileChooser.new_nonexisting_filechooser_and_go title, default_dir, default_file
     end
 
     def show_blocking_message_dialog(message, title = message.split("\n")[0], style= JOptionPane::INFORMATION_MESSAGE)
-      SwingHelpers.show_blockiing_message_dialog message, title, style
+      bring_to_front
+      SwingHelpers.show_blocking_message_dialog message, title, style
     end
     
     # call dispose on this to close it if it hasn't been canceled yet...
     def show_non_blocking_message_dialog message, close_button_text = 'Close'
+      bring_to_front
       # lodo NonBlockingDialog it can get to the top instead of being so buried...
       NonBlockingDialog.new(message, close_button_text)
     end
@@ -573,17 +579,20 @@ module SensibleSwing
     include_class javax.swing.UIManager
     
     def get_user_input(message, default = '', cancel_ok = false)
+      bring_to_front
       SwingHelpers.get_user_input message, default, cancel_ok
     end
     
     def show_copy_pastable_string(message, value)
+      bring_to_front
       RubyClip.set_clipboard value            
       get_user_input message + " (has been copied to clipboard)", value, true
     end
     
     # also caches directory previously selected ...
     def new_existing_file_selector_and_select_file title, dir = nil
-      dir ||= LocalStorage[caller.inspect] = File.dirname(got)
+      bring_to_front
+      dir ||= LocalStorage[caller.inspect]
       got = FileDialog.new_previously_existing_file_selector_and_go title, dir
       LocalStorage[caller.inspect] = File.dirname(got)
       got

data/lib/gui/sensible-cinema-create.rb

@@ -56,27 +56,42 @@ module SensibleSwing
       
       @parse_srt = new_jbutton("Scan a subtitle file (.srt) to detect profanity times automatically" )
       @parse_srt.tool_tip = <<-EOL
-        You can download a .srt file and use it to automatically search for profanities.
-        Basically download it from opensubtitles.org (possibly from other sites, too),
-        (enter dvd name in the search box, click a result, click one from the list with an English flag, then click 'Download(zip)')
-        Once you download the zip, unzip it, and then compare the timestamps in it with those on the DVD (see the button "Watch DVD unedited (realtime mplayer)")
-        NB that you'll first want/need to *carefully* double check your subtitle
-        file with the actual DVD.  (ex: smplayer, hit the 'o' button to display
-        the current timestamp, then go to the end, stop at some point with text
-        and hit the '.' key until a subtitle very first displays.
-        Next convert that number to 29.97 fps (using the button given).
-        The numbers should match precisely.  If they don't, edit this file
-        so that it will have some offsets given.
+        You can download a .srt file and use it to programmatically search for the location of various profanities.
+        Basically download it from opensubtitles.org (or engsub.net et al),
+        (for opensubtitles.org enter dvd title in the search box, click on a result, click one from the list with an English flag, 
+        then choose 'Download(zip)', then unzip the file)
+        NB that you'll want/need to *carefully* double check your subtitle file for accuracy. Here's how:
+        Now carefully compare a beginning timestamp in it with the actual words in the .srt file 
+        with the actual DVD.
+        (see the button "Watch DVD unedited (realtime mplayer)")
+        (smplayer can kind of do it, too, play it, hit the 'o' button to display
+        the OSD timestamp, then go to just before the verbiage, 
+        and hit the '.' key until a subtitle very first appears.
+        Next convert that number to 29.97 fps (using the button for that).
       EOL
 
       @parse_srt.on_clicked do
-        filename = new_existing_file_selector_and_select_file("Pick srt file to scan for profanity:")
-        add_to_beginning = get_user_input("How much time to subtract from the beginning of each subtitle entry (ex: 1:00 -> 1:01 becomes 0:59 -> 1:01)", "0.0")
-        add_to_end = get_user_input("How much time to add to the end of each subtitle entry (ex: 1:00 -> 1:01 becomes 1:00 -> 1:02)", "0.0")
-        parsed = SubtitleProfanityFinder.edl_output filename, {}, add_to_beginning.to_f, add_to_end.to_f # flight: TODO necessary typically ??? 0.35, 0.25 
-        File.write(EdlTempFile, "# add these into your mute section if you deem them mute-worthy\n" + parsed)
-        open_file_to_edit_it filename, true
-        sleep 0.3 if OS.mac? # add delay...
+        srt_filename = new_existing_file_selector_and_select_file("Pick srt file to scan for profanity:")
+        # TODO nuke
+        add_to_beginning = "0.0"#get_user_input("How much time to subtract from the beginning of every subtitle entry (ex: (1:00,1:01) becomes (0:59,1:01))", "0.0")
+        add_to_end = "0.0"#get_user_input("How much time to add to the end of every subtitle entry (ex: (1:00,1:04) becomes (1:00,1:05))", "0.0")
+        
+        @show_parse_instructions ||= show_blocking_message_dialog "Now enter some values which will adjust the .srt time signatures\nso that it matches the movie more precisely.\nWe also need these values to coordinate it with other devices that may have different timestamps.\nFor DVD playback this will be the timestamps mplayer shows within the console window [the mplayer unedited button]."
+        open_file_to_edit_it srt_filename
+        sleep 0.5 # let it open first
+        bring_to_front
+        start_text = get_user_input("enter the text from any subtitle entry near beginning [like \"Hello, welcome to our movie.\"]", "...")
+        start_srt = get_user_input("enter beginning timestamp within the .srt file #{File.basename(srt_filename)[0..10]}... for \"#{start_text}\"", "00:00:00,000")
+        start_movie_ts = get_user_input("enter beginning timestamp within the movie itself for said text", "0:00:00")
+        
+        end_text = get_user_input("enter the text from a subtitle entry far within or near the end of the movie", "...")
+        end_srt = get_user_input("enter the beginning timestamps within the .srt for \"#{end_text}\"", "02:30:00,000")
+        end_movie_ts  = get_user_input("enter beginning timestamps within the movie itself for \"#{end_text}\"", "2:30:00.0 or 9000.0")
+        
+        parsed_profanities = SubtitleProfanityFinder.edl_output srt_filename, {}, add_to_beginning.to_f, add_to_end.to_f, start_srt, start_movie_ts, end_srt, end_movie_ts
+        File.write EdlTempFile, "# add these into your mute section if you deem them mute-worthy\n" + parsed_profanities +
+          %!\n\n#Also add these two lines for later coordination:\n"beginning_subtitle" => ["#{start_text}", "#{start_movie_ts}"],! +
+           %!\n"ending_subtitle_entry" => ["#{end_text}", "#{end_movie_ts}"]!
         open_file_to_edit_it EdlTempFile
       end
 
@@ -96,7 +111,7 @@ module SensibleSwing
         id_string
       }
 
-      @convert_seconds_to_ts = new_jbutton( "Convert 3600 <-> 1:00:00 style timestamps" )
+      @convert_seconds_to_ts = new_jbutton( "Convert 3600.0 <-> 1:00:00 style timestamps" )
       @convert_seconds_to_ts.on_clicked {
         input = get_user_input("Enter \"from\" timestamps, like 3600 or 1:40:00:", "1:00:00.1 or 3600.1")
         while(input)
@@ -248,7 +263,13 @@ module SensibleSwing
       else
         file_from = get_grabbed_equivalent_filename_once dvd_friendly_name, dvd_title_track # we don't even care about the drive letter anymore...
       end
-      show_blocking_message_dialog("warning: file #{file_from} is not a .mpg or .ts file--it may not work properly all the way--if it's mkv and fails consider first converting to ts by using tsmuxer.") unless file_from =~ /\.(ts|mpg|mpeg)$/i
+      if file_from =~ /\.mkv/i
+        show_blocking_message_dialog "warning .mkv files from makemkv have been known to be off timing wise, please convert to a .ts file using tsmuxer first if it came from makemkv"
+      end
+      if file_from !~ /\.(ts|mpg|mpeg)$/i
+        show_blocking_message_dialog("warning: file #{file_from} is not a .mpg or .ts file--it may not work properly all the way, but we'll try...") 
+      end
+      
       save_to_edited = get_save_to_filename dvd_friendly_name
       fulli = MencoderWrapper.calculate_fulli_filename save_to_edited
       if exit_early_if_fulli_exists
@@ -301,4 +322,4 @@ module SensibleSwing
     
     
   end
-end
+end