Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make tick sound more stable using background long audio playing #106

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hwiorn
Copy link

@hwiorn hwiorn commented Mar 30, 2023

org-pomodoro play tick sound every second using run-with-timer and sometimes it makes tick sound lag and no mode-line updates a little while.

The tick sound is important to focus more in a pomodoro session. However, the dragging bit tempo sound makes quite distraction instead of focus to a task. To me, Really, this made me hesitate to use org-pomodoro for a long time. So I used web pomodoro timer such as pomofocus.

I generate the tick.wav to a 10 seconds MP3 file(tick_long.mp3). And I added the background playing code using mpv, mplayer, ffplay (ffmpeg) or play (sox).

(use-package org-pomodoro
  :custom
  (org-pomodoro-ticking-sound-long t)
  (org-pomodoro-ticking-sound-p t)
  (org-pomodoro-keep-killed-pomodoro-time t))

@hwiorn
Copy link
Author

hwiorn commented Mar 30, 2023

I think 60 seconds tick would be better performance for low-end devices.
Because other pomodoro apps and web services use long clock audio file for same reason.
But this size is quite big which is 938 KiB and 10 seconds audio doesn't have lagging issue currently.

  • 0.2 seconds tick sound WAV(original): 37 KiB
  • 0.2 seconds tick sound MP3: 4.1 KiB
  • 10 seconds tick sound WAV: 1.7 MiB
  • 10 seconds tick sound MP3: 157 KiB
  • 60 seconds tick sound WAV: 11 MiB
  • 60 seconds tick sound MP3: 938 KiB

@hwiorn hwiorn mentioned this pull request Mar 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant