Fanart Handler Plugin v2.2.3.197 (and music fanart scraper) (1 Viewer)

Status
Not open for further replies.

cul8er

Retired Team Member
  • Premium Supporter
  • August 31, 2006
    2,159
    2,679
    Home Country
    Sweden Sweden
    • Thread starter
    • Moderator
    • #51
    Hi,

    Version 2.2.4 will be released very soon.

    Change log;
    • New feature: New option in configuration to enable or disable asynchronously loading of random images.
    • New feature: Fanart support in my videos movie info screen (thanks Deda)
    • Bug fixed: Black fanart in some occasions
    • Bug fixed: Artist Thumb filename starts with a 'space' character when downloading thumbs for multi-artists tracks

    About the New option in configuration
    If this new option is checked (default) Fanart Handler will work as before. By that I mean images will load asynchronously . And by this I mean for example: You are in plugins screen and choose to go back to basichome. Basichome will open fast but not all random background images has been loaded yet but will be displayed a short while after basichome is opened. This allows for fast switching back to for example basichome. If you uncheck this new option Fanart Handler will load images synchronously. By this I mean all images will load prior to displaying the new screen. Will look better but be a little slower in MP. Your choice ;)

    cheers
    cul8er
     

    Deda

    Lead Dev MP1 Videos
  • Premium Supporter
  • March 18, 2009
    2,423
    2,385
    Zagreb
    Home Country
    Croatia Croatia
    Hi cul8er, another tip for FA for My Videos:

    There is a problem in sequel title FA handling (ie. Die Hard and Die Hard 2). Currently FA handler when selecting titles in movie list, cuts off trailing numbers and pass artist name and result can be wrong.

    ---------------------
    Problem case:

    Movie titles: Die Hard & Die Hard 2

    Movie Fanart files: Die Hard.jpg & Die Hard 2.jpg

    Fanart returns both files no matter what is selected in movie list because of:

    1. Initial scraping reads FA filenames and cuts off trailing number and store in FA database Artist (title) as Die Hard so for both files, Artist will have the same name Die Hard

    2. Upon movie selection in My Videos FA cuts off number and pass title without it for searching FA database artist Die Hard and will return both fanart no matter what title is selected.

    further problem is for multiple fanart per movie and how to name other FA files for the same title (indexing)

    --------------------
    Possible solution which I hope will not break fanart database search logic for oher sections (Music, MovingPics...:

    1. Skip cutting of trailing numbers for My Videos
    2. Introduce indexing rules for mutliple fanart per title (ie Title {X}.jpg where X = 0 to 9 ->10 fanart per title)
    Fanart will cutoff that index for MyVideos only and store title (artist) without it, also selected title will go in Fanart database search with full name and will retreive right fanart. Like that, it will not break anything else, only if user have self defined extra fanarts for one video he will need to add curly brackets for index number ie. Title1.jpg ->Title{1}.jpg.

    I created test patch which possibly can fix this so if you can look at it and see how it fits or is there a possible conflict.

    Regards,
    Deda
     

    cul8er

    Retired Team Member
  • Premium Supporter
  • August 31, 2006
    2,159
    2,679
    Home Country
    Sweden Sweden
    • Thread starter
    • Moderator
    • #53
    Hi Deda,

    Thanks. Patch applied and works great.

    cheers
    cul8er
     

    cul8er

    Retired Team Member
  • Premium Supporter
  • August 31, 2006
    2,159
    2,679
    Home Country
    Sweden Sweden
    • Thread starter
    • Moderator
    • #55
    Hi,

    Next version will also contain Artist thumb scraping from LastFM (and correct image cropping to thumbnail 1:1 format). Everything is working and I'm now looking into the possibility to add Album scrape from LastFM. Then we would finally have artist, album and fanart scraping in an automatic way withing the MP client.

    cheers
    cul8er
     

    bodiroga

    Portal Pro
    January 1, 2008
    241
    46
    Home Country
    Spain Spain
    Hi,

    Next version will also contain Artist thumb scraping from LastFM (and correct image cropping to thumbnail 1:1 format). Everything is working and I'm now looking into the possibility to add Album scrape from LastFM. Then we would finally have artist, album and fanart scraping in an automatic way withing the MP client.

    cheers
    cul8er

    Hi cul8er!

    Don't you think that it would be better to release the Music scraper (artist, albums and backdrops) as a separate plugin or also include it in Mediaportal's music configuration? I really appreciate all the work you are doing with this plugin (it's amazing!), but I think that you are including in it many things that are making it "confusing".

    I would keep the original plugin (the one that gives controls to skinner to have fanart all over their skins) as Fanart Handler and make a new one (Music Pic Scraper or something like that) with this features. You could also include in this one the plugin you wrote for XFactor that provides artist and album biography to the skin.

    What do you think about it? I know that it's not a necessary change, but I think that that way it would be easier for newer users to configure their Mediaportal configuration.

    Many many thanks for all your work cul8er, I really admire you since your XFactor skin development. You rock!

    Best regards,

    Aitor
     

    ysmp

    Design Group
  • Team MediaPortal
  • May 17, 2008
    1,863
    744
    Seoul.
    Home Country
    South Korea South Korea
    Hi cul8er!

    Don't you think that it would be better to release the Music scraper (artist, albums and backdrops) as a separate plugin or also include it in Mediaportal's music configuration? I really appreciate all the work you are doing with this plugin (it's amazing!), but I think that you are including in it many things that are making it "confusing".

    I would keep the original plugin (the one that gives controls to skinner to have fanart all over their skins) as Fanart Handler and make a new one (Music Pic Scraper or something like that) with this features. You could also include in this one the plugin you wrote for XFactor that provides artist and album biography to the skin.

    What do you think about it? I know that it's not a necessary change, but I think that that way it would be easier for newer users to configure their Mediaportal configuration.

    Many many thanks for all your work cul8er, I really admire you since your XFactor skin development. You rock!

    Best regards,

    Aitor

    hi, i for one like one plugin to handel all ,like it's now ,,, MediaPortal alrady have to meny plugin to split plugin that are not necessary
    will bring more and more plugin ,wich new user alrady complain that MP hard to configer...
    i think the fanartHandler have a good Documentation and axplaination abut how to use , just need to read it..:)
     

    SebNYD

    MP Donator
  • Premium Supporter
  • December 26, 2008
    48
    2
    Home Country
    France France
    Re : Re: Fanart Handler Plugin v2.2.3.197 (and music fanart scraper)

    Hi cul8er,

    How is it going with v2.2.4 ? :)
    I'm especially interested by the bug fix that you mentionned sometimes ago regarding the black/empty fan arts that occurs sometimes.

    Thanks,

    Seb
     

    Deda

    Lead Dev MP1 Videos
  • Premium Supporter
  • March 18, 2009
    2,423
    2,385
    Zagreb
    Home Country
    Croatia Croatia
    Now it's time for 2.2.4 :) We need compatible version of FA for newest MP svn
     

    seco

    Retired Team Member
  • Premium Supporter
  • August 7, 2007
    1,575
    1,239
    Home Country
    Finland Finland
    Please fix the logging configuration of this plugin.

    Following line which is used two times by the Fanart Handler breaks every other plugin logging configuration:

    LoggingConfiguration config = new LoggingConfiguration();

    Instead, this should be

    LoggingConfiguration config = LogManager.Configuration ?? new LoggingConfiguration();
     
    Status
    Not open for further replies.

    Users who are viewing this thread

    Top Bottom