#metabrainz

/

      • CatQuest
      • 2019-06-16 16718, 2019

      • CatQuest
        (thing I could think of; haveing a medium with tracks and then *removing* them, but it still also works)
      • 2019-06-16 16711, 2019

      • SothoTalKer
        removing tracks would be a destructive edit, no?
      • 2019-06-16 16748, 2019

      • CatQuest
        aw man i hate that
      • 2019-06-16 16756, 2019

      • CatQuest
        ... but you're probably right
      • 2019-06-16 16735, 2019

      • SothoTalKer
        you could remove all tracks and then tick the "unknown tracklist" checkbox, i guess
      • 2019-06-16 16750, 2019

      • SothoTalKer
        i'm too dumb to login to test anyway :p
      • 2019-06-16 16704, 2019

      • CatQuest
        just use "mb" aspassport
      • 2019-06-16 16708, 2019

      • CatQuest
        erh word
      • 2019-06-16 16717, 2019

      • SothoTalKer
        naughty
      • 2019-06-16 16744, 2019

      • CatQuest
        all passwwords are reset to "mb" on test
      • 2019-06-16 16754, 2019

      • CatQuest
        to aid testing
      • 2019-06-16 16703, 2019

      • CatQuest
        it's not naughty
      • 2019-06-16 16731, 2019

      • D4RK-PH0ENiX has quit
      • 2019-06-16 16707, 2019

      • SothoTalKer
        so yeah, it works :)
      • 2019-06-16 16712, 2019

      • D4RK-PH0ENiX joined the channel
      • 2019-06-16 16731, 2019

      • SothoTalKer
        reosarevok: when is barcode and catalog numbers for mediums coming? ;)
      • 2019-06-16 16714, 2019

      • pristine__
      • 2019-06-16 16734, 2019

      • pristine__
        ruaok: Can you relate?
      • 2019-06-16 16748, 2019

      • pristine__
        I am pretty happy with my results :)
      • 2019-06-16 16718, 2019

      • zas
        Can someone explain huge loads on trille one can see at https://stats.metabrainz.org/d/000000048/hetzner-… ? It has very bad impact on mb website/web service containers response times.
      • 2019-06-16 16738, 2019

      • zas
        Containers running on trille https://www.irccloud.com/pastebin/LQ5ye7Zc/
      • 2019-06-16 16740, 2019

      • zas
        very high load (up to 40) happened between 13:50 and 16:00 UTC yesterday and 23:00 and 02:40 last night
      • 2019-06-16 16750, 2019

      • Cyna
        yvanzo: see this log https://gist.github.com/anirudhjain75/a3d49412a20… Makes it clear here
      • 2019-06-16 16758, 2019

      • iliekcomputers
        pristine__: the top artists, those are from collaborative filtering?
      • 2019-06-16 16710, 2019

      • pristine__
        iliekcomputers: no
      • 2019-06-16 16753, 2019

      • pristine__
        Just the top 20 artists of feb 19.
      • 2019-06-16 16700, 2019

      • iliekcomputers
        Ah
      • 2019-06-16 16702, 2019

      • iliekcomputers
        Do you plan to run it with the entire data, or is just a recent month intentional?
      • 2019-06-16 16755, 2019

      • iliekcomputers
        Also, we have data for May in the cluster now too, I'd be more curious to run it on that.
      • 2019-06-16 16733, 2019

      • pristine__
        There is no plan as now since we are not sure how often we will be training the model and generating recommendations.
      • 2019-06-16 16700, 2019

      • pristine__
        I can run it with the data of month May and provide you with the results.
      • 2019-06-16 16701, 2019

      • Gazooo has quit
      • 2019-06-16 16755, 2019

      • Gazooo joined the channel
      • 2019-06-16 16712, 2019

      • reosarevok
        SothoTalKer: thanks for volunteering!
      • 2019-06-16 16707, 2019

      • reosarevok
        SothoTalKer: the report thing sounds reasonable, actually :)
      • 2019-06-16 16711, 2019

      • reosarevok
        Thanks!
      • 2019-06-16 16719, 2019

      • reosarevok
        The medium thing, thanks for volunteering :p
      • 2019-06-16 16732, 2019

      • flamingspinach has quit
      • 2019-06-16 16733, 2019

      • flamingspinach joined the channel
      • 2019-06-16 16712, 2019

      • pristine__
      • 2019-06-16 16741, 2019

      • pristine__
        iliekcomputers: ^
      • 2019-06-16 16751, 2019

      • pristine__
        for May.
      • 2019-06-16 16714, 2019

      • Freso
        SothoTalKer reosarevok, I’d say the report should be in addition to, not instead of. Unless you’re making it so that you can no longer add entirely media‐less releases.
      • 2019-06-16 16730, 2019

      • reosarevok
        Yeah, I thought in addition to :)
      • 2019-06-16 16735, 2019

      • reosarevok
        But we should have one
      • 2019-06-16 16701, 2019

      • Freso
        Yep.
      • 2019-06-16 16719, 2019

      • Freso
        Did you ever look at my genre stat ticket?
      • 2019-06-16 16741, 2019

      • Freso
        It’s still not linked to anything anyway.
      • 2019-06-16 16705, 2019

      • Lotheric
        genre stat !!
      • 2019-06-16 16709, 2019

      • Lotheric
        +1
      • 2019-06-16 16711, 2019

      • Lotheric
        :)
      • 2019-06-16 16720, 2019

      • Lotheric
        good idea
      • 2019-06-16 16722, 2019

      • Freso
        Lotheric: If you have any thoughts: https://tickets.metabrainz.org/browse/MBS-10225
      • 2019-06-16 16723, 2019

      • BrainzBot
        MBS-10225: Add statistics for genre data
      • 2019-06-16 16759, 2019

      • Lotheric
        voted
      • 2019-06-16 16714, 2019

      • Lotheric
        I'm very curious to see stats on genres :)
      • 2019-06-16 16746, 2019

      • Freso
        Yep. Me too.
      • 2019-06-16 16707, 2019

      • SothoTalKer
        Freso, reosarevok: do we still need that option? I mean when it is implemented you can just tick both boxes "i don't know the medium type" & "i don't know the tracklist":
      • 2019-06-16 16716, 2019

      • SothoTalKer
      • 2019-06-16 16757, 2019

      • Freso
        SothoTalKer: Seems better to leave it out entirely, esp. if you don’t know the amount of media.
      • 2019-06-16 16724, 2019

      • Freso
        If you claim that a release has 1 medium though you don’t know the type or tracklist of, but it really has 4 media…
      • 2019-06-16 16729, 2019

      • SothoTalKer
        the checkbox should probably be renamed then, that also the mediums are unknown "The tracklist for this release is currently unknown."
      • 2019-06-16 16759, 2019

      • SothoTalKer
        like "mediums & tracklist are unknown"
      • 2019-06-16 16714, 2019

      • yvanzo
        zas: found unusual entries in mb ws logs.
      • 2019-06-16 16727, 2019

      • SothoTalKer
        (i hope it's not me)
      • 2019-06-16 16721, 2019

      • ruaok
        yvanzo: zas is busy today. what sorts of weird entries?
      • 2019-06-16 16744, 2019

      • yvanzo
        ruaok: yup, just replied to him about debug progress :)
      • 2019-06-16 16759, 2019

      • yvanzo
      • 2019-06-16 16709, 2019

      • reosarevok
        SothoTalKer: that's a fair point, could change it to something like "We have no information about this release's media and tracklist"
      • 2019-06-16 16715, 2019

      • reosarevok
        Freso: what do you think? :)
      • 2019-06-16 16736, 2019

      • D4RK-PH0ENiX has quit
      • 2019-06-16 16710, 2019

      • yvanzo
        Cyna: 'state' should probably not be part of 'Props' in FormRowNameWithGuessCase.js
      • 2019-06-16 16746, 2019

      • yvanzo
        See root/static/scripts/account/components/EditProfileForm.js for example
      • 2019-06-16 16727, 2019

      • D4RK-PH0ENiX joined the channel
      • 2019-06-16 16746, 2019

      • BrainzGit
        [musicbrainz-server] reosarevok merged pull request #1098 (master…MBS-10218): MBS-10218: Fix page param (cover_art -> cover-art) https://github.com/metabrainz/musicbrainz-server/…
      • 2019-06-16 16747, 2019

      • BrainzBot
        MBS-10218: Release page: Cover Art tab not selected / highlit https://tickets.metabrainz.org/browse/MBS-10218
      • 2019-06-16 16736, 2019

      • D4RK-PH0ENiX has quit
      • 2019-06-16 16756, 2019

      • D4RK-PH0ENiX joined the channel
      • 2019-06-16 16729, 2019

      • Cyna
        if I want to have the input field value change, I have to pass in state
      • 2019-06-16 16759, 2019

      • Cyna
        I'm using hooks withs const so the only way I can get the state is by passing it through props
      • 2019-06-16 16709, 2019

      • SothoTalKer
        reosarevok: good enough for me, i guess :)
      • 2019-06-16 16737, 2019

      • yvanzo
        CatQuest: found 2 typos: 'saluang' and 'violin family' descriptions have </br> instead of <br/>
      • 2019-06-16 16735, 2019

      • SothoTalKer
        is it <br/> or <br />?
      • 2019-06-16 16728, 2019

      • yvanzo
        either way, but </br> is invalid (closing a tag that has not been opened)
      • 2019-06-16 16708, 2019

      • SothoTalKer
        i prefer <p></p> anyway :D
      • 2019-06-16 16752, 2019

      • BrainzGit
        [musicbrainz-server] reosarevok merged pull request #1105 (master…MBS-10233): MBS-10233: Use React templates when canceling edit https://github.com/metabrainz/musicbrainz-server/…
      • 2019-06-16 16754, 2019

      • BrainzBot
        MBS-10233: ISE when trying to cancel a "add release annotation" edit https://tickets.metabrainz.org/browse/MBS-10233
      • 2019-06-16 16731, 2019

      • SothoTalKer
        reosarevok: you could get it merged to beta aswell, i guess? :x
      • 2019-06-16 16719, 2019

      • yvanzo
        reosarevok: should the two above be marked as regressions?
      • 2019-06-16 16711, 2019

      • yvanzo
        SothoTalKer: definitely going to beta today or tomorrow :)
      • 2019-06-16 16735, 2019

      • SothoTalKer
        yay
      • 2019-06-16 16729, 2019

      • reosarevok
        yvanzo: probably :)
      • 2019-06-16 16739, 2019

      • reosarevok
        SothoTalKer: not from my phone I can't :)
      • 2019-06-16 16711, 2019

      • SothoTalKer
        oh
      • 2019-06-16 16712, 2019

      • yvanzo
        not from my phone either, it's broken (warranty expired about 1 month ago) :P
      • 2019-06-16 16715, 2019

      • Cyna
        bitmap: Can you help me with the state issue... yvanzo said I shouldn't be passing state as Props
      • 2019-06-16 16733, 2019

      • Cyna
        I'm having flow error because of during one of the two types of FormRowSelect. https://gist.github.com/anirudhjain75/a3d49412a20…
      • 2019-06-16 16723, 2019

      • yvanzo
        Cyna: That might be legit after all but it is unclear which component this 'state' relates to.
      • 2019-06-16 16715, 2019

      • Cyna
        state comes only from editForm.js
      • 2019-06-16 16746, 2019

      • yvanzo
        Is there any type definition for this 'state'?
      • 2019-06-16 16752, 2019

      • Cyna
        Yes
      • 2019-06-16 16716, 2019

      • Cyna
        Its in the Props part of wherever it is used
      • 2019-06-16 16700, 2019

      • Cyna
        for example, formRowNameWithGuesscase component
      • 2019-06-16 16745, 2019

      • Cyna
      • 2019-06-16 16754, 2019

      • Cyna
        Its causing issue on editForm because FormRowSelect accepts value as both string or number. while the state property that is passed in is either of string or number but not both
      • 2019-06-16 16730, 2019

      • yvanzo
        Cyna: A temporary workaround could be to enforce annoted type for target.value in onChange parameter given to FormRowSelect calls in EditForm.js
      • 2019-06-16 16717, 2019

      • Cyna
        any idea how do I do it... seems confusing to me
      • 2019-06-16 16743, 2019

      • yvanzo
        with : any
      • 2019-06-16 16702, 2019

      • yvanzo
        ((target.value: any): integer)
      • 2019-06-16 16719, 2019

      • yvanzo
        ((target.value: any): number)
      • 2019-06-16 16705, 2019

      • Cyna
        ```onChange={(e) => {
      • 2019-06-16 16706, 2019

      • Cyna
        const target = e.target;
      • 2019-06-16 16706, 2019

      • Cyna
        if (target instanceof HTMLInputElement) {
      • 2019-06-16 16706, 2019

      • Cyna
        return setState({
      • 2019-06-16 16706, 2019

      • Cyna
        ...state,
      • 2019-06-16 16706, 2019

      • Cyna
        locale: {
      • 2019-06-16 16706, 2019

      • Cyna
        ...state.locale,
      • 2019-06-16 16707, 2019

      • Cyna
        value: ((target.value: any): number),
      • 2019-06-16 16707, 2019

      • Cyna
        },
      • 2019-06-16 16708, 2019

      • Cyna
        });
      • 2019-06-16 16708, 2019

      • Cyna
        }
      • 2019-06-16 16709, 2019

      • Cyna
        return undefined;
      • 2019-06-16 16709, 2019

      • yvanzo
        Cyna: yup, should do. A definitive fix would be to improve FormRowSelect with parameterized type
      • 2019-06-16 16729, 2019

      • yvanzo
        This is not supported by flow yet, see https://github.com/facebook/flow/issues/7672
      • 2019-06-16 16701, 2019

      • Gazooo has quit
      • 2019-06-16 16753, 2019

      • Gazooo joined the channel
      • 2019-06-16 16706, 2019

      • Freso
        reosarevok: Late but: SGTM.
      • 2019-06-16 16729, 2019

      • Freso
        SothoTalKer: `<br/>` vs. `<br />` is mostly a legacy thing. Any browser from the last 5 (if not more) years should handle either fine. When XHTML was first introduced, many browsers would not understand `<br/>` but would understand `<br />`, so `<br />` was the recommended way for most compatibility. I guess it’s still recommended if you need to support IE 3 or whatever. :)
      • 2019-06-16 16714, 2019

      • Freso
        Probably more like 10 or more years.
      • 2019-06-16 16744, 2019

      • Freso
        XHTML is from 2000, so ~15 or so years or more recent is probably a good estimate. :p
      • 2019-06-16 16740, 2019

      • CatQuest
        not even *my* browser is *that* old :D
      • 2019-06-16 16754, 2019

      • Lotheric_ joined the channel
      • 2019-06-16 16716, 2019

      • Lotheric has quit
      • 2019-06-16 16739, 2019

      • Lotheric_ is now known as Lotheric
      • 2019-06-16 16755, 2019

      • D4RK-PH0ENiX has quit