#metabrainz

/

      • alastairp
        yes
      • 2015-09-17 26010, 2015

      • Freso
        Nice.
      • 2015-09-17 26037, 2015

      • ruaok nods at everything
      • 2015-09-17 26047, 2015

      • ruaok
        Freso: I'll explain that as i talk.
      • 2015-09-17 26058, 2015

      • Freso
        ruaok: I figured, but I'm not there for the talk. :)
      • 2015-09-17 26004, 2015

      • ruaok
        :(
      • 2015-09-17 26009, 2015

      • Freso
        I know.
      • 2015-09-17 26018, 2015

      • Freso
        (Hence why I was asking about video.)
      • 2015-09-17 26026, 2015

      • ruaok
        a bit later today we should add LB to the carousell on the meb site.
      • 2015-09-17 26004, 2015

      • Gentlecat
        ruaok: you can just add it now and deploy when the time comes
      • 2015-09-17 26017, 2015

      • ruaok is out of time.
      • 2015-09-17 26021, 2015

      • ruaok
        I need to leave soon.
      • 2015-09-17 26028, 2015

      • ariscop joined the channel
      • 2015-09-17 26058, 2015

      • Gentlecat
        I'll add that after I figure out how fix javascript
      • 2015-09-17 26048, 2015

      • github joined the channel
      • 2015-09-17 26048, 2015

      • github
        [listenbrainz-server] alastair closed pull request #22: Date fix for scraper (master...fix-scraper-date-issue) https://github.com/metabrainz/listenbrainz-server…
      • 2015-09-17 26048, 2015

      • github has left the channel
      • 2015-09-17 26006, 2015

      • Mineo
        wait, what?
      • 2015-09-17 26017, 2015

      • alastairp
        ??
      • 2015-09-17 26039, 2015

      • Mineo
        I just opened a ticket about some timestamp issues with the scraper and now that pops up :P
      • 2015-09-17 26022, 2015

      • Mineo
        but it doesn't look like it's related
      • 2015-09-17 26022, 2015

      • xBytez has quit
      • 2015-09-17 26039, 2015

      • alastairp
        ah, we’re aware of a timezone-related thing on the actual display page
      • 2015-09-17 26000, 2015

      • alastairp
        we’re pretty sure we’re storing the right value, but it might not be parsed and displayed in the correct timezone
      • 2015-09-17 26039, 2015

      • ruaok
        ok, I'm outta here. reachable on telegram and hangouts.
      • 2015-09-17 26040, 2015

      • alastairp
        2 hours ago == the difference between utc and cest
      • 2015-09-17 26043, 2015

      • ruaok
        ttttttfn!
      • 2015-09-17 26048, 2015

      • ruaok has quit
      • 2015-09-17 26012, 2015

      • UmkaDK has quit
      • 2015-09-17 26035, 2015

      • github joined the channel
      • 2015-09-17 26035, 2015

      • github
        [listenbrainz-server] alastair pushed 2 new commits to master: https://github.com/metabrainz/listenbrainz-server…
      • 2015-09-17 26035, 2015

      • github
        listenbrainz-server/master 10715a5 Alastair Porter: Merge branch 'unicode'
      • 2015-09-17 26035, 2015

      • github
        listenbrainz-server/master 0c1b53c Alastair Porter: Merge branch 'counter'
      • 2015-09-17 26035, 2015

      • github has left the channel
      • 2015-09-17 26038, 2015

      • UmkaDK joined the channel
      • 2015-09-17 26053, 2015

      • JonnyJD joined the channel
      • 2015-09-17 26053, 2015

      • Freso
        Ta ta ta ta ta ta for now? Is he firing machine guns at ud? D:
      • 2015-09-17 26055, 2015

      • Freso
        *us
      • 2015-09-17 26000, 2015

      • alastairp
        Mineo: thanks for double-checking &. I'll wipe the database and ask kepstin to try again before I close it
      • 2015-09-17 26043, 2015

      • github joined the channel
      • 2015-09-17 26043, 2015

      • github
        [listenbrainz-server] alastair deleted unicode at f8f5781: https://github.com/metabrainz/listenbrainz-server…
      • 2015-09-17 26043, 2015

      • github has left the channel
      • 2015-09-17 26050, 2015

      • github joined the channel
      • 2015-09-17 26050, 2015

      • github
        [listenbrainz-server] alastair deleted fix-scraper-date-issue at 0a4d707: https://github.com/metabrainz/listenbrainz-server…
      • 2015-09-17 26050, 2015

      • github has left the channel
      • 2015-09-17 26000, 2015

      • Gentlecat
        alastairp: well, that was definitely easier :D
      • 2015-09-17 26013, 2015

      • alastairp
        who knew? browsers are good at making the back button work
      • 2015-09-17 26032, 2015

      • alastairp
        ok. I'm about to delete the listenbrainz database
      • 2015-09-17 26037, 2015

      • Gentlecat
        D:
      • 2015-09-17 26039, 2015

      • github joined the channel
      • 2015-09-17 26039, 2015

      • github
        [listenbrainz-server] gentlecat pushed 1 new commit to listens-experiment: https://github.com/metabrainz/listenbrainz-server…
      • 2015-09-17 26039, 2015

      • github
        listenbrainz-server/listens-experiment 0aa4926 Roman Tsukanov: Just send another request to load new page...
      • 2015-09-17 26039, 2015

      • github has left the channel
      • 2015-09-17 26032, 2015

      • alastairp
        can someone with “Unknown date” scrobbles try and do an import?
      • 2015-09-17 26046, 2015

      • alastairp
        Mineo: can you check your timezone issue again?? it looks like it's taking into account timezone now with our testing (though we didn't do anything that we can see)
      • 2015-09-17 26001, 2015

      • Mineo
        I can't open the dropdown menu on my username anymore :(
      • 2015-09-17 26008, 2015

      • alastairp
        refresh
      • 2015-09-17 26011, 2015

      • alastairp
        we updated some css
      • 2015-09-17 26017, 2015

      • Mineo
        ah
      • 2015-09-17 26028, 2015

      • Mineo
        nope, it's still saying "2 hours ago" for tracks I just listened to
      • 2015-09-17 26011, 2015

      • alastairp
        can you give us:
      • 2015-09-17 26026, 2015

      • alastairp
        - inspect element on the time in lastfm, and see what the date is
      • 2015-09-17 26031, 2015

      • alastairp
        - your current time
      • 2015-09-17 26038, 2015

      • alastairp
        - your musicbrainz username
      • 2015-09-17 26013, 2015

      • alastairp
        ah, we think we have it - the date that lastfm gives seems to be in utc, but we’re interpreting it as a localtime and then converting to utc
      • 2015-09-17 26009, 2015

      • Mineo
        I've added the 3 things to the ticket
      • 2015-09-17 26014, 2015

      • alastairp
        thanks. I think we've found it
      • 2015-09-17 26026, 2015

      • alastairp
        I blame javascript
      • 2015-09-17 26051, 2015

      • Mineo
        ah, so if you correct it on the listens overview again, you're just converting it back to the utc time lastfm was showing, that makes sense
      • 2015-09-17 26014, 2015

      • alastairp
        "If the date format is in an ISO-9601 format with no timezone, UTC is assumed. If the date format is not in ISO-8601 with no timezone, the user's local timezone is assumed" <- actual javascript docs
      • 2015-09-17 26022, 2015

      • Mineo
        9601, 8601, the case for 7601 is obviously missing :P
      • 2015-09-17 26044, 2015

      • alastairp
        oh yes, 8601
      • 2015-09-17 26046, 2015

      • alastairp
        but same thing
      • 2015-09-17 26052, 2015

      • github joined the channel
      • 2015-09-17 26052, 2015

      • github
        [listenbrainz-server] alastair pushed 1 new commit to master: https://github.com/metabrainz/listenbrainz-server…
      • 2015-09-17 26052, 2015

      • github
        listenbrainz-server/master 4e79181 Alastair Porter: Treat date from last.fm as UTC...
      • 2015-09-17 26052, 2015

      • github has left the channel
      • 2015-09-17 26059, 2015

      • alastairp
        javascript!
      • 2015-09-17 26004, 2015

      • alastairp
        fixed, we'll deploy soon
      • 2015-09-17 26027, 2015

      • github joined the channel
      • 2015-09-17 26027, 2015

      • github
        [listenbrainz-server] alastair closed pull request #16: Remember current page (timestamp) when switching between pages (master...listens-experiment) https://github.com/metabrainz/listenbrainz-server…
      • 2015-09-17 26027, 2015

      • github has left the channel
      • 2015-09-17 26014, 2015

      • reosarevok joined the channel
      • 2015-09-17 26053, 2015

      • LordSputnik
        Leo_Verto: Leftmost: Bookzombie: meeting
      • 2015-09-17 26057, 2015

      • LordSputnik
        gah
      • 2015-09-17 26000, 2015

      • LordSputnik
        That didn't work :(
      • 2015-09-17 26002, 2015

      • LordSputnik
        Bookzombie: meeting
      • 2015-09-17 26018, 2015

      • LordSputnik
        Ok, so Bookzombie is very broken...
      • 2015-09-17 26041, 2015

      • LordSputnik
        Leo_Verto: Leftmost: anyway, there'll be a meeting in here tomorrow at 20:30 UTC usual time, this is just a reminder :D
      • 2015-09-17 26043, 2015

      • Gentlecat
        meetings
      • 2015-09-17 26036, 2015

      • LordSputnik
        Leo_Verto: how easy would it be for you to set me up with access to the Bookzombie server?
      • 2015-09-17 26014, 2015

      • LordSputnik
        Leo_Verto: if the answer is "not very", perhaps we could make a new Heroku account with one of the @bb.org emails, or host it on Google (nomo, with the chat log bot, ruaok?)
      • 2015-09-17 26009, 2015

      • github joined the channel
      • 2015-09-17 26009, 2015

      • github
        [listenbrainz-server] alastair closed pull request #20: Rework the URL structure: (master...improved-url-structure) https://github.com/metabrainz/listenbrainz-server…
      • 2015-09-17 26009, 2015

      • github has left the channel
      • 2015-09-17 26050, 2015

      • alastairp
      • 2015-09-17 26052, 2015

      • github joined the channel
      • 2015-09-17 26053, 2015

      • github
        [listenbrainz-server] alastair pushed 1 new commit to master: https://github.com/metabrainz/listenbrainz-server…
      • 2015-09-17 26053, 2015

      • github
        listenbrainz-server/master bc99d2a Alastair Porter: Make nginx listen on api.listenbrainz.org as well (send to same app)
      • 2015-09-17 26053, 2015

      • github has left the channel
      • 2015-09-17 26002, 2015

      • Mineo
        alastairp: "32 minutes ago" :)
      • 2015-09-17 26018, 2015

      • xBytez joined the channel
      • 2015-09-17 26018, 2015

      • xBytez has quit
      • 2015-09-17 26018, 2015

      • xBytez joined the channel
      • 2015-09-17 26025, 2015

      • Gentlecat
        we should add github irc hooks for other projects too :)
      • 2015-09-17 26033, 2015

      • alastairp
        too much content
      • 2015-09-17 26001, 2015

      • UmkaDK has quit
      • 2015-09-17 26006, 2015

      • UmkaDK_ joined the channel
      • 2015-09-17 26007, 2015

      • Gentlecat
      • 2015-09-17 26009, 2015

      • LordSputnik has quit
      • 2015-09-17 26047, 2015

      • xxer joined the channel
      • 2015-09-17 26045, 2015

      • xxer has left the channel
      • 2015-09-17 26054, 2015

      • zas joined the channel
      • 2015-09-17 26042, 2015

      • zxbnbs joined the channel
      • 2015-09-17 26003, 2015

      • madmouser1
        hi all, I have a basic plugin now that I can see in te context menu and teh options page for it is displaying, I have added teh python code to genearate the BPM and currently it is in a vairable; Question: how do I write that value now back to the file ?
      • 2015-09-17 26037, 2015

      • Freso
        It looks like I'm stuck at "Sending page 570 of 573 to ListenBrainz".
      • 2015-09-17 26020, 2015

      • [Kid] joined the channel
      • 2015-09-17 26030, 2015

      • [Kid]
        any issues with API right now?
      • 2015-09-17 26043, 2015

      • [Kid]
        i keep getting 503 when trying to update album info
      • 2015-09-17 26059, 2015

      • [Kid]
        for musicbrainz
      • 2015-09-17 26051, 2015

      • Mineo
      • 2015-09-17 26037, 2015

      • Mineo
        madmouser1: I guess you already have a reference to a picard.file.File object somewhere - those have a metadata attribute which is a dictionary where the keys are tag names
      • 2015-09-17 26015, 2015

      • Mineo
        picard already defines a tag called "bpm" (https://picard.musicbrainz.org/docs/mappings/) so you should just need to set that (metadata["bpm"] = the value you calculated)
      • 2015-09-17 26004, 2015

      • madmouser1
        Mineo: thanks I am nor a developer and just rying my hand at creating a plugin, but your comment do help as that is what I did but get an error so let me dig some more, I did teh following:
      • 2015-09-17 26010, 2015

      • madmouser1
        Mineo: metadata["bpm"] = calculated_bpm
      • 2015-09-17 26036, 2015

      • madmouser1
        I do have
      • 2015-09-17 26037, 2015

      • madmouser1
        from picard.metadata import register_album_metadata_processor, register_track_metadata_processor
      • 2015-09-17 26028, 2015

      • Mineo
        can you copy all of the code and the error message somewhere, like http://lpaste.net/ ?
      • 2015-09-17 26052, 2015

      • madmouser1
        I get this error : NameError: global name 'metadata' is not defined
      • 2015-09-17 26047, 2015

      • madmouser1
      • 2015-09-17 26018, 2015

      • madmouser1
        it is a mess please excuse, I am still exploring / learning and reverse engineering ...
      • 2015-09-17 26050, 2015

      • zxbnbs has left the channel
      • 2015-09-17 26033, 2015

      • madmouser1
        ok the mappings page say that cannot be updated by stock Picard, does that mean I need to use something else to update the tag e.g. mutagen ?
      • 2015-09-17 26012, 2015

      • Mineo
        no, that just means that picard by default does not write anything to the bpm tag
      • 2015-09-17 26031, 2015

      • Mineo
        ok, so before we come to the metadata thing, just let me clarify that you don't need to register metadata processors in your example
      • 2015-09-17 26009, 2015

      • Mineo
        metadata processors are run automatically after tracks/albums have been loaded, but what you want is to add a context menu option to calculate the bpm
      • 2015-09-17 26009, 2015

      • Mineo
        both are independent from each other: processors run automatically, the context menu callbacks need the user to click the entry in the context menu
      • 2015-09-17 26057, 2015

      • Mineo
        onwards to your metadata problem:
      • 2015-09-17 26049, 2015

      • Mineo
        in the function at http://lpaste.net/141181#line112, `file` is an instance of the `File` class (you can see its definition here: https://github.com/musicbrainz/picard/blob/master…)
      • 2015-09-17 26017, 2015

      • Mineo
        you're already accessing its filename attribute (defined at https://github.com/musicbrainz/picard/blob/master…) in some places of your code
      • 2015-09-17 26059, 2015

      • Mineo
        what the file object also has is a metadata attribute (defined at https://github.com/musicbrainz/picard/blob/master…) which is the dictionary I mentioned earlier
      • 2015-09-17 26035, 2015

      • Mineo
        you can access the metadata attribute in the same way you already access the filename: file.metadata :)
      • 2015-09-17 26052, 2015

      • alastairp
        Freso: hmm, this is weird
      • 2015-09-17 26007, 2015

      • alastairp
        I’m actually convinced it’s a counting problem, not a programming problem
      • 2015-09-17 26024, 2015

      • madmouser1
        thank you so much for your detailed explanation and help Mineo
      • 2015-09-17 26034, 2015

      • madmouser1
        let me get it right & fix it
      • 2015-09-17 26015, 2015

      • Fractalizator__ joined the channel
      • 2015-09-17 26005, 2015

      • Fractalizator__ has quit
      • 2015-09-17 26020, 2015

      • Fractalizator joined the channel
      • 2015-09-17 26021, 2015

      • Fractalizator has quit