0:01 AM
D4RK-PH0ENiX joined the channel
2018-04-01 09150, 2018
0:06 AM
ephemer0l_ has quit
2018-04-01 09100, 2018
0:23 AM
ephemer0l_ joined the channel
2018-04-01 09117, 2018
0:54 AM
kartikeyaSh_ircc has quit
2018-04-01 09123, 2018
1:23 AM
G0re joined the channel
2018-04-01 09121, 2018
1:25 AM
Gore|woerk has quit
2018-04-01 09102, 2018
1:33 AM
Major_Lurker has quit
2018-04-01 09129, 2018
1:33 AM
Major_Lurker joined the channel
2018-04-01 09111, 2018
1:36 AM
Leo_Verto
2018-04-01 09157, 2018
2:09 AM
d4rkie joined the channel
2018-04-01 09106, 2018
2:11 AM
d4rkie has quit
2018-04-01 09111, 2018
2:11 AM
D4RK-PH0ENiX has quit
2018-04-01 09128, 2018
2:16 AM
D4RK-PH0ENiX joined the channel
2018-04-01 09122, 2018
2:21 AM
D4RK-PH0ENiX has quit
2018-04-01 09153, 2018
2:23 AM
D4RK-PH0ENiX joined the channel
2018-04-01 09159, 2018
2:24 AM
D4RK-PH0ENiX has quit
2018-04-01 09106, 2018
2:25 AM
D4RK-PH0ENiX joined the channel
2018-04-01 09114, 2018
2:26 AM
D4RK-PH0ENiX has quit
2018-04-01 09141, 2018
2:26 AM
D4RK-PH0ENiX joined the channel
2018-04-01 09131, 2018
2:28 AM
d4rkie joined the channel
2018-04-01 09101, 2018
2:30 AM
sentriz has quit
2018-04-01 09144, 2018
2:30 AM
d4rkie has quit
2018-04-01 09135, 2018
2:31 AM
D4RK-PH0ENiX has quit
2018-04-01 09119, 2018
2:35 AM
D4RK-PH0ENiX joined the channel
2018-04-01 09121, 2018
5:58 AM
D4RK-PH0ENiX has quit
2018-04-01 09116, 2018
6:09 AM
D4RK-PH0ENiX joined the channel
2018-04-01 09102, 2018
7:34 AM
ruaok pokes iliekcomputers and ferbncode
2018-04-01 09134, 2018
7:47 AM
iliekcomputers
Moin!
2018-04-01 09152, 2018
7:47 AM
ruaok
moiin!
2018-04-01 09153, 2018
7:47 AM
ruaok
2018-04-01 09117, 2018
7:48 AM
ruaok
thats the dumb recommender running.
2018-04-01 09130, 2018
7:48 AM
ruaok
unzipping lb data now to try the load_data
2018-04-01 09105, 2018
7:53 AM
iliekcomputers
ruaok: ferbncode: we still need to make the dataframes persistent
2018-04-01 09128, 2018
7:53 AM
iliekcomputers
2018-04-01 09159, 2018
7:53 AM
ruaok
yeah, I noticed that the script trailed off a bit, but I wanted to start playing with it anyway.
2018-04-01 09115, 2018
7:57 AM
ruaok
the old recommendation training ran 10 minutes on the cluster. O_O
2018-04-01 09154, 2018
7:57 AM
iliekcomputers
yike
2018-04-01 09157, 2018
7:57 AM
iliekcomputers
yikes
2018-04-01 09123, 2018
7:58 AM
ruaok
yeah. and that was with 4 * 27GB of ram.
2018-04-01 09115, 2018
8:00 AM
ruaok
and the LB dump appears corrupt. I can reproduce mineo's problem.
2018-04-01 09132, 2018
8:00 AM
outsidecontext joined the channel
2018-04-01 09145, 2018
8:03 AM
outsidecontext has quit
2018-04-01 09146, 2018
8:05 AM
drsaunde has quit
2018-04-01 09154, 2018
8:18 AM
ruaok
iliekcomputers: the other LB dump is corrupt too. :(
2018-04-01 09104, 2018
8:19 AM
ruaok
2018-04-01 09134, 2018
8:19 AM
ruaok
we need to fix it so that it won't recommend tracks you've got in your listen history.
2018-04-01 09100, 2018
8:20 AM
UmkaDK has quit
2018-04-01 09148, 2018
8:21 AM
iliekcomputers
2018-04-01 09118, 2018
8:22 AM
iliekcomputers
not sure why that isn't working
2018-04-01 09148, 2018
8:25 AM
bukwurm joined the channel
2018-04-01 09112, 2018
8:27 AM
ruaok
maybe it is recommending different versions of the same recording?
2018-04-01 09108, 2018
8:30 AM
akhilesh has quit
2018-04-01 09115, 2018
8:30 AM
akhilesh joined the channel
2018-04-01 09136, 2018
8:35 AM
iliekcomputers
maybe you didn't listen to the track in 2017?
2018-04-01 09150, 2018
8:35 AM
ruaok
ah! great observation!
2018-04-01 09123, 2018
9:03 AM
ruaok
the latest data dump inside the LB container is not corrupt.
2018-04-01 09129, 2018
9:03 AM
ruaok
copying that to the master.
2018-04-01 09121, 2018
9:05 AM
sentriz joined the channel
2018-04-01 09116, 2018
9:23 AM
UmkaDK joined the channel
2018-04-01 09159, 2018
9:36 AM
ruaok starts an LB data load
2018-04-01 09129, 2018
9:44 AM
bukwurm
LordSputnik: The tests on bb-data indeed fail. There's been a schema change in last two years?
2018-04-01 09115, 2018
9:45 AM
bukwurm
Here's a gist: relation "bookbrainz._editor_entity_visits" does not exist
2018-04-01 09136, 2018
9:45 AM
bukwurm
column "link_phrase" of relation "relationship_type" does not exist
2018-04-01 09131, 2018
9:46 AM
bukwurm
(and so does reverse_link_phrase)
2018-04-01 09144, 2018
9:46 AM
bukwurm
Same with "editor_type"
2018-04-01 09125, 2018
9:47 AM
bukwurm
Nopes, not editor_type, sorry.
2018-04-01 09135, 2018
9:48 AM
bukwurm
Initially ran into some permission issues, thought that were causing it - but I added all permissions.
2018-04-01 09153, 2018
9:49 AM
iliekcomputers
ruaok: did it stop with an error?
2018-04-01 09101, 2018
9:50 AM
ruaok
iliekcomputers: ferbncode: yep.
2018-04-01 09111, 2018
9:50 AM
UmkaDK has quit
2018-04-01 09113, 2018
9:50 AM
ruaok
out of heap space trying to import a user with 28k listens.
2018-04-01 09129, 2018
9:51 AM
ruaok
I think we need to brainstorm how to refactor the import to not have to do much work on the client
2018-04-01 09146, 2018
9:51 AM
iliekcomputers
first idea that comes to mind is
2018-04-01 09100, 2018
9:52 AM
iliekcomputers
we modify the listenbrainz dumps code to add username to the listen json doc
2018-04-01 09113, 2018
9:52 AM
iliekcomputers
after that create a new dump and load it directly using sc.textFile
2018-04-01 09151, 2018
9:52 AM
ruaok
doh. that's a great idea.
2018-04-01 09103, 2018
9:53 AM
ruaok
given that we're running into this problem, others will have the same problem.
2018-04-01 09105, 2018
9:53 AM
ruaok
let's do that.
2018-04-01 09132, 2018
9:53 AM
iliekcomputers
alright, on it
2018-04-01 09151, 2018
9:53 AM
LordSputnik
2018-04-01 09112, 2018
9:54 AM
LordSputnik
That's the data the client will send to the server for relationships (from the edition editor)
2018-04-01 09128, 2018
9:54 AM
bukwurm
LordSputnik: Ok
2018-04-01 09149, 2018
9:55 AM
bukwurm
LordSputnik: I relationships I have (got through the pg_dump and it's same with the schameSpy diagram on readthedocs) doesn't have linkphrase?
2018-04-01 09159, 2018
9:55 AM
bukwurm
How do I update it?
2018-04-01 09114, 2018
9:56 AM
ruaok
Mineo: well spotted, the import runs faster with that correct. :)
2018-04-01 09123, 2018
9:56 AM
LordSputnik
bukwurm: let me check what's going on
2018-04-01 09128, 2018
9:56 AM
LordSputnik
2018-04-01 09141, 2018
9:56 AM
ruaok
iliekcomputers: we should still proceed with the LB dump change -- that will still make this faster.
2018-04-01 09111, 2018
9:57 AM
ruaok
but now it dies with an assertion errore:
2018-04-01 09112, 2018
9:57 AM
ruaok
2018-04-01 09133, 2018
9:57 AM
bukwurm
2018-04-01 09153, 2018
9:57 AM
LordSputnik
bukwurm: ok, so the relationship thing is fairly easy - that's part of the new relationship editor stuff and not yet applied on the server
2018-04-01 09159, 2018
9:57 AM
LordSputnik
So you'll need to run the migration scrip
2018-04-01 09107, 2018
9:58 AM
bukwurm
LordSputnik: Ok
2018-04-01 09111, 2018
9:58 AM
LordSputnik
2018-04-01 09117, 2018
9:58 AM
LordSputnik
"up.sql" will upgrade you there
2018-04-01 09137, 2018
9:58 AM
bukwurm
LordSputnik: Ok
2018-04-01 09101, 2018
9:59 AM
bukwurm
LordSputnik: Also, one side question.
2018-04-01 09109, 2018
9:59 AM
LordSputnik
And you're right that there's no _editor_entity_visits on the server, that'll need looking into
2018-04-01 09132, 2018
9:59 AM
bukwurm
bookbrainz_test is using same schema and sequences as bookbrainz db
2018-04-01 09159, 2018
9:59 AM
bukwurm
This is causing "insert into "bookbrainz"."relationship_set" ("id") values ($1) returning "id" - duplicate key value violates unique constraint "relationship_set_pkey""
2018-04-01 09101, 2018
10:00 AM
D4RK-PH0ENiX has quit
2018-04-01 09127, 2018
10:00 AM
bukwurm
Any idea how to fix it? I am not well versed with postgres
2018-04-01 09104, 2018
10:01 AM
bukwurm
But I get the notion that sequence of bookbrainz db is pointing to the next value to be inserted with few relations
2018-04-01 09129, 2018
10:01 AM
bukwurm
And as the bookbrainz-data relations (tables) are still empty, this error comes up
2018-04-01 09130, 2018
10:01 AM
LordSputnik
bukwurm: is that during tests?
2018-04-01 09135, 2018
10:01 AM
bukwurm
Yes!
2018-04-01 09139, 2018
10:01 AM
LordSputnik
That usually means that the tables haven't been truncated properly
2018-04-01 09102, 2018
10:02 AM
LordSputnik
That will only happen when we specify an ID when inserting data, and the ID already exists in the table
2018-04-01 09114, 2018
10:02 AM
bukwurm
LordSputnik: Ok
2018-04-01 09119, 2018
10:02 AM
bukwurm
Let me check
2018-04-01 09142, 2018
10:02 AM
LordSputnik
If you rerun the test when the relationship issue is fixed, the truncation stuff should work properly :)
2018-04-01 09147, 2018
10:02 AM
bukwurm
Ok that's written in the err message itself :P
2018-04-01 09104, 2018
10:03 AM
bukwurm
LordSputnik: Ok that great!
2018-04-01 09108, 2018
10:03 AM
bukwurm
On it!
2018-04-01 09109, 2018
10:03 AM
LordSputnik
(we should also change the tests so that truncation doesn't break when something else breaks)
2018-04-01 09124, 2018
10:03 AM
bukwurm
Ok
2018-04-01 09100, 2018
10:04 AM
ruaok
2018-04-01 09124, 2018
10:04 AM
bukwurm
LordSputnik: I'll look into it too. :D
2018-04-01 09140, 2018
10:05 AM
iliekcomputers
2018-04-01 09147, 2018
10:05 AM
github joined the channel
2018-04-01 09147, 2018
10:05 AM
github
[listenbrainz-server] paramsingh opened pull request #397: Add username to listen json docs in dumps (production...usernames-in-dumps)
https://git.io/vxKNM
2018-04-01 09147, 2018
10:05 AM
github has left the channel
2018-04-01 09101, 2018
10:10 AM
bukwurm_
LordSputnik: Probably should update the schemaSpy diagram on the docs. That's a lifesaver for beginners!
2018-04-01 09153, 2018
10:10 AM
LordSputnik
bukwurm_: definitely, that applies to everything on the docs really!
2018-04-01 09155, 2018
10:10 AM
D4RK-PH0ENiX joined the channel
2018-04-01 09125, 2018
10:12 AM
UmkaDK joined the channel
2018-04-01 09148, 2018
10:12 AM
D4RK-PH0ENiX has quit
2018-04-01 09102, 2018
10:13 AM
D4RK-PH0ENiX joined the channel
2018-04-01 09116, 2018
10:15 AM
LordSputnik
bukwurm_: aha! So I think _editor_entity_visits used to exist. Then we restored the database from a dump, and it's not in the dumps (it's personal info), so we lost it
2018-04-01 09129, 2018
10:17 AM
github joined the channel
2018-04-01 09129, 2018
10:17 AM
github
[listenbrainz-server] mayhem closed pull request #397: Add username to listen json docs in dumps (production...usernames-in-dumps)
https://git.io/vxKNM
2018-04-01 09129, 2018
10:17 AM
github has left the channel
2018-04-01 09105, 2018
10:18 AM
LordSputnik
2018-04-01 09132, 2018
10:18 AM
bukwurm
LordSputnik: Ok
2018-04-01 09123, 2018
10:20 AM
akhilesh_ joined the channel
2018-04-01 09124, 2018
10:21 AM
bukwurm
LordSputnik: Does BB-site has functionality to add explorer achievement?
2018-04-01 09130, 2018
10:21 AM
LordSputnik
Yup
2018-04-01 09138, 2018
10:21 AM
LordSputnik
I'm not sure why there was no error
2018-04-01 09153, 2018
10:21 AM
bukwurm
Weird
2018-04-01 09107, 2018
10:22 AM
bukwurm
LordSputnik: Another thing, bb-site uses unminified react
2018-04-01 09130, 2018
10:22 AM
bukwurm
Is it by design?
2018-04-01 09158, 2018
10:22 AM
akhilesh_
LordSputnik: hey :)
2018-04-01 09117, 2018
10:25 AM
bukwurm
2018-04-01 09139, 2018
10:25 AM
ruaok triggers a dump on LB
2018-04-01 09142, 2018
10:25 AM
travis-ci joined the channel
2018-04-01 09143, 2018
10:25 AM
travis-ci
2018-04-01 09143, 2018
10:25 AM
travis-ci has left the channel
2018-04-01 09107, 2018
10:27 AM
LordSputnik
bukwurm: well, it should get minified along with everything else in the build process
2018-04-01 09122, 2018
10:27 AM
LordSputnik
I think that's merged now, maybe not updated on the server
