Notices


Reply
Thread Tools
Posts: 804 | Thanked: 1,598 times | Joined on Feb 2010 @ Gdynia, Poland
#121
Originally Posted by Estel View Post
OK, first of all, sorry for not providing terminal output, while reporting for the first time (it should be first thing to do *faceplam* - for reasons unknown, I was sure that the bug must be happening for everyone and is easy to reproduce) - to make it a more proper bug report (sorry for little messy way last time), I'm using version 0.5.2-6.

The relevant error messages from terminal (from while clicking on cover art that I would like to set):

[...large portion of code...]

If you're not able to reproduce this error, maybe it is something in my setup broken? If that would be suspected case, what should I attempt to reinstall?

Cheers,
/Estel
Could you also post your version of python? Is it pure, stock Maemo python or some upgraded version?
 

The Following User Says Thank You to misiak For This Useful Post:
Estel's Avatar
Posts: 5,028 | Thanked: 8,613 times | Joined on Mar 2011
#122
Sure - here is list of everything python-related that I have installed, including versions:

Code:
gnome-python 2.18.0-2maemo6
hildon-desktop-python-loader 0.1.0-1maemo3
python 2.5.2-3maemo4
python-all 2.5.2-3maemo4
python-cairo 1.4.12-1.2maemo2
python-central 0.6.11.1maemo2
python-conic 0.1-6
python-crypto 2.0.1+dfsg1-4maemo0
python-dbus 0.83.0-1maemo3
python-evolution 0.0.4-maemo2
python-gconf 2.26.1-1maemo1
python-glade2 2.12.1-6maemo10
python-gobject 2.16.1-1maemo2
python-gst0.10 0.10.14-2maemo5
python-gtk2 2.12.1-6maemo10
python-gtkhtml2 2.19.1-2maemo2
python-hildon 0.9.0-1maemo18
python-hildondesktop 0.1.0-1maemo3
python-imaging 1.1.6-3maemo2
python-location 0.2-1
python-lxml 2.2.6-1maemo1
python-minimal 2.5.2-3maemo4
python-mutagen 1.14-2
python-notify 0.1.1-2maemo1
python-numeric 24.2-9maemo4
python-numpy 1:1.4.0-1maemo4
python-osso 0.4-0maemo5
python-protobuf 2.4.0a-0maemo1
python-pygame 1.9.1release-0maemo1
python-pyside.qtcore 1.0.5-1maemo1
python-pyside.qtdeclarative 1.0.5-1maemo1
python-pyside.qtgui 1.0.5-1maemo1
python-pyside.qtmaemo5 1.0.5-1maemo1
python-pyside.qtnetwork 1.0.5-1maemo1
python-pyside.qtxml 1.0.5-1maemo1
python-qtmobility12 0.2.2-0maemo1
python-qtmobility12.connectivity 0.2.2-0maemo1
python-qtmobility12.contacts 0.2.2-0maemo1
python-qtmobility12.feedback 0.2.2-0maemo1
python-qtmobility12.gallery 0.2.2-0maemo1
python-qtmobility12.location 0.2.2-0maemo1
python-qtmobility12.messaging 0.2.2-0maemo1
python-qtmobility12.multimediakit 0.2.2-0maemo1
python-qtmobility12.organizer 0.2.2-0maemo1
python-qtmobility12.publishsubscribe 0.2.2-0maemo1
python-qtmobility12.sensors 0.2.2-0maemo1
python-qtmobility12.serviceframework 0.2.2-0maemo1
python-qtmobility12.systeminfo 0.2.2-0maemo1
python-qtmobility12.versit 0.2.2-0maemo1
python-scapy 2.0.1-1maemo2
python-simplejson 2.0.9-1maemo1
python-support 1.0.6maemo1
python-sympy 0.7.3-3maemo3
python2.5 2.5.4-1maemo6
python2.5-minimal 2.5.4-1maemo6
python2.5-qt4 4.7.5-maemo2
python2.5-qt4-common 4.7.5-maemo2
python2.5-qt4-core 4.7.5-maemo2
python2.5-qt4-dbus 4.7.5-maemo2
python2.5-qt4-gl 4.7.5-maemo2
python2.5-qt4-gui 4.7.5-maemo2
python2.5-qt4-maemo5 4.7.5-maemo2
python2.5-qt4-multimedia 4.7.5-maemo2
python2.5-qt4-network 4.7.5-maemo2
python2.5-qt4-phonon 4.7.5-maemo2
python2.5-qt4-script 4.7.5-maemo2
python2.5-qt4-sql 4.7.5-maemo2
python2.5-qt4-svg 4.7.5-maemo2
python2.5-qt4-test 4.7.5-maemo2
python2.5-qt4-webkit 4.7.5-maemo2
python2.5-qt4-xml 4.7.5-maemo2
python2.5-qt4-xmlpatterns 4.7.5-maemo2
python2.5-sip4 4.11-maemo0
python2.5-webkit 1.0.2-4
Cheers,
/Estel
__________________
N900's aluminum backcover / body replacement
-
N900's HDMI-Out
-
Camera cover MOD
-
Measure battery's real capacity on-device
-
TrueCrypt 7.1 | ereswap | bnf
-
Hardware's mods research is costly. To support my work, please consider donating. Thank You!
 

