maemo.org - Talk

maemo.org - Talk (https://talk.maemo.org/index.php)
-   Multimedia (https://talk.maemo.org/forumdisplay.php?f=32)
-   -   [ANNOUNCE] Alpha release of Open Media Player (https://talk.maemo.org/showthread.php?t=72156)

sixwheeledbeast 2012-08-09 20:25

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

Originally Posted by gidzzz (Post 1249516)
May I know how you came to such conclusion? I see no difference when comparing two simultaneously running instances of OMP in htop. I even compared with an ancient version from before OMP entered repositories, but I still don't know what you mean.

I was getting 18% RAM for OMP, Pulseaudio was bouncing around 50% CPU and dbus-wrapper 30% CPU. These were higher values than I normal had.
I was comparing the two versions with top on separate N900's.

I have just been testing different versions and a reboot has fixed it. Sorry :o, this looks like it may have been my device, as updating the other N900 didn't cause this problem.

Quote:

Originally Posted by gidzzz (Post 1249516)
I assume that you're speaking of buttons which actually do something (otherwise I don't think it's a problem with OMP). I will disable key repeating for some shortcuts.

If it helps CSSU hildon-desktop had a similar issue before it was updated. Disabling key-repeat sounds like a good solution.

Quote:

Originally Posted by gidzzz (Post 1249516)
Hmmm... I don't remember listing any shortcuts for that. If you press Shift+Enter, you should get a keyboard-friendly version of that menu.

Keyboard friendly version is great.

Thanks again.

Estel 2012-08-11 02:51

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

Originally Posted by Skry (Post 1248486)
About the volume keys as someone suggested to make them skip track.. So, press = +- volume and hold = next/prev track, how about it?

Nice job so far!

No, thanks. I would like to keep posibility of adjusting volume quickly, without "abusing" my volume buttons by fast-clicking.

Quote:

Originally Posted by the_envy (Post 1248518)
it can be even simplier: volume rocker is used for track skipping only when the phone is locked and it's working as it is when the phone is in normal unlocked mode. So for example, you can skip tracks in your pocket whithout even pulling out n900 and if you want to change the volume you can open the keyboard to unlock it (or lock switch) and then use volume keys for their regular use.

What kind of idea is that? 0_o

Why not going the opposite, and unlocking Your device for skipping tracks, manually? Adjusting volume when device is locked is *much* more appreciated.

/Estel

HELLASISGREECE 2012-08-11 03:59

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

Originally Posted by Estel (Post 1250273)
Adjusting volume when device is locked is *much* more appreciated.

/Estel

Indeed!!

It my only grudge at my N91 8GB. I wish it had that "feature"

Skry 2012-08-11 04:43

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

Originally Posted by Estel (Post 1250273)
No, thanks. I would like to keep posibility of adjusting volume quickly, without "abusing" my volume buttons by fast-clicking.

Erm, that could and should be an option me thinks. I don't see why we could not have the option to configure how those buttons function. I never hold those buttons when I adjust volume so could use a feature like this. Besides, buttons are made for abusing :)

D@vIcHoJD 2012-08-11 04:51

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

Originally Posted by Skry (Post 1250292)
Erm, that could and should be an option me thinks. I don't see why we could not have the option to configure how those buttons function. I never hold those buttons when I adjust volume so could use a feature like this. Besides, buttons are made for abusing :)

Me Too. I also think it would be an excellent option to change track with the volume buttons.:D

Regards.

rotoflex 2012-08-12 02:44

Re: [ANNOUNCE] Alpha release of Open Media Player
 
qwerty's last app he wrote for the N900 was an app to control volume & skipping tracks of the media player when the screen was locked, & as I recall the button functionality was high-function, well thought-out, & very straightforward. Maybe something like rocker switch for volume, & camera button to skip to next track: I can't remember the name of the app.

Edit: Here it is, his last post: Extended Locked Media Player Control. Quite a loss that he left, but understandable how he got so p**** off by Nokia.

http://talk.maemo.org/showthread.php?t=59700

Quote:

When locked, fully pressing the camera button in will pause/play the track; the volume keys will function as normal but if you keep the camera button half-pressed in, the volume keys will now go to the previous/next track.

D@vIcHoJD 2012-08-12 03:40

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

Originally Posted by rotoflex (Post 1250710)
qwerty's last app he wrote for the N900 was an app to control volume & skipping tracks of the media player when the screen was locked, & as I recall the button functionality was high-function, well thought-out, & very straightforward. Maybe something like rocker switch for volume, & camera button to skip to next track: I can't remember the name of the app.

Edit: Here it is, his last post: Extended Locked Media Player Control. Quite a loss that he left, but understandable how he got so p**** off by Nokia.

http://talk.maemo.org/showthread.php?t=59700

Do not work, I get some errors: (failedin file dbus-connection.c line 5257; filed in file dbus-connection.c line 2657 and Failed to init MAFW: Discovery initialization failed: error).

romanianusa 2012-08-14 21:43

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Hmmm...great app but can you make the photo album big like the Instinctiv (only portray)?
Also lyric not working?

gidzzz 2012-08-15 15:22

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Make sure that you have something enabled in Settings -> Lyrics providers.



New update is out. It brings new plugins, speed-ups, fixes for bugs and crashes, and other adjustments. The changelog is huge, read on if you dare.
  • Support for offline lyrics providers and additional provider description.
  • Added plugins to use lyrics from MaeLyrica and Sonata.
  • Significantly improved overall speed of loading a playlist in the playlist browser.
  • Much faster saving of playlists after editing.
  • Saved playlists load adaptively, prioritizing currently visible items.
  • Improved way of lading saved playlists and current playlist makes saving, sharing, shuffling, etc. immediately work correctly.
  • Group operations on automatic and imported playlists wait for loading to finish before taking effect.
  • Improved responsiveness while searching in a playlist, artist's albums, a single album and a genre.
  • Massively improved speed of adding a saved playlist to now playing from context menu.
  • Opening video window from the playback indicator does not start playback.
  • Eliminated delay in controlling playback with "Prevent playback interruptions" enabled.
  • Fixed problems with video resuming and thumbnails when "Prevent playback interruptions" is enabled.
  • Fixed automatic playlists showing 1 song if they are empty.
  • Shuffle buttons can be highlighted when navigating with keyboard.
  • Fixed a bug with not all songs being added from "Never played" to now playing when using context menu.
  • Reworked album art selection dialog and made it resize itself properly when rotating.
  • Invoking context menu in Now Playing with a shortcut when song progress is visible results in album art's menu.
  • Disabled autorepeat for shortcuts where it was not desirable.
  • Fixed S, E, R shortcuts not working in Now Playing when playlist is visible.
  • Selecting a song from the playlist in Now Playing window starts playing it also if playback was paused.
  • Fixed a crash caused by opening Now Playing, quickly closing the underlying window while it is still visible and then trying to open Now Playing again.
  • Fixed a rare crash when changing a track in an unfortunate moment.
  • Updated translations.

the_envy 2012-08-15 20:16

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Is there any way to exclude some folders from adding in OMP the same way it skips "Ringtones" folder?

michaaa62 2012-08-15 21:07

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Tracker-cfg is a gui for editing the tracker config file /home/user/.config/tracker.cfg

romanianusa 2012-08-15 22:39

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Can you make so the lyrics show up in "Entertainment view"? I like that view better.

Also is there a way you can make so you can scroll the album art like they did on iPHONE?

tiempjuuh 2012-08-16 05:29

Re: [ANNOUNCE] Alpha release of Open Media Player
 
The program itself is brilliant. But the Dutch translation is poor. Is there any way to translate it?

pali 2012-08-16 09:25

Re: [ANNOUNCE] Alpha release of Open Media Player
 
@gidzzz:

I spent a lot of time to bring proper wired headset button support. I found that problem is in binary prolog file policy.plc. Now after long time I was able to decompile it to source files, fix support for button and compile again.

But file policy.plc is part of system package policy-settings-rx51, the only way how to distribute it is via CSSU. Decompiled source code of this package is now in gitorious and new fixed deb package is in cssu-devel:

http://gitorious.org/community-ssu/policy-settings-rx51
http://maemo.merlin1991.at/cssu/comm...settings-rx51/

When policy-settings-rx51 is updated and device reboot headset button working automatically then headset (with microphone) is inserted. No other hacks, code is needed.

I created also patch for OMP which only remove old (hack) code and show information that for headset button is needed CSSU.

Merge request is here:
http://gitorious.org/qt-mediaplayer/...rge_requests/7

tiempjuuh 2012-08-16 12:21

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Is it possible to supply the correctly translated .ts file to implement it to OMP?

gidzzz 2012-08-16 13:50

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

Originally Posted by pali (Post 1252505)
When policy-settings-rx51 is updated and device reboot headset button working automatically then headset (with microphone) is inserted. No other hacks, code is needed.

I created also patch for OMP which only remove old (hack) code and show information that for headset button is needed CSSU.

Thanks, the button works like a charm with new policy-settings-rx51, but I don't quite get this patch. All I can see it does is it changes the note and removes support for pause/resume on disconnection/connection of wired headsets. Isn't adding 'or CSSU(-devel) installed' to 'Requires "Prevent playback interruptions" disabled' all that has to (and can) be changed on OMP's side?



Quote:

Originally Posted by tiempjuuh (Post 1252581)
Is it possible to supply the correctly translated .ts file to implement it to OMP?

I'm not sure what you mean by "implement". If you want to make OMP use your translation, you should generate a .qm file from .ts and replace "/opt/openmediaplayer/lang/openmediaplayer.nl.qm" (or name it "openmediaplayer.nl_tj.qm" and use "nl_tj" language code in the settings, this way it won't be overwritten during an update). You can use Qt Linguist for that, or attach .ts here and I'll do it for you.

If you want you translation to be packaged with OMP, Transifex is the way to go. However, ade always seemed to do an excellent job with the translation (at least when it comes to keeping it up-to-date, I am not able to judge other aspects), so I'd also like to hear something from him.

ade 2012-08-16 15:25

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

Originally Posted by gidzzz (Post 1252619)
If you want you translation to be packaged with OMP, Transifex is the way to go. However, ade always seemed to do an excellent job with the translation (at least when it comes to keeping it up-to-date, I am not able to judge other aspects), so I'd also like to hear something from him.

I was somewhat surprised by the judgement "But the Dutch translation is poor". Seems like an oversimplified reaction and not a solid base for cooperation. Of course there is always room for improvement, and I don't know what university degree in Dutch language tiempjuuh has :rolleyes:
He has now asked if he can join the translation team. If he can convince me he can do meaningful modifications (some suggestions in PM for example) he is welcome if you ask me. Just the label "poor" is no convincing argument.

pali 2012-08-16 15:47

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

Originally Posted by gidzzz (Post 1252619)
Thanks, the button works like a charm with new policy-settings-rx51, but I don't quite get this patch. All I can see it does is it changes the note and removes support for pause/resume on disconnection/connection of wired headsets. Isn't adding 'or CSSU(-devel) installed' to 'Requires "Prevent playback interruptions" disabled' all that has to (and can) be changed on OMP's side?

My first code for headset button in OMP working only if there is no running libplayback application. It turning on/off jackbias via amixer.

But with new patched policy.plc old amixer code in OMP can break policy.plc and nothing will work.

I really suggest to remove old hack code. In merge request is only deleted function updateWiredHeadset which call amixer, nothing more.

If somebody want headset button working it really needs CSSU update. CSSU is right place to update system packages... Also only today was policy-settings-rx51 pushed to cssu-devel but in next days it will be moved to cssu-testing. So merge request can wait until package will be in cssu testing or stable.

romanianusa 2012-08-16 16:51

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Nice player but there are several things i want to point out:

1. The Entertainment View should have the option to be set as default view. That view also should designated as the Car View. The Car View you provided is not attractive...sorry i like to see album arts to identify my song.
2. The art album for the Entertainment View should take up the entire screen on the left side since it's fullscreen. Do not allow it to dance around with the art and scroll lists...it is distracting. lol
3. The Entertainment View should also have lyrics

Note: Playlist isn't working properly

The Entertainment View is the ONLY view that attract me to this prog...otherwise i am going back to Instinctiv. Thx for your time.

sixwheeledbeast 2012-08-16 17:19

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

Originally Posted by romanianusa (Post 1252678)
1. The Entertainment View should have the option to be set as default view. That view also should designated as the Car View. The Car View you provided is not attractive...sorry i like to see album arts to identify my song.

Car view should be basic with big buttons, IMO it's well laid out and see no point in adding any more features.

Quote:

Originally Posted by romanianusa (Post 1252678)
Note: Playlist isn't working properly

Why? This isn't helpful feedback!

gidzzz 2012-08-16 17:49

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

Originally Posted by pali (Post 1252650)
I really suggest to remove old hack code. In merge request is only deleted function updateWiredHeadset which call amixer, nothing more.

I'm OK with removing the hacky code, but all the time I have a feeling that you don't realize one thing. updateWiredHeadset() is indeed responsible for setting jackbias with amixer, but it also takes care of pausing music when you pull the headphone plug out. Removing this means that Disconnect headset to pause and Reconnect headset to resume will work only for Bluetooth headsets.

Aren't QProcess::startDetached("amixer", args << "on"); and its "off" counterpart the only things that really have to be removed?



Quote:

Originally Posted by romanianusa (Post 1252678)
The Entertainment View is the ONLY view that attract me to this prog...otherwise i am going back to Instinctiv. Thx for your time.

I consider Entertainment View merely a small addition to OMP, so it doesn't get updates frequently. I think you should go back to Instinctiv, but you can keep checking now and then if anything has changed.

romanianusa 2012-08-16 18:06

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

Originally Posted by sixwheeledbeast (Post 1252689)
Car view should be basic with big buttons, IMO it's well laid out and see no point in adding any more features.


Why? This isn't helpful feedback!

That's your opinion but i think Entertainment View is perfect for car view. I wish there is an option to make it default so everyone can have their own choices. I am not complaining..just expressing my opinion...that if i were to create my own program...that is what i will do.

In regard to playlist. When i saved playlist and proceed to play it...it won't play. However, if i hit the shuffle button under playlist, it work. And if i go back to playlist and select a song, it work and then sometimes it's not working again. Not sure what the bug is...

gidzzz 2012-08-16 18:18

Re: [ANNOUNCE] Alpha release of Open Media Player
 
I see, there's one line missing in the code used for browsing saved playlists. It's quite important, so I'll prepare a quick update as soon as I have a free moment.

Edit: It's ready for downloading. No other changes than dealing with the aforementioned issue.

romanianusa 2012-08-17 14:08

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

Originally Posted by gidzzz (Post 1252706)
I see, there's one line missing in the code used for browsing saved playlists. It's quite important, so I'll prepare a quick update as soon as I have a free moment.

Edit: It's ready for downloading. No other changes than dealing with the aforementioned issue.

Great!

Here's my wishlist:

1. Add fullscreen mode in option for default view.
2. Add default view for Entertainment View as option.
2. Repeat mode for single song
3. Optimize prog to save battery
4. Seek slider in Entertainment View need fix.
5. Album Arts size increase
6. Portrait mode a la Instinctiv

HELLASISGREECE 2012-08-17 14:20

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

Originally Posted by romanianusa (Post 1252959)
Great!

Here's my wishlist:

1. Add fullscreen mode in option for default view.
5. Album Arts size increase

Those would be awesome aditions, yes : )

pali 2012-08-17 14:29

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

Originally Posted by gidzzz (Post 1252699)
I'm OK with removing the hacky code, but all the time I have a feeling that you don't realize one thing. updateWiredHeadset() is indeed responsible for setting jackbias with amixer, but it also takes care of pausing music when you pull the headphone plug out. Removing this means that Disconnect headset to pause and Reconnect headset to resume will work only for Bluetooth headsets.

Aren't QProcess::startDetached("amixer", args << "on"); and its "off" counterpart the only things that really have to be removed?

Sorry, I totally forgot that updateWiredHeadset also handle pause on disconnect. My fault...

I updated merge request, see: https://gitorious.org/qt-mediaplayer...rge_requests/7

tiempjuuh 2012-08-17 20:36

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Ade, you're absolutely right. Poor isn't the right word. The translations aren't poor, really I'm glad someone has done the job to translate it! However, some translations could be improved, so I'd like to 'review' (I know, again the wrong word, but atm I don't know a synonym :P) the translations and improve them. :)

ade 2012-08-17 21:31

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

Originally Posted by tiempjuuh (Post 1253077)
Ade, you're absolutely right. Poor isn't the right word. The translations aren't poor, really I'm glad someone has done the job to translate it! However, some translations could be improved, so I'd like to 'review' (I know, again the wrong word, but atm I don't know a synonym :P) the translations and improve them. :)

This sound like a more balanced judgement. I'll grant you access to the dutch translation team.

anthonie 2012-08-19 17:34

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Rapidly OMP has replaced the default player for me. I do have a request though...

Is there any way to move the lyrics files to a more mount-accessible location, somewhere under MyDocs, so they can be easily retreived and added when connecting the N900 to a PC? It would make sharing those lyrics via BlueTooth a lot easier also. Android carrying friends love the lyrics function but raise their eyebrows when it takes a while to get them the lyrics too.

sixwheeledbeast 2012-08-19 19:46

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

Originally Posted by anthonie (Post 1253557)
Rapidly OMP has replaced the default player for me. I do have a request though...

Is there any way to move the lyrics files to a more mount-accessible location, somewhere under MyDocs, so they can be easily retreived and added when connecting the N900 to a PC? It would make sharing those lyrics via BlueTooth a lot easier also. Android carrying friends love the lyrics function but raise their eyebrows when it takes a while to get them the lyrics too.

