mpv-scripts icon indicating copy to clipboard operation
mpv-scripts copied to clipboard

SmartSkip: Additional optional features

Open Eisa01 opened this issue 1 year ago • 5 comments

Some ideas that might be good to develop for SmartSkip?

  • [x] #93
  • [x] #94
  • [x] #95
  • [x] #96
  • [x] #98
  • [x] #99
  • [x] feat: Make keybinds configurable in user config
  • [x] feat: notification only for autoskip without forcing it with an option to press to toggle it
  • [ ] feat: Make skip_to_end_behavior universal, e.g.: reach_end_behavior, seek_end_behavior.
  • [ ] feat: Customizable skip silence settings within mpv.
  • [ ] feat: Add next-file inside this script.
  • [ ] feat: Make playlist sortable
  • [ ] feat: Add option to change next_playlist_last_behavior
  • [ ] feat: Add option to change next_chapter_last_behavior
  • [ ] feat: write_chapter to also include delete when chapters are 0.
  • [ ] feat: option for chapter at silence skip initiation point of successful (when chapters count is more than 1?).
  • [ ] feat: Add optional timepos for first_prev_chapter_time, to change it from 1 second to more if needed.
  • [ ] feat: write chapter to compare previous and old chapters before auto saving.

Eisa01 avatar Sep 06 '23 21:09 Eisa01

Could we also had a toggle for enabling or disabling the video when skipping to silence.

The reason for this is because i have a script that activates when there are audio tracks only, but it also activates when skipping to silence. Minor annoyance would be awesome if this is added!

NaiveInvestigator avatar Jan 12 '24 06:01 NaiveInvestigator

@NaiveInvestigator The only reason i am not providing it as an option as it causes problems when the video is on and silence detection is initiated.

I believe a better approach is to have that other script utilise file extension for detection of audio as well.

Another thing I can do is provide a script message that can be integrated with other scripts so that when silence-skip is triggered they can consider it in their scripts.

Eisa01 avatar Jan 12 '24 08:01 Eisa01

ah sure, it seems the effort is not worth it ig. ty for the considering it though! It is just a minor annoyance so doesnt matter too much.

NaiveInvestigator avatar Jan 12 '24 15:01 NaiveInvestigator

ah sure, it seems the effort is not worth it ig. ty for the considering it though! It is just a minor annoyance so doesnt matter too much.

NaiveInvestigator avatar Jan 12 '24 15:01 NaiveInvestigator