Home of the Squeezebox™ & Transporter® network music players.
Page 14 of 14 FirstFirst ... 4121314
Results 131 to 134 of 134
  1. #131
    Former Squeezebox Guy andyg's Avatar
    Join Date
    Jan 2006
    Location
    Pittsburgh, PA
    Posts
    7,396

    Thoughts regarding new database schema ?

    On Jun 18, 2008, at 5:41 PM, Fred wrote:

    >
    > erland;313043 Wrote:
    >> Does anyone still reading this thread have any clue on the principles
    >> the solution for the new database schema needs to follow to be
    >> accepted
    >> by the Logitech developer team ?

    >
    > Anything reasonable that gets the job done and lots of patches.
    >
    > Because the model needs changing, the code that uses it needs
    > changing.
    > That opens the door to any major change like removing DBIx. Question 4
    > is linked to DBIx.
    >
    > I'd say the answer to 1 and 2 is do what works and is efficient. The
    > only case to take into account that you didn't is upgrading (migrating
    > from schema X to X+1 - ideally with no rescan).
    >
    > The way we discussed it originally was:
    > * We parse with Perl to feed a table with tags
    > * We use as much raw SQL as needed to populate the DB (insert into XXX
    > values (select xxx)) - try to avoid the Perl loop on data.
    > * If DBIx goes, so be it
    >
    > Not sure anything major changed...


    Any solution needs the input of the rest of the Logitech team and we
    can't even begin to think about this issue right now. That's why it's
    targeted to 7.3 (and may even slip to later). Please don't go off and
    try to do something crazy like rip out DBIC. I already mentioned we
    don't want patches for this, it's too big for someone to hack on by
    themselves without a proper design in place first.

    There are other things we want to work on related to this:

    Faster taglib-based scanner
    Integration with OS filesystem notification libraries: ChangeNotify,
    inotify, FSEvents

  2. #132
    Senior Member
    Join Date
    Jun 2005
    Location
    Switzerland
    Posts
    610
    Andy makes good points.

    Quote Originally Posted by andyg View Post
    Any solution needs the input of the rest of the Logitech team
    It needs the input of the Logitech team - I am not part of it and did not post like I was. I just wanted to summarize what the spirit of the Campfire discussion had been - I have no memory of any hard rules mentioned.

    Please don't go off and
    try to do something crazy like rip out DBIC. I already mentioned we
    don't want patches for this, it's too big for someone to hack on by
    themselves without a proper design in place first.
    Re-reading the post it's not very clear, but I meant the DESIGN can be anything reasonable that works and that people like and feel comfortable with so that lots of people have the *capacity* of submitting patches (and hopefully the inclination when the time to implement the design comes - which is not now).

    Sorry for any confusion. It's way too early for patches - we're just throwing ideas in the air to see which ones end up going down. Apparently most do

    Fred

  3. #133
    Senior Member
    Join Date
    Nov 2005
    Posts
    199
    Quote Originally Posted by andyg View Post
    Any solution needs the input of the rest of the Logitech team and we
    can't even begin to think about this issue right now.
    Adult supervision arrives. The party's over.

    Quote Originally Posted by andyg View Post
    [color=blue]
    That's why it's targeted to 7.3 (and may even slip to later).
    These features are needed now. In this thread

    http://forums.slimdevices.com/showthread.php?t=49088

    MuckleEck proposes to tag a 40 CD box set as a single album with

    album = "Dvorak - The Masterworks" and
    album artist as "Dvorak"

    He has a total of 405 CDs of classical CDs to tag.

    Bill

  4. #134
    Senior Member erland's Avatar
    Join Date
    Dec 2005
    Location
    Sweden
    Posts
    11,039
    Quote Originally Posted by Listener View Post
    Adult supervision arrives. The party's over.
    SqueezeCenter is open source, so anyone that likes to make modifications can do so and distribute the changes as patches. However, I think it is very reasonable that Logitech likes to be involved in the design of the code they are going to maintain later on.

    I'm very happy that Logitech (Andy) tells us when they feel that we are getting close to spend time doing stuff that they might need to re-implement a totally different way later on. Most of the things discussed in this thread might result in a major impact of the current SqueezeCenter code, so the final solution really needs proper design. I'm sure many of the people participating in this thread would be able to do the design work, but it isn't possible until Logitech has the time to tell us at least the major goal and principles of the solution.

    Quote Originally Posted by Listener View Post
    These features are needed now.
    Everyone is aware that they are needed for classical and jazz music, but there are a lot of other features that are needed for pop/rock music which is used by 90% of the users so it's just a matter of prioritizing the work.

    So if a user can't wait for 7.3 or 8.0 for these features, he will either have to use third party plugins or adapt to the current SqueezeCenter functionality.
    Erland Isaksson (My homepage)
    Developer of many plugins/applets

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •