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)

MohammadAG 2011-09-15 19:26

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

Originally Posted by sakya (Post 1089127)
I also noticed a higher cpu usage than the stock media player.
Play a track, stay in the now playing window, execute top: the mediaplayer uses 7-10% of the cpu (the stock media player is at 2-3%).
Leaving the player on the list window drops the % usage of the cpu to 3-4%

I don't know if this is the cause (I didn't make any test):
Code:

connect(positionTimer, SIGNAL(timeout()), mafwrenderer, SLOT(getPosition()));
Tha player is asking for the current time every second. It would be probably better to just add one second to the current position on the timer timeout and ask the renderer for the current position only when the state changes (play/pause/stop).

For the music window performance: I think that using a ListView/TableView and QAbstractListModel/QAbstractTableModel would be a lot faster than a QListWidget.

Just my two cents. ;)

Other than that, many thanks for all the work and efforts in developing this player.

The stock player does this too, but it suspends updates when you're not on the task switcher and the media player is not the focused window.
The actual CPU usage is probably from the progress bar, rather than the updates themselves.

I've never been successful with models, that's why I don't use them.

sakya 2011-09-15 19:48

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

Originally Posted by MohammadAG (Post 1089141)
The actual CPU usage is probably from the progress bar, rather than the updates themselves.

Do you mean the QSlider? I noticed that Qt uses a high cpu for QProgressbar (never understood/investigated why) but probably I never used a slider (on Maemo, on a pc obviously the "high" load isn't noticeable.

robin92 2011-09-15 21:36

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

Originally Posted by gidzzz (Post 1088936)
That's strange, does it happen all the time and is it only OMP that is unresponsive? Although I didn't experience anything like that, it's possible that it's been accidentally fixed, as there were lots of changes under the hood since the last build. ;)

I wanted to finish up a few things before posting this, but that report of unresponsiveness keeps me wondering if it still occurs. Here goes the simplified changelog (full is always here):
  • Indicator in the bottom right hides when the playlist is empty (but from time to time it appears when it shouldn't, overlapping the searchbox).
  • Instant opening of Now Playing window after the first time.
  • Song time counter goes higher than 59:59.
  • Fixed Now Playing window sometimes not displaying metadata in the list.
  • It seems rock stable. If you can crash it, make sure you let me know. :)

How to install? I have first release installed right now, how I do?

towhatend 2011-09-15 21:40

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

Originally Posted by robin92 (Post 1089209)
How to install? I have first release installed right now, how I do?

Do the same thing!

robin92 2011-09-15 21:49

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

Originally Posted by towhatend (Post 1089211)
Do the same thing!

sudo python mediaplayer-install.sh?

towhatend 2011-09-15 21:55

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

Originally Posted by robin92 (Post 1089215)
sudo python mediaplayer-install.sh?

Nah, itīs sh and not python. ;)

sudo sh install_mediaplayer.sh should do it, put --desktop-file at the end if you like to have a desktopfile, too.

Trestry 2011-09-15 22:21

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Does anyone here tried playing .pls files? I added a .pls file as a radio station. The default mediaplayer plays them fine, but when I try to play the radiostation with OMP it plays the next radiostation in the list.

Thanks anyway for this great media player!

tuxoutpost 2011-09-16 04:02

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

Originally Posted by Trestry (Post 1089227)
Does anyone here tried playing .pls files? I added a .pls file as a radio station. The default mediaplayer plays them fine, but when I try to play the radiostation with OMP it plays the next radiostation in the list.

Thanks anyway for this great media player!

Leave off the end of the URL i.e. "listen.pls" it is not needed.

tuxoutpost 2011-09-16 04:06

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Tried to add a Radio Station "http://216.66.69100:7222/" and received an invalid url error message.

Had to add the station with the default Media Player. Once it was added it played fine with the Open Media Player.

F2thaK 2011-09-16 05:41

Re: [ANNOUNCE] Alpha release of Open Media Player
 
good to see someones working on this..

gidzzz 2011-09-17 03:07

Re: [ANNOUNCE] Alpha release of Open Media Player
 
1 Attachment(s)
Quote:

Originally Posted by ThePooBurner (Post 1089094)
In the "now playing" playlist, tapping a song does not play the song, as in mediaplayer.

Hmmm... that's not how it is supposed to work. Even single tapping should change the song. Maybe the playlist was still loading and the responsiveness was crippled.

Quote:

Originally Posted by sakya (Post 1089127)
For the music window performance: I think that using a ListView/TableView and QAbstractListModel/QAbstractTableModel would be a lot faster than a QListWidget.

