Open mvaranda opened 1 year ago
Thanks, I'll take a closer look when I have time, maybe next weekend.
Sorry about the delay, I now had time to try this. Unfortunately playback freezes for me, not related your code, the same for master. Probably some GStreamer upgrades that I need to sort out. For now, I can only offer some very initial impressions:
Gtk.Settings
?Hi Osmo, following my comments:
Creating the initial cache takes a long time – it needs to be in some way opt-in, so that users don't get an annoying suprise delay. Not sure what the best way to do this is – maybe a button in the video toolbar to generate and show the waveform?
The mouse UI probably needs changing cursors, otherwise it's undiscoverable.
Dark vs. light theme should be automatic, there's some initialization code in applicationman, after which you can probably use Gtk.Settings?
The waveforms I see look very "flat" and uninformative. Did you look into if some preprocessing helps when generating the cache data? Such as selecting a particular channel, limiting to particular frequencies, some kind of normalization, clipping, non-linear scale in the visualization etc.
Thanks for creating this app as well as for your help. I understand that the present code may have some not so elegant integration but it seems to be doing the work. I also understand that both of us may not have too much time to spend in this project as we both have a busy life (an it is almost summer). I believe that having this code merged to "audio-waveform" branch would make possible to other developers and even smart users to try using these features and provide some feedback. Thanks once again, Marcelo