The Following User Says Thank You to Estel For This Useful Post:
Posts: 1,203 | Thanked: 3,027 times | Joined on Dec 2010
#123
os.rename error has been mentioned before. I'll take a look later today. If I can find a possible solution I'll upload a test version here before pushing to repo.

0.5.2-6 vs 0.5.2-8 is only 3 changes in search code for initial results displayed. Just looks to see if any results were returned for search and moves to a different term. -7 had a broken attempt at optifying.
 

The Following 3 Users Say Thank You to Android_808 For This Useful Post:
Posts: 1,203 | Thanked: 3,027 times | Joined on Dec 2010
#124
I don't get the "None None" near the thumbnail: section of the console output when run, but it may be because all my albums have artwork. I'll have to transfer more from PC when I get a chance and see what happens.

os.rename does have issues when files are on different devices. For mussorgsky both source and dest reside in /home/user/ (.cache/mussorgsky, .cache/media-art/*, and .thumbnails/cropped/).

Just to make sure, please try the attached deb. It won't fix the issue but should print variables (img_path & filename then thumb_path & thumbnail) before attempting os.rename() to see if they're at fault.

Edit: None None does't appear for me with no artwork set. I'll try to go through some more code tomorrow as your also missing alternative-0 lines.

Edit: Remove old test build.

Last edited by Android_808; 2014-02-02 at 18:57.
 

The Following 2 Users Say Thank You to Android_808 For This Useful Post:
Estel's Avatar
Posts: 5,028 | Thanked: 8,613 times | Joined on Mar 2011
#125
Thanks, Android_808 - just to confirm it, it is still relevant to check .deb you've attached and provide output, after your edit?
__________________
N900's aluminum backcover / body replacement
-
N900's HDMI-Out
-
Camera cover MOD
-
Measure battery's real capacity on-device
-
TrueCrypt 7.1 | ereswap | bnf
-
Hardware's mods research is costly. To support my work, please consider donating. Thank You!
 
Posts: 1,203 | Thanked: 3,027 times | Joined on Dec 2010
#126
yeah. try it using different buttons/covers as well. are there any other blank buttons other than the far right with popular albums?

i've only just got up. got a couple hours this morning then out till late afternoon. i'll take laptop with me but web connection is poor so it may have to wait till later.

Edit: None None output is produced when theres not enough results. I managed to reproduce it with Him - And Love Said No that only gives 4 out of 5 results plus the remove option. That's one possibility down

Last edited by Android_808; 2014-02-02 at 10:14.
 
Estel's Avatar
Posts: 5,028 | Thanked: 8,613 times | Joined on Mar 2011
#127
Here is output from using debug-enabled version:
http://p.mufff.in/?d7d3540678d41b70#...aKm/W30KdS72M=

As for blank buttons - even for very popular albums, it looks like this (still, only the right-most button work, the "small disc" doesn't):


For less popular albums, sometimes, the "small discs" (big dots) replace 2 or three covers from the right. Just like with popular albums, clicking on the "small discs" doesn't work - the far-right "default album art" is the only one working.

/Estel
__________________
N900's aluminum backcover / body replacement
-
N900's HDMI-Out
-
Camera cover MOD
-
Measure battery's real capacity on-device
-
TrueCrypt 7.1 | ereswap | bnf
-
Hardware's mods research is costly. To support my work, please consider donating. Thank You!
 
Posts: 1,203 | Thanked: 3,027 times | Joined on Dec 2010
#128
Thumbnail: /home/user/.cache/mussorgsky/alternative-1thumb is missing which would be causing the None None here. This would be the cause of the small disc. I don't get why yet as there are more results as using same search term on Google site provides more.

Small disc has never been clickable so that appears Ok.

**** CRITICAL **** image in path error is fixed in this build. There was a check of two img_path and thumb_path to see if they exist, which if not was trying to print the **** CRITICAL **** message using a non existant variable called "path". I've split it into 2 checks so it can output either img_path or thimb_path as necessary.

Side note: Long term goal I think should be a C++ Qt replacement. There are some issues I have found such as metadata/tags not editable for mp4 files. I would also like to offer other services to look up artwork such as Amazon. If I could use boo scripts I could just reuse those from AlbumArtDownloaderXUI and allow automated updates when they fix scipts.
Attached Files
File Type: deb mussorgsky_0.5.2-8_all.deb (31.8 KB, 75 views)
 

The Following 3 Users Say Thank You to Android_808 For This Useful Post:
Posts: 1,203 | Thanked: 3,027 times | Joined on Dec 2010
#129
any feedback on latest build estel? i have another i'm testing at the moment.

can anyone having trouble confirm /home/user/.cache/mussorgsky is on same partition as /home/user/.thumbnails/cropped and not symlinked elsewhere.
 
Estel's Avatar
Posts: 5,028 | Thanked: 8,613 times | Joined on Mar 2011
#130
You may have just hit the "bingo!". My /home/user/.thumbnails is on encrypted partition, symlinked from there (of course, it is mounted and transparently decrypted during usage, but the point is, that it's on different partition).

As for feedback from latest build (just had time to test it, sorry that it took a little longer than I expected):

Now, I don't get the "empty" small disc, all result placeholders are "filled" (except for the last one that set default no-album-art image, but that it's expected and desirable, I think). After the *second* tap on any artwork, the dialog closes - although, selected image is *not* set, dialog just closes. first tap does nothing (except for throwing error'ish lines in terminal).

Output (first tap, that does nothing):
Code:
~ $ mussorgsky
Tracker backend up
OK
Searching (album + artist): https://www.google.com/search?tbm=isch&q=Tori%20Amos+Boys%20for%20Pele&tbs=isz:m&iar:s
Working 0
Working 3
Working 4
Working 2
Working 1
None None
Thumbnail: /home/user/.cache/mussorgsky/alternative-4thumb
Thumbnail: /home/user/.cache/mussorgsky/alternative-2thumb
Thumbnail: /home/user/.cache/mussorgsky/alternative-0thumb
Thumbnail: /home/user/.cache/mussorgsky/alternative-1thumb
Thumbnail: /home/user/.cache/mussorgsky/alternative-3thumb
/home/user/.cache/mussorgsky/alternative-1 /home/user/.cache/mussorgsky/alternative-1thumb
Setting /home/user/.cache/mussorgsky/alternative-1thumb as image
/home/user/.cache/mussorgsky/alternative-2 /home/user/.cache/mussorgsky/alternative-2thumb
Setting /home/user/.cache/mussorgsky/alternative-2thumb as image
/home/user/.cache/mussorgsky/alternative-3 /home/user/.cache/mussorgsky/alternative-3thumb
Setting /home/user/.cache/mussorgsky/alternative-3thumb as image
/home/user/.cache/mussorgsky/alternative-4 /home/user/.cache/mussorgsky/alternative-4thumb
Setting /home/user/.cache/mussorgsky/alternative-4thumb as image
Debug: img_path: /home/user/.cache/mussorgsky/alternative-2 filename: /home/user/.cache/media-art/album-7215ee9c7d9dc229d2921a40e899ec5f-79e732a8e74d1d03a06059bfa004a015.jpeg
Debug: thumb_path: /home/user/.cache/mussorgsky/alternative-2thumb thumbnail: /home/user/.thumbnails/cropped/a7e2377a11b65947e7a7c2558ff11a42.jpeg
Traceback (most recent call last):
  File "/usr/lib/mussorgsky/aa_selection_dialog.py", line 167, in click_on_img
    tmp_thumb)
  File "/usr/lib/mussorgsky/album_art_thread.py", line 164, in save_alternative
    os.rename (thumb_path, thumbnail)
OSError: [Errno 18] Invalid cross-device link
After second tap:
Code:
**** CRITICAL **** image in path /home/user/.cache/mussorgsky/alternative-2 doesn't exist!
/Estel
__________________
N900's aluminum backcover / body replacement
-
N900's HDMI-Out
-
Camera cover MOD
-
Measure battery's real capacity on-device
-
TrueCrypt 7.1 | ereswap | bnf
-
Hardware's mods research is costly. To support my work, please consider donating. Thank You!
 

The Following 3 Users Say Thank You to Estel For This Useful Post:
Reply

Tags
album covers, media player, mussorgsky


 
Forum Jump


All times are GMT. The time now is 10:33.