It might be true, but I'm not sure if it's worth the effort. As soon as the lists are fully populated, I get scrolling that is almost as smooth as silk, even with over 2800 songs and the loading time seems to be more dependent on how fast MAFW can serve the requests rather than how fast QListWidget adds the results. Should I get more songs for my library? I just couldn't reproduce the general slowness I've been reading about...

Oh, and I thought it would be nice to post a new build. ;) Here come the essential news:
  • Massive speed gains when shuffling whole library. Still not as swift as the stock player, but it should be usable.
  • I thought I fixed blank playlist in Now Playing before, but this time I hope it's for everyone. :)
  • Playlist state indicator no longer pops up to cover the searchbox.
  • It's possible to create and delete radio bookmarks.

neboja 2011-09-17 06:34

Re: [ANNOUNCE] Alpha release of Open Media Player
 
2 Attachment(s)
nice nice nice! :)

ok, now i must ask for this: there is implementation of lyrics in OMP... end when you tap on album art, it switch from playlist, to lyrics, than tap again for progress bar...
my personal opinion is that it should have a settings that enable/disable lyrics in now playing (maybe i don't want that extra tap)... what do you think?
is it simple to integrate???
another thing: when in portrait mode, a playlist part in now playing is a bit smaller than in landscape (???)... when in landscape we see 5 songs/files (whatever) but in portrait, only 4... could a media art part be smaller (not much, just a little bit) so a more songs could see in playlist (5 or even 6)??? that would be much more "finger friendly",... this way you have your thumb on a relatively small area, and viewing pls is a bit difficult (depends how FAT is your thumb) :p :)

oh yes... another thing:
look at the difference between album art for OMP and stock MP

OMP
http://talk.maemo.org/attachment.php...1&d=1316240983

stock MP
http://talk.maemo.org/attachment.php...1&d=1316241056

on omp is kinda fragmented (not jitter-ed) , don't you think???

titi974 2011-09-17 13:46

Re: [ANNOUNCE] Alpha release of Open Media Player
 
am I the only one that finds that all songs view is very slow, I mean when I choose a song it's not playing the one I chose and then the entire app goes so sloooow :s

MohammadAG 2011-09-17 14:15

Re: [ANNOUNCE] Alpha release of Open Media Player
 
There's no threading, it adds all items in the playlist then plays the song you selected.

T4UR14N 2011-09-17 15:20

Re: [ANNOUNCE] Alpha release of Open Media Player
 
app crashes on song/playlist delete & when fm transmitter is enabled from omp menu

anybody else experience this ?

gidzzz 2011-09-18 23:07

Re: [ANNOUNCE] Alpha release of Open Media Player
 
2 Attachment(s)
Quote:

Originally Posted by titi974 (Post 1090176)
when I choose a song it's not playing the one I chose

I'm aware of that and I've got a possible fix, but I'm still investigating and looking for the best solution.

Quote:

Originally Posted by neboja (Post 1089913)
my personal opinion is that it should have a settings that enable/disable lyrics in now playing (maybe i don't want that extra tap)... what do you think?
is it simple to integrate???

It should be fairly easy. I'll give it a try tonight or tomorrow.

Quote:

Originally Posted by neboja (Post 1089913)
when in landscape we see 5 songs/files (whatever) but in portrait, only 4... could a media art part be smaller (not much, just a little bit) so a more songs could see in playlist (5 or even 6)??? that would be much more "finger friendly"

I think that's a good point, but smaller cover art probably wouldn't look as nice. There's also possibility of removing the reflection, but I really like that subtle touch. In the future I want to implement editing of the playlist in Now Playing and maybe then I'll be forced to decide on the best way to enlarge the playlist area.

Quote:

Originally Posted by neboja (Post 1089913)
on omp is kinda fragmented (not jitter-ed) , don't you think???

Covers in stock MP are smoother, but in OMP they look more detailed. :)

excelar8 2011-09-19 01:20

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Pardon the noob question but I've installed the omp_20110917.tar.gz file. Now if I place that folder into the main N900 folder, what code do I need to input into X-terminal to update my OMP?

TMavica 2011-09-19 01:32

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

Originally Posted by excelar8 (Post 1091480)
Pardon the noob question but I've installed the omp_20110917.tar.gz file. Now if I place that folder into the main N900 folder, what code do I need to input into X-terminal to update my OMP?

Put omp_20110917.tar.gz in MyDocs, then unzip it to there

