#metabrainz

/

      • supersandro2000 has quit
      • supersandro2000 joined the channel
      • leonh has quit
      • yvanzo has quit
      • yvanzo joined the channel
      • leonh joined the channel
      • Zastai has quit
      • supersandro2000 has quit
      • supersandro2000 joined the channel
      • prabal has quit
      • SomalRudra has quit
      • prabal joined the channel
      • jmp_music__ has quit
      • shivam-kapila has quit
      • SomalRudra joined the channel
      • shivam-kapila joined the channel
      • jmp_music__ joined the channel
      • sumedh joined the channel
      • sumedh has quit
      • davic has quit
      • davic_ joined the channel
      • supersandro2000 has quit
      • supersandro2000 joined the channel
      • davic_ is now known as davic
      • sumedh joined the channel
      • supersandro2000 has quit
      • supersandro2000 joined the channel
      • v6lur joined the channel
      • jmp_music__
        Morning!!
      • reosarevok
        Hi!
      • Gazooo79494 has quit
      • Gazooo79494 joined the channel
      • Zastai joined the channel
      • Mr_Monkey
        pristine___: Saw your comment on the feedback PR, let me run it locally and see what's affecting the state, and how to best solve it
      • pristine___
        Cool. Thanks :)
      • Mr_Monkey: if you running it locally, can you also check the rendering in mobile view? . I think we may need `MediaQuery` for that
      • reosarevok has quit
      • reosarevok joined the channel
      • zas
        bitmap: can you please have a look at paco's disk space? we get alerts about it since yesterday, prolly related to pg
      • ruaok
        once again it happened once I logged in, but I really dont think it was me...
      • 😇
      • alastairp
        mmm
      • someone told me that youtube's iframe api is being removed in a few weeks (what we use in acousticbrainz)
      • there's a "data api" to look up this info apparently, but it has data limits
      • I wonder if we should just add in BP here?
      • Mr_Monkey
        pristine___: It looks like the pagination in the recommendations page sets `currRecPage` wrong in `handleClickPrevious` and `handleClickNext` and should be inverted. When I navigate from page 1 to page 2 the url is changed to `?page=1` and going from page 2 to page 1 URL changes to `?page=2`
      • sumedh has quit
      • The feedback UI is looking good much better now, well done !
      • _lucifer
        alastairp: what functionality of the iframe api does ab use ?
      • alastairp
        there's an iframe which takes an artist and track name and generates a playlist (we just show the first one)
      • the idea is to have a preview of the song
      • _lucifer
        oh ok, actually that's the only part of the api being deprecated
      • alastairp
        haha
      • _lucifer
        >This is a deprecation announcement for the embedded player functionality that lets you configure the player to load search results. This announcement affects the IFrame Player API's queueing functions for lists, cuePlaylist and loadPlaylist.
      • sumedh joined the channel
      • bp sounds a great idea then. we can also put it in cb also.
      • shivam-kapila
        Hi people
      • BrainzGit
        [bookbrainz-site] MonkeyDo opened pull request #529 (master…update-pg-node-lts-14): chore: Update bookbrainz-data package https://github.com/bookbrainz/bookbrainz-site/p...
      • Mr_Monkey
        Zastai: Hi! Thanks for reporting the issues on LB. I think there are two separate issues. I can't comment on the playing-now functionality, but I also see the issue you're talking about.
      • Let's see about the embedded player issues.
      • I can't seem to reproduce either of your issues (interrupted playback in spotify app and switching between three recent tracks), so I'll start by asking you to unlink and re-link your spotify account on https://listenbrainz.org/profile/connect-spotify , and see if the player and playing-now functionality behave any better.
      • What browser and version are you using? Perhaps I can try to replicate on your browser.
      • Zastai
        latest firefox
      • Mr_Monkey
        Thanks, I'll try
      • Zastai
        i can't reproduce the issue now either. I don't suppose API calls are likely to have any effect on the behaviour
      • after unlinking and relinking, playing-now still returns nothing, despite a track being played
      • aha - i reproduced the player issue by accident
      • music was playing. switching between reports and listense did nothing special
      • going to listens shows an empty player at first, then fills it with the track playing in spotify (which is not shown in the listen list)
      • so far so good
      • i then checked the API call for playing now - no result
      • I then refreshed the LB page (at listens)
      • player is empty, might have briefly loaded the playing track but then shows alternatingly the last three listened tracks
      • playback has stopped
      • travis-ci joined the channel
      • travis-ci
        Project bookbrainz-site build #3510: passed in 4 min 39 sec: https://travis-ci.org/bookbrainz/bookbrainz-sit...
      • travis-ci has left the channel
      • Zastai
        going to the spotify app, it says "listening on ListenBrainz player"; playback queue (the release radar playlist) is empty except for the supposed active song
      • trying to play a track in the app does nothing (switches back to the track shown on LB)
      • had to switch the device back to the app in order to regain control
      • going to try to see if the API call is what triggered it
      • started playback again (in app). LB player shows that track just fine. F5 reloads it fine
      • after API call and F5: still fine
      • so it looks like semi-random intermittent behaviour, sadly
      • https://beta-api.listenbrainz.org/1/user/Zastai... is still reporting nothing, even though I'm on the second track since the player issue (which was after relinking)
      • and now after refreshing LB again, it shows a playing now (actually the previous track, not the current one) - API returns it too
      • Mr_Monkey back from lunch
      • Mr_Monkey
        Hmm, weird behaviour. I'll keep trying to trigger it.
      • Ozric Tentacles <3
      • iliekcomputers: Could we get together at some point and fix the issue with websockets in LB? I think the "playing now" feature on the listens page might be broken because of a CORS issue with socket.io
      • Zastai: Next time you see that player issue happen, could you please open the developer console and copy the entire console logs into a pastebin ? I keep refreshing and trying different scenarios but can't reproduce the issue so far.
      • Zastai
        ok, will try to reproduce it again later
      • Mr_Monkey
        Thanks for your help
      • bitmap
        zas: looking into it
      • sumedh has quit
      • BrainzGit
        [bookbrainz-site] MonkeyDo merged pull request #529 (master…update-pg-node-lts-14): chore: Update bookbrainz-data package https://github.com/bookbrainz/bookbrainz-site/p...
      • sumedh joined the channel
      • sumedh has quit
      • sumedh joined the channel
      • bitmap
        zas: ruaok: afaik what's growing on paco is /var/lib/postgresql/data/base/pgsql_tmp, but I'm not sure what queries are creating these or why they aren't being cleaned up. we'll probably have to enable log_temp_files and restart PG to see
      • ruaok
        let me make sure that I don;t accidentally have some large tables laying around.
      • pristine___
        > pristine___: It looks like the pagination in the recommendations page sets `currRecPage` wrong in `handleClickPrevious` and `handleClickNext` and should be inverted. When I navigate from page 1 to page 2 the url is changed to `?page=1` and going from page 2 to page 1 URL changes to `?page=2`
      • Mr_Monkey: it's because I am using the previous state value here (not the updated one)
      • On it
      • Mr_Monkey
        Ah, right !
      • ruaok
        bitmap: nothing abnormal here.
      • should we do "vacuum analyze full" maybe?
      • bitmap
        hmm
      • ruaok
        I did try to run a bad query yesterday that OOMed PG. I wonder if there is fallout from that.
      • #rookiemove
      • bitmap
        iiuc pgsql_tmp is supposed to just have temporary files for queries that don't fit in work_mem and should be deleted once the query is done
      • yeah, I'm not sure. maybe we should stop PG and just clear this dir and restart
      • atm it's 115GB and has too many files for ls to even list :) for comparison w/ floyd it's empty there
      • pristine___
        Mr_Monkey: so I was thinking, what should the page look like for user != currentUser. Maybe we can disable the drop down in this case and only show the feedback.
      • travis-ci joined the channel
      • travis-ci
        Project bookbrainz-site build #3512: passed in 4 min 58 sec: https://travis-ci.org/bookbrainz/bookbrainz-sit...
      • travis-ci has left the channel
      • ruaok
        bitmap: feel free to do that. all of my stuff can be regenerated should something get borked.
      • bitmap
        aight, running the delete. didn't have to restart PG since they weren't used by any process (the backend they were created by no longer exists)
      • ruaok
        thx
      • Mr_Monkey
        pristine___: Disabled dropdown, surely. Which feedback do you show though? For example, if I go to see your recommendations, do I see your feedback? It would need to be very very clear that it is yours and not mine.
      • Is there any interest in allowing me to give feedback to your suggestions? I don't think so.
      • pristine___
        So if I visit /recs/Mr_Monkey, I should ideally see your recs and feedback given by you, no?
      • Mr_Monkey
        If it's shown in exactly the same way as for my feedback, that would be confusing ("I don't remember giving feedback to these songs, what's going on?". It'll need to be made clear that it is your feedback, but I'm not sure how.
      • pristine___
      • The user name, in this case 'Rob' is good enough for me to understand that's it's not my feedback, no?
      • Mr_Monkey: or we fade the color lil to show it is disabled something like that.
      • Not sure either, though I somehow feel user name should do the trick
      • reosarevok
        This showing "Your top artist playlist" if it's not you is not very good
      • What about printing "This" instead of "Your" if user != profile?
      • We generally differentiate in MB
      • Mr_Monkey
        I agree with that reo, although it's a separate concern for another PR
      • ruaok, alastairp : Any opinion on whether we want to show another user's feedback when looking at their recommendations? i.e. if I look at Rob's recommendations, should I see the feedback Rob gave?
      • ruaok
        I don't think so.
      • because if I am spending time with someone else and a song comes on that I like, I want to go to their timeline and like the song from there.
      • conversely I'd like to see if I've already liked it.
      • Mr_Monkey
        It feels uneasy for me UX-wise to see someone else's feedback at the same spot where I'm supposed to give feedback on mine.
      • ruaok
        I share that.
      • pristine___
        Hmm... What should the other user see there?
      • ruaok
        I dont feel strongly about showing my feedback on someone else's feed. I do care about being able to like a track from anywhere its shown.
      • Mr_Monkey
        Are we completely replacing the current love/hate hearts with the 'faces' feedback component?
      • pristine___
        No.
      • Not for now at least
      • Mr_Monkey
        If not, I would keep the faces feedback component for currentUsers's recs pages, and show the love/hate component instead if != currentUser
      • pristine___
        Mr_Monkey: I don't understand this completely. What exactly will be shown in that case?
      • We can't show the heart and heart_broken icons
      • Mr_Monkey
        Why is that?
      • pristine___
        So we are allowing people to give feedback if they load someone else's rec page, but this feedback will essentially be for "listens" (1, -1) and not "recs" (The five emoticons)
      • Is that?