#metabrainz

/

      • MajorLur_ joined the channel
      • 2020-09-24 26822, 2020

      • MajorLur_ is now known as MajorLurker_
      • 2020-09-24 26846, 2020

      • MajorLurker_ has quit
      • 2020-09-24 26833, 2020

      • milkii has quit
      • 2020-09-24 26838, 2020

      • milkii joined the channel
      • 2020-09-24 26814, 2020

      • ishaanshah
        Morning!
      • 2020-09-24 26838, 2020

      • ishaanshah
        ruaok: lots of 500's on artist msid-mbid lookup...
      • 2020-09-24 26843, 2020

      • ishaanshah
        any reason why?
      • 2020-09-24 26845, 2020

      • ishaanshah
      • 2020-09-24 26849, 2020

      • ishaanshah
      • 2020-09-24 26857, 2020

      • milkii has quit
      • 2020-09-24 26846, 2020

      • milkii joined the channel
      • 2020-09-24 26847, 2020

      • kori has quit
      • 2020-09-24 26825, 2020

      • kori joined the channel
      • 2020-09-24 26832, 2020

      • thomasross has quit
      • 2020-09-24 26849, 2020

      • kori has quit
      • 2020-09-24 26821, 2020

      • kori joined the channel
      • 2020-09-24 26856, 2020

      • kori has quit
      • 2020-09-24 26836, 2020

      • shivam-kapila
        ishaanshah: the api is not working for any user
      • 2020-09-24 26842, 2020

      • shivam-kapila
        also morning
      • 2020-09-24 26803, 2020

      • shivam-kapila
        ishaanshah:
      • 2020-09-24 26803, 2020

      • shivam-kapila
        ```psycopg2.errors.UndefinedTable: relation "mapping.msid_mbid_mapping" does not exist
      • 2020-09-24 26803, 2020

      • shivam-kapila
        LINE 12: ... FROM mapping.ms...
      • 2020-09-24 26803, 2020

      • shivam-kapila
        ^
      • 2020-09-24 26803, 2020

      • shivam-kapila
        ```
      • 2020-09-24 26825, 2020

      • kori joined the channel
      • 2020-09-24 26855, 2020

      • kori has quit
      • 2020-09-24 26839, 2020

      • kori joined the channel
      • 2020-09-24 26812, 2020

      • kori has quit
      • 2020-09-24 26853, 2020

      • kori joined the channel
      • 2020-09-24 26848, 2020

      • BrainzGit
        [musicbrainz-server] reosarevok opened pull request #1710 (master…MBS-11121): MBS-11121: Normalize Apple Music beta links https://github.com/metabrainz/musicbrainz-server/…
      • 2020-09-24 26850, 2020

      • BrainzBot
        MBS-11121: Update iTunes/Apple Music URL cleanup https://tickets.metabrainz.org/browse/MBS-11121
      • 2020-09-24 26810, 2020

      • kori has quit
      • 2020-09-24 26823, 2020

      • BrainzGit
        [musicbrainz-server] reosarevok opened pull request #1711 (master…MBS-8650): MBS-8650: Lower how long an IPI must be to 5 https://github.com/metabrainz/musicbrainz-server/…
      • 2020-09-24 26824, 2020

      • BrainzBot
        MBS-8650: Short IPIs are considered invalid https://tickets.metabrainz.org/browse/MBS-8650
      • 2020-09-24 26827, 2020

      • kori joined the channel
      • 2020-09-24 26816, 2020

      • v6lur joined the channel
      • 2020-09-24 26808, 2020

      • kori has quit
      • 2020-09-24 26840, 2020

      • kori joined the channel
      • 2020-09-24 26819, 2020

      • reosarevok
        Heh, just realized our reused-emails issue is also a problem for the otherwise trivial MBS-122
      • 2020-09-24 26819, 2020

      • BrainzBot
        MBS-122: Combine forgot username and forgot password pages/functions https://tickets.metabrainz.org/browse/MBS-122
      • 2020-09-24 26844, 2020

      • reosarevok
        bitmap: wanna port your script? :p https://tickets.metabrainz.org/browse/MBS-9563
      • 2020-09-24 26845, 2020

      • BrainzBot
        MBS-9563: Implement expand-collapse-release-groups.user.js as server code
      • 2020-09-24 26811, 2020

      • kori has quit
      • 2020-09-24 26830, 2020

      • alastairp
        _lucifer: you weren't on the approved list of committers for jenkins, which I think is why the jobs weren't running. I've added you
      • 2020-09-24 26858, 2020

      • _lucifer
        alastairp: ah! that makes sense. thanks!
      • 2020-09-24 26802, 2020

      • reosarevok
        bitmap, yvanzo: MBS-1575 - wontfix to not break the URLs, or change because these are not destination URLs people link to anyway?
      • 2020-09-24 26803, 2020

      • BrainzBot
        MBS-1575: Terminology: delete vs. remove release https://tickets.metabrainz.org/browse/MBS-1575
      • 2020-09-24 26825, 2020

      • reosarevok
        (everything else has been changed, except the url is still /delete)
      • 2020-09-24 26807, 2020

      • ruaok
        ishaanshah: ooops, sorry for that. we were trying to debug a filespace issue on paco yesterday and the mapping didn't regenerate correctly. I'll have it fixed in an hour.
      • 2020-09-24 26852, 2020

      • reosarevok
        outsidecontext: are you maintaining libdiscid now? Wondering if the bug mentioned in MBS-1588 still exists at all
      • 2020-09-24 26853, 2020

      • BrainzBot
        MBS-1588: More user friendly page when trying to add a CD with a bad TOC https://tickets.metabrainz.org/browse/MBS-1588
      • 2020-09-24 26858, 2020

      • reosarevok
        (not that nicer errors are a bad thing)
      • 2020-09-24 26856, 2020

      • outsidecontext
        reosarevok: I don't know if I do maintain it :D I did not intent to actually, but applied some changes recently so probably that means I volunteered
      • 2020-09-24 26804, 2020

      • reosarevok
        haha
      • 2020-09-24 26811, 2020

      • reosarevok
        I just asked because it's all you in recent history
      • 2020-09-24 26825, 2020

      • outsidecontext
        about the issue not sure, my gut feeling would be that the underlying issue is probably solved. that bug report is rather old, and there had been quite a few changes afterwards in around 2013. but hard to tell without knowing the actual disc and how the TOC was bad
      • 2020-09-24 26852, 2020

      • reosarevok
        Ok
      • 2020-09-24 26804, 2020

      • reosarevok
        Well, improving the error page won't hurt either way
      • 2020-09-24 26813, 2020

      • reosarevok
        But good to know the whole thing has been improved in the meantime anyway
      • 2020-09-24 26823, 2020

      • reosarevok
        alastairp: hey, wanna necro your 6 year old code? https://bitbucket.org/chirlu/musicbrainz-server/c…
      • 2020-09-24 26801, 2020

      • alastairp
        wow, that's a blast from the past
      • 2020-09-24 26825, 2020

      • yvanzo
        reosarevok: can be changed if needed, it will probably break URLs for that ticket only.
      • 2020-09-24 26854, 2020

      • kori joined the channel
      • 2020-09-24 26838, 2020

      • outsidecontext
        reosarevok: I really can't tell for sure, the info is very sparse there. The issue on that TOC is really that the total sectors are smaller then the last offset. but to see how this came to be one would need to check an actual disc producing something like this. can also be OS dependent, a glitch in reading the disc or maybe even have been generated with something other than libdiscid.
      • 2020-09-24 26838, 2020

      • outsidecontext
        anyway, I guess improving the error page for wrongly submitted TOCs wouldn't hurt and is basically independent of this.
      • 2020-09-24 26809, 2020

      • outsidecontext
        explaining that the submitted TOC has some invalid format and maybe link to discid documentation would help
      • 2020-09-24 26846, 2020

      • outsidecontext
        tested libdiscid, at least it does not accept the offsets if you feed it the TOC directly
      • 2020-09-24 26855, 2020

      • _lucifer
        alastairp: two queries regarding the PRs
      • 2020-09-24 26858, 2020

      • _lucifer
        > My preference is for these error messages to not have a . at the end of them
      • 2020-09-24 26816, 2020

      • _lucifer
        so only "You have to select a license" right?
      • 2020-09-24 26844, 2020

      • _lucifer
        second, i am not sure how to add a test case for the cache issue ?
      • 2020-09-24 26848, 2020

      • v6lur has quit
      • 2020-09-24 26828, 2020

      • v6lur joined the channel
      • 2020-09-24 26828, 2020

      • kori has quit
      • 2020-09-24 26801, 2020

      • Gazooo794 has quit
      • 2020-09-24 26849, 2020

      • Gazooo794 joined the channel
      • 2020-09-24 26827, 2020

      • alastairp
        reosarevok: off the top of your head do you know if flash messages in MB end with a . ?
      • 2020-09-24 26827, 2020

      • alastairp
      • 2020-09-24 26840, 2020

      • alastairp
        hah, and here is me saying ! at the end of a message is a bit naff and we shouldn't use them
      • 2020-09-24 26816, 2020

      • alastairp
        _lucifer: does CB have any tests where we mock the cache? I think I have a few of that kind in AB
      • 2020-09-24 26852, 2020

      • alastairp
        we could have a test where we test the specific get_x_by_id, mocking the cache and the BU get method, and just put in each of the types of necessary data
      • 2020-09-24 26827, 2020

      • alastairp
        e.g., mock cache.get returns None, verify that the db get method is called. mock cache.get returns something, verify that db get isn't called
      • 2020-09-24 26843, 2020

      • alastairp
        also, verify that the argument to mock cache.get is expected (e.g. 'artist_[artistid]')
      • 2020-09-24 26819, 2020

      • alastairp
      • 2020-09-24 26846, 2020

      • _lucifer
        alastairp: yes, i was able to find one in `test_statistics` where cache is mocked to return `None`
      • 2020-09-24 26824, 2020

      • alastairp
        perfect
      • 2020-09-24 26844, 2020

      • _lucifer
        so, 2 tests, in one mock cache and in other mock db, right?
      • 2020-09-24 26856, 2020

      • reosarevok
        alastairp: I don't know if all do, but imo they should
      • 2020-09-24 26806, 2020

      • reosarevok
        ! is nice in moderation
      • 2020-09-24 26832, 2020

      • iliekcomputers
        tfw you spend 2 weeks on a design doc for a 4 week project
      • 2020-09-24 26852, 2020

      • alastairp
        sure, but by spending 2 weeks on it you make sure that it doesn't blow out to 10 weeks
      • 2020-09-24 26858, 2020

      • ruaok
        I bet that is going to be worth it.
      • 2020-09-24 26817, 2020

      • kori joined the channel
      • 2020-09-24 26822, 2020

      • iliekcomputers
        true
      • 2020-09-24 26823, 2020

      • alastairp
        _lucifer: I think it might be OK. Do those methods have any tests?
      • 2020-09-24 26838, 2020

      • iliekcomputers
        getting people to agree on what needs doing is harder than just doing the thing :D
      • 2020-09-24 26810, 2020

      • alastairp
        reosarevok: that'd be a nice thing to put into a style-guide. I'm trying to decide what the best one for us is
      • 2020-09-24 26824, 2020

      • ruaok
        iliekcomputers: the story of my last 20 years!
      • 2020-09-24 26824, 2020

      • alastairp
        we also use these messages to say things like "field x is required"
      • 2020-09-24 26841, 2020

      • alastairp
        and I'm not sure I like . at the end of those, but I can be convinced
      • 2020-09-24 26804, 2020

      • alastairp
        yes, ! sometimes is good. That example that I pasted probably iskn't too bad
      • 2020-09-24 26819, 2020

      • CatQuest
        deff
      • 2020-09-24 26831, 2020

      • _lucifer
        alastairp: i didn't understand you completely. which methods are you referring to?
      • 2020-09-24 26802, 2020

      • alastairp
        critiquebrainz/frontend/external/musicbrainz_db/artist.py get_artist_by_id
      • 2020-09-24 26840, 2020

      • kori has quit
      • 2020-09-24 26801, 2020

      • _lucifer
        alastairp: no that method has no test
      • 2020-09-24 26832, 2020

      • alastairp
        OK, so let's add tests for all of these get_x_by_id methods, with the cache mocked as we discussed
      • 2020-09-24 26850, 2020

      • alastairp
        is there a sample mb database available in the tests?
      • 2020-09-24 26840, 2020

      • _lucifer
        yeah, there is a test_db argument used in init_db so must be
      • 2020-09-24 26812, 2020

      • _lucifer
        `postgresql://critiquebrainz:critiquebrainz@db_test:5432/critiquebrainz`
      • 2020-09-24 26823, 2020

      • _lucifer
        it points to this for the tests
      • 2020-09-24 26825, 2020

      • alastairp
        but that's the test critiquebrainz database, right?
      • 2020-09-24 26847, 2020

      • alastairp
      • 2020-09-24 26812, 2020

      • alastairp
        (that's a yes..., I was just checking so that we can do a test that actually loads this data from the musicbrainz database)
      • 2020-09-24 26811, 2020

      • _lucifer
        yeah, this db will be empty or it has some sample data we can use right away?
      • 2020-09-24 26854, 2020

      • kori joined the channel
      • 2020-09-24 26810, 2020

      • heyoni has quit
      • 2020-09-24 26832, 2020

      • alastairp
        that's what I'm asking you
      • 2020-09-24 26836, 2020

      • alastairp
        I don't know
      • 2020-09-24 26853, 2020

      • _lucifer
        oh ok! i'll check that
      • 2020-09-24 26811, 2020

      • heyoni joined the channel
      • 2020-09-24 26841, 2020

      • _lucifer
        it's empty alastairp
      • 2020-09-24 26823, 2020

      • kori has quit
      • 2020-09-24 26825, 2020

      • d4rkie joined the channel
      • 2020-09-24 26837, 2020

      • Nyanko-sensei has quit
      • 2020-09-24 26808, 2020

      • kori joined the channel
      • 2020-09-24 26851, 2020

      • alastairp
        _lucifer: right. I don't know if the tests do anything to load dummy data into the db or not. Perhaps all we should do is mock the call to the brainzutils method too?
      • 2020-09-24 26831, 2020

      • alastairp
        I don't like this much, because we're basically mocking everything, but I think the actual method is tested in BU anyway, so perhaps it's not strictly necessary
      • 2020-09-24 26822, 2020

      • _lucifer
        yeah, i agree. the get_x_by_id method is just choosing between the cache and db
      • 2020-09-24 26836, 2020

      • _lucifer
        so we can just check which one is it calling
      • 2020-09-24 26850, 2020

      • _lucifer
        and the argument is right or not ?
      • 2020-09-24 26851, 2020

      • alastairp
        yes
      • 2020-09-24 26844, 2020

      • BrainzGit
        [listenbrainz-server] mayhem opened pull request #1107 (master…msb-mapping-improvements): Cleanup from getting the mapping into production https://github.com/metabrainz/listenbrainz-server…
      • 2020-09-24 26838, 2020

      • kori has quit
      • 2020-09-24 26820, 2020

      • kori joined the channel
      • 2020-09-24 26844, 2020

      • kori has quit
      • 2020-09-24 26850, 2020

      • d4rkie has quit
      • 2020-09-24 26830, 2020

      • Nyanko-sensei joined the channel
      • 2020-09-24 26852, 2020

      • kori joined the channel
      • 2020-09-24 26815, 2020

      • kori has quit
      • 2020-09-24 26824, 2020

      • MajorLurker has quit
      • 2020-09-24 26826, 2020

      • kori joined the channel
      • 2020-09-24 26806, 2020

      • kori has quit
      • 2020-09-24 26826, 2020

      • BestSteve has quit
      • 2020-09-24 26800, 2020

      • BestSteve joined the channel
      • 2020-09-24 26837, 2020

      • kori joined the channel
      • 2020-09-24 26859, 2020

      • pristine___
        ruaok: "&" is a join phrase in MB?
      • 2020-09-24 26815, 2020

      • ruaok
        it can be
      • 2020-09-24 26831, 2020

      • pristine___
        Hmm, so should not be removed while removing punctuation.