X terminal:
1. root
2. cd /home/user/MyDocs/omp_20110917
3. sh install_mediaplayer.sh --desktop-file

Done

neboja 2011-09-19 08:02

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

i have more suggestions to you (i see that you are currently maintaining this project):

could it be implemented a playlist creation "from folder"?
i think it was discussed somewhere many times, and it is thing that would be most useful for music playing etc....

i think i mentioned this somewhere before: type and search trough current playlist in now playing (very very useful!!!)

and i'm not thinking that i ask this for myself, but i think that these are suggestions to make OMP a better and more useful player!

keep up the good work! (too all dev-s) ;)

MohammadAG 2011-09-19 10:05

Re: [ANNOUNCE] Alpha release of Open Media Player
 
gidzzz, what's your gitorious username?

neboja 2011-09-19 10:22

Re: [ANNOUNCE] Alpha release of Open Media Player
 
can i answer that? :)

https://gitorious.org/~gidzzz

Zentenario 2011-09-19 10:23

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Great, MAG & gidzzz!!! One more app in portrait mode. I'm charmed with your work (all devs, of course, CSSU and the rest). I'm just wondering if calendar app can be also updated to portrait mode; it's the one and only app of "missing" portrait apps.;):rolleyes:

towhatend 2011-09-19 15:47

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

Originally Posted by Zentenario (Post 1091644)
Great, MAG & gidzzz!!! One more app in portrait mode. I'm charmed with your work (all devs, of course, CSSU and the rest). I'm just wondering if calendar app can be also updated to portrait mode; it's the one and only app of "missing" portrait apps.;):rolleyes:

Canīt do, not open source.

neboja 2011-09-19 16:17

Re: [ANNOUNCE] Alpha release of Open Media Player
 
hmmmm... much fun in git i see :) :) :)

edit: a little off-topic, but i must ask: what program(s) you use for creating this app???

MohammadAG 2011-09-19 17:30

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Qt Creator :)

gidzzz 2011-09-19 17:52

Re: [ANNOUNCE] Alpha release of Open Media Player
 
I use gedit and grep. :]

TomJ 2011-09-20 07:52

Re: [ANNOUNCE] Alpha release of Open Media Player
 
(Question about calendar)

Quote:

Originally Posted by towhatend (Post 1091818)
Canīt do, not open source.

No, but it's a similar situation to the Media Player; closed source and limited front end to an open source backend. The calendar backend can do a lot that the front end can't cope with (categories, flexible repetition etc) so a replacement front end would be a boon.

zdanee 2011-09-20 08:20

Re: [ANNOUNCE] Alpha release of Open Media Player
 
No butterflies? :)
http://imgs.xkcd.com/comics/real_programmers.png

StocChr 2011-09-20 17:59

Re: [ANNOUNCE] Alpha release of Open Media Player
 
same issues like mag's and capiperez version
( issue 3 is fixed )
see.. http://talk.maemo.org/showpost.php?p...2&postcount=58

gidzzz 2011-09-20 18:34

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

Originally Posted by StocChr (Post 1092698)
same issues like mag's and capiperez version
( issue 3 is fixed )
see.. http://talk.maemo.org/showpost.php?p...2&postcount=58

I think I fixed number 2 just a minute ago. :)

Do you know if those files from the screenshot have proper track number metadata?

StocChr 2011-09-21 06:07

Re: [ANNOUNCE] Alpha release of Open Media Player
 
the problem is not a track number or the metadata
i fixed the sorting problem in the Qt Creator...
- singlealbumview.ui
- singleplaylistview.ui
i enabled the option "sortingEnabled" in Object: songList -> Class: QListWidget
- singleartistview.ui
- singleplaylistview.ui
i enabled the option "sortingEnabled" in Object: albumList -> Class: QListWidget

after this modifications all files ( albums, artists, songs, etc. ) are sorted correctly

gidzzz 2011-09-21 12:38

Re: [ANNOUNCE] Alpha release of Open Media Player
 
No, they're not. It causes the lists to be sorted alphabetically, thus it only works if you have a number in every title. If you have correct metadata, it actually breaks the proper order.

Moreover, I've got 5 files: "101.mp3", "102.mp3", "103.mp3", "201.mp3", "202.mp3". They have no metadata and in the browser they are displayed in the correct order (it's only 5 files, might be random ;) ). Maybe it's not just that you don't have track numbers assigned, but rather have them assigned incorrectly.

So, I'd rather say check your metadata.

Edit: Yep, it was random. :p

