#musicbrainz-devel

/

      • ruaok joined the channel
      • outsidecontext joined the channel
      • ruaok_ joined the channel
      • alastair1 joined the channel
      • warp
        hello!
      • ruaok joined the channel
      • adhawkins
        navap: Does Cubby really give unlimited storage? At what cost?
      • reosarevok joined the channel
      • ijabz joined the channel
      • ocharles joined the channel
      • ocharles
        yay, working desktop again
      • reosarevok
        :)
      • arch?
      • ocharles
        yea
      • somewhat annoyingly banshee segfaults when I import my music :(
      • reosarevok
        VLC :
      • *:p
      • Who wants libraries
      • warp
        vlc is terrible here.
      • reosarevok
        Or rather, who wants underachieving libraries... kepstin-work, finsih Riker :p
      • ocharles
        i'm installing quodlibet for now
      • reosarevok
        *finish
      • warp: terrible how? it's relatively lightweight and if you drop some files in it will play them
      • ocharles
        that's not really how I like to manage my music :)
      • warp
        reosarevok: I'm not sure. the sound is often stuttery or broken.
      • ocharles
        warp: what port is git.musicbrainz.org for writing again?
      • 10015?
      • warp
        reosarevok: I expect it's not buffering enough so that it cannot read the file fast enough when something else is doing stuff with the USB disk.
      • originssh://mbgitrw [at git.musicbrainz.org]:10015/musicbrainz-server.git (fetch)
      • originssh://mbgitrw [at git.musicbrainz.org]:10015/musicbrainz-server.git (push)
      • ocharles
        thanks
      • reosarevok
        ocharles: it's not how I would ideally manage it either
      • But library players are so depressing when you consistently use MB
      • warp
        reosarevok: maybe I can tweak it somewhere, but vlc also doesn't do gapless, so I haven't cared enouh to look into it.
      • reosarevok
        It's like "oh, great, I have so much info... oh, you can't use 90% of it? sweeet"
      • So I've gone back to letting Picard manage my "library" by folders
      • warp
        I manage it manually (picard cannot move folders, so cannot do this for me)
      • ocharles
        warp: artist.aliases should be an array
      • (of objects)
      • warp
        oh yes. I do have that elsewhere.
      • ocharles
        yea
      • in the next example it is :)
      • I'm not so sure about the artist objects in artist credits
      • they do not have the same schema as other artist objects
      • warp
        why don't they have the same schema?
      • ocharles
        in artist objects, omission of certain properties implies they are null, but in artist credits it implies we're just not showing you that data (but it might not be null)
      • right?
      • warp
        hm, I don't think our XML is very clear about that.
      • in general our XML omits stuff you didn't request
      • ocharles
        oh, this is just to be the same as /ws/2/
      • i guess it's ok then
      • warp
        so the distinction between omitted and null isn't clear in our XML either. is there even a distinction? :)
      • ocharles
        no
      • kurtjx joined the channel
      • kurtjx joined the channel
      • reosarevok
        Hmm
      • Do we actually have null ACs?
      • warp
        reosarevok: properties of artists can be null. e.g. not every artist has a disambig comment.
      • reosarevok
        Sure, but then surely if it does have one we'd show it?
      • warp
        reosarevok: in the webservice, not quite.
      • reosarevok
        Why shouldn't we?
      • (if I'm making very silly questions, stop me :) )
      • warp
        reosarevok: for example, a release has mediums with tracklists with tracks with recordings
      • reosarevok: but if you request http://musicbrainz.org/ws/2/release/59211ea4-ff... , you won't see any of that in the output.
      • reosarevok: just that something is missing from the webservice output doesn't mean the value is null or empty in the database.
      • reosarevok
        Ok, I guess my point was more "we either show them all, and then if we don't it's not there, or show none, and then none are shown"
      • (but it'd be quite confusing if we showed some but not some others of the same thing with the same query)
      • warp
        reosarevok: in general the output of the webservice is somewhat tailored for specific use cases, and information we didn't think was relevant is omitted by default to save bandwidth.
      • reosarevok: and that's why you can do ?inc=foo to make the server include those things which you think are relevant.
      • reosarevok
        Sure, but then, if we just don't include something, there's no possible confusion so why does it matter?
      • warp
        reosarevok: well, the confusion is then that the client who gets to interpret the webservice response can never know if something was merely omitted from the results or if it is actually null in the database.
      • reosarevok
        But reasonably enough, if it cares it should actually query for it?
      • warp
        yes, so you when you write your code to interpret the results, you have to take into account whether certain bits of data or normally included with the type of request you did.
      • reosarevok
        So, the JSON doesn't need to take this into account, the documentation does
      • And to avoid it being unclear it just needs clear documentation
      • That's what I read from this :)
      • warp
        I'd be in favour of just Doing It Right
      • reosarevok
        For ws/3? :p
      • warp
        so that e.g. if an artist doesn't have a disambig comment, we include "disambiguation": null
      • reosarevok
        Oh, that sounds reasonable, yes
      • ocharles
        new schema doesn't allow null for disambiguation, but that's an aside :)
      • warp
        so that if the "disambiguation" key is missing you know that the server omitted it.
      • ocharles: which new schema?
      • ocharles
        warp: NES
      • warp
        ah, so disambig is just "" then?
      • ocharles
        yea
      • warp
        seems... odd.
      • ocharles
        NULL does not mean absent
      • it means unknown
      • but we know that this artist has an empty comment
      • warp
        oh, right. you are correct.
      • ocharles
        same why a null joinphrase is a bit funky
      • (and has caused us problems)
      • warp
        ocharles: any other comments on /Web_Service/JSON ?
      • ocharles
        warp: i haven't quite looked in detail yet, but nothing stands out as wrong
      • warp
        do you think it's ok to make the relations one list?
      • ocharles
        yea, i don't see why not
      • warp
        it's seperate in the xml, but that just seemed annoying to me :)
      • ocharles: ok, thanks for taking a look. I'll post it to mb-devel.
      • kepstin-work has left the channel
      • reosarevok
        ocharles, warp: should we make this impossible, or do we agree it's fun enough to keep? http://musicbrainz.org/user/nikki
      • (age)
      • ocharles
        my original ticket description said max age of 100 at that time
      • reosarevok
        Sure
      • ocharles
        so i was surprised when nikki was allowed to do that
      • reosarevok
        this is way under 100 :p
      • ocharles looks
      • ocharles
        oh right
      • yes, your date of birth should not be greater than now()
      • reosarevok
        I guess it doesn't necessarily matter unless we start graphing them, though
      • ocharles
        data integrity matters to me, regardless of where it is :)
      • reosarevok
        :p
      • Fiiiine
      • nikki, when you read this, blame ocharles :p
      • ocharles
        all it takes is someone later to say "oh, I'll compute x!" and they suddenly are showing wrong data
      • more often than not, you won't realise until it's infected more of your data
      • reosarevok
      • This is what happens when our 3 main female editors don't set their gender :p
      • I guess the graphs should have an "unset" option
      • ocharles ponders if lmfao should even be able to generate those graphs
      • ocharles
        I guess it's kinda public data
      • oh wait, we dump editor names anyway, so this seems fine
      • ignore me :)
      • voiceinsideyou joined the channel
      • reosarevok
        I wouldn't be against limiting the data in the dumped editor table to only names, too
      • (or well, the bare minimum for it to import)
      • ocharles
        indeed!
      • can you ticket that?
      • reosarevok
      • ocharles
        thanks
      • reosarevok
        reosarevok has changed the topic to: http://musicbrainz.org/#devel | Agenda: text column constraints (ocharles), editor data in dumps (reotab)
      • And that's the other
      • ocharles
        :)
      • reosarevok
        (since the rest is public on the site so we should discuss whether we want it or not)
      • Leftmost joined the channel
      • hawke_2 joined the channel
      • Mineo is -1 years old on MB :-)
      • Tsk
      • Apparently we have 27 people who are 27
      • warp
        reosarevok: I don't have much of an opinion. it seems odd to me that we show age at all.
      • kepstin-work joined the channel
      • reosarevok
        Why? It's nice to have an idea of who is behind the username
      • warp
        reosarevok: I don't think there's much of a problem with dumping birth dates.
      • ocharles
        alright, time to get myself to Bloc weekend
      • warp
        it will take someone a year to determine the exact birth date from the age we show.
      • ocharles
        see you all on monday!
      • warp
        so a determined person will be able to get the birth date from the age.
      • reosarevok
        ocharles: enjoy!
      • warp: sure. so?