MyDocs isn't a great place to put this, also with it being a FAT partition you can't symlink there either.

Rather than move the file (I think the current file location is best, even if it is hard to get to) a good solution would be to add the share feature in the the lyrics box.

Hold down on the lyrics and the options could be :-
Edit Lyrics, Reload Lyrics, Share.

You could then share the text file like you can a song.

anthonie 2012-08-20 10:47

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

Originally Posted by sixwheeledbeast (Post 1253592)
MyDocs isn't a great place to put this, also with it being a FAT partition you can't symlink there either.

I know. The symlink is out of the question, but I was thinking more of a configuration option, allowing you to choose, even if it is not ideal, a fat32 partition, or whatever else you'd like.

Quote:

Rather than move the file (I think the current file location is best, even if it is hard to get to) a good solution would be to add the share feature in the the lyrics box.

Hold down on the lyrics and the options could be :-
Edit Lyrics, Reload Lyrics, Share.

You could then share the text file like you can a song.
I do like the option described, but it does have a major disadvantage, very much like other sharing options on the n900 have that as well: One can only share one file at a time. A hideous task if someone wants a lot of files from you...

Right now, I share lyrics by tarring up a complete subdir from the OMP lyrics directory, mv that to MyDocs and then share it over BlueTooth. From that procedure I came up to my above request. I guess a temp copy from the current directory to a MyDocs location would be better than a move action.