StocChr 2011-09-21 13:19

Re: [ANNOUNCE] Alpha release of Open Media Player
 
short mp3 file description:

filename:
101 Artist - Titel.mp3
metadata:
titel: 101 Artist - Titel.mp3
interpret: Albumname
album: Albumname
genre: Pop
no Tracknumbers are defined in metadata

the quintessence is, that he sorts the songs acc. the metadata "titel" from 1, 2, 3, ... till z ( my experience with no assigned tracknumbers )

MohammadAG 2011-09-21 14:12

Re: [ANNOUNCE] Alpha release of Open Media Player
 
I've fixed that locally but my git setup is messed up right now (for qt-mediaplayer at least).

Anyway, QListWidget shouldn't do the sorting to match the stock player, MAFW should, you can pass "+title" in mafw_source_browse's sorting parameter to get a sorted reply (one of the lists does this already, singlealbumwindow.cpp I think)

gidzzz 2011-09-21 14:21

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Exactly. :)

gidzzz 2011-09-21 20:49

Re: [ANNOUNCE] Alpha release of Open Media Player
 
1 Attachment(s)
There were some good things brewing on git and I think it's about time to post a build for people who don't like compiling. ;)
  • Option to enable/disable lyrics (I decided to disable by default, as this is a feature depending on Internet connection).
  • Smarter playlist (re)loading and small bugfixes in "Now playing".
  • Smooth cover art and thumbnails.
  • Changing cover art also generates a thumbnail for album browser.
  • Cover art detection supports "cover.jpg" and "front.jpg".
  • Fixed lyrics reloading.
  • Fixed shuffle always starting with the same item.
  • Fixed playback not starting after shuffling an album.
  • Improved sorting order when browsing an album.
  • More bugfixes, this list is already too long. :p
And one last thing... There was a strange bug when choosing a song from "All songs". I'm not sure if I really fixed it, so I've got a request for people with big libraries (about 3000 songs, more is better): go to "All songs", scroll to the very end, select one of the songs and see if playback starts correctly.

ironm8 2011-09-21 21:22

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

Originally Posted by gidzzz (Post 1093466)
There were some good things brewing on git and I think it's about time to post a build for people who don't like compiling. ;)
  • Option to enable/disable lyrics (I decided to disable by default, as this is a feature depending on Internet connection).
  • Smarter playlist (re)loading and small bugfixes in "Now playing".
  • Smooth cover art and thumbnails.
  • Changing cover art also generates a thumbnail for album browser.
  • Cover art detection supports "cover.jpg" and "front.jpg".
  • Fixed lyrics reloading.
  • Fixed shuffle always starting with the same item.
  • Fixed playback not starting after shuffling an album.
  • Improved sorting order when browsing an album.
  • More bugfixes, this list is already too long. :p
And one last thing... There was a strange bug when choosing a song from "All songs". I'm not sure if I really fixed it, so I've got a request for people with big libraries (about 3000 songs, more is better): go to "All songs", scroll to the very end, select one of the songs and see if playback starts correctly.

Thanks alot for investing your time in this :)

Is it possible to make OMP take CD Number Sorting in the IDv3 tags under consideration? it messed up quite a few of my albums.

gidzzz 2011-09-21 22:14

Re: [ANNOUNCE] Alpha release of Open Media Player
 
Does stock player do that? Unless I overlooked something, MAFW doesn't support fetching such metadata and things get a bit complicated. It would have to be extracted manually and slow things down. Although one album might have not enough songs to make it noticable.

datjomp 2011-09-21 23:02

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

Originally Posted by TMavica (Post 1091484)
Put omp_20110917.tar.gz in MyDocs, then unzip it to there

X terminal:
1. root
2. cd /home/user/MyDocs/omp_20110917
3. sh install_mediaplayer.sh --desktop-file

Done

Sorry for asking but is --desktop-file suppose to add an icon to the desktop?
When I run sh install_mediaplayer.sh --desktop-file i get this --->
NowPlayingIndicator::autoSetVisibility
Is the omp.png supposed to be entered somwhere in the command?

Thanks, looks really promising :)

gidzzz 2011-09-21 23:13

Re: [ANNOUNCE] Alpha release of Open Media Player
 
"--desktop-file" installs a file that can be picked up by the menu system, so OMP is available from the applications menu or can placed as a shortcut on the desktop, just like any other shortcut. "omp.png" icon is copied automatically to its destination if you use "--desktop-file".


All times are GMT. The time now is 16:37.

vBulletin® Version 3.8.8