michaaa62 2012-08-20 11:43

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

MyDocs isn't a great place to put this, also with it being a FAT partition you can't symlink there either.
That is not the whole truth. Fat32 is not supporting symlinks ...
BUT you could link to a file or folder on that FAT32 partition from your ext3-partition!

som3a 2012-08-20 17:26

Re: [ANNOUNCE] Alpha release of Open Media Player
 
hi there ...

i have a little strange problem .. i was using OMP and it was all great before i did the last apt-get update, apt-get upgrade .. so the lyrics layout goes ! i didn't find it i didn't know what the problem !?!

and i have a request ,, is it possible to add an repeat single song option to this app ? it will be amazing for example if i clicked the repeat button twice it goes to single song repeat !

tcbl50 2012-08-20 18:06

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

Originally Posted by som3a (Post 1253835)
hi there ...


and i have a request ,, is it possible to add an repeat single song option to this app ? it will be amazing for example if i clicked the repeat button twice it goes to single song repeat !

was just going to ask the same thing!! its the only reason why i use mediabox sometimes!

anthonie 2012-08-20 18:06

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Quote:

Originally Posted by som3a (Post 1253835)
hi there ...

i have a little strange problem .. i was using OMP and it was all great before i did the last apt-get update, apt-get upgrade .. so the lyrics layout goes ! i didn't find it i didn't know what the problem !?!

Provided you have lyrics enabled in your settings, they can be found by clicking the album art (or the spot where they should be if you have none) of the current song twice.

som3a 2012-08-20 18:19

Re: [ANNOUNCE] Alpha release of Open Media Player
 
oh thanks @anthonie .. i was just forget that !! :)

@tcbl50 .. it's a good option to add to this app ,,

tcbl50 2012-08-21 15:41

Re: [ANNOUNCE] Alpha release of Open Media Player
 
just wanted to ssk if its possible to add crossfading of tracks!

anthonie 2012-08-21 16:33

Re: [ANNOUNCE] Alpha release of Open Media Player
 
By the way, anyone else having trouble getting sound from the FM radio to work, after installing Thumb2?

It used to work fine on both machines but stopped working after changing KP50 for Thumb2. No error messages or anything, just no sound.

Estel 2012-08-21 16:35

Re: [ANNOUNCE] Alpha release of Open Media Player
 
It works for me without problems using thumb, but it's not related to Open Media Player - at all (neither FM Radio, nor thumb), so please, stop off-topic.

anthonie 2012-08-21 16:45

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Sorry, wrong topic indeed!


All times are GMT. The time now is 23:25.

vBulletin® Version 3.8.8