20:52 PM
ijabz_
so please if you have a minute to just check in that mmd change I could finish it off
20:52 PM
ruaok
nice, thats a good deal. :)
20:52 PM
crap. I forgot to check that in. too damn sleepy last night.
20:52 PM
ijabz_
np
20:53 PM
ruaok
comitted.
20:55 PM
aCiD2
ruaok: ping
20:55 PM
ruaok
wail.
20:55 PM
aCiD2
:)
20:55 PM
do you want me to take some of the tasks you've got assigned to you for b1?
20:56 PM
ruaok goes to look
20:56 PM
I'm feeling fairly confident my remaining tasks might not take what the estimate is
20:56 PM
ruaok
the direct search is a task that I may not get to.
20:57 PM
Cd stubs are also not going to happen for b1, but that is not a big deal.
20:57 PM
aCiD2
183?
20:57 PM
ruaok
no.
20:57 PM
aCiD2
oh, 204?
20:57 PM
ruaok
yes.
20:57 PM
aCiD2
cool, i'm going to give you a ship it on the external search then, and I'll take that issue
20:58 PM
ruaok
that involves changing Search.pm to load the data that indexed search provides.
20:58 PM
perfect!
20:58 PM
aCiD2
seems doable :)
20:58 PM
ruaok
the biggest issue there is writing another query for finding releases that belong to recordings.
20:58 PM
thanks, much appreciated.
20:59 PM
aCiD2
I need to spend an hour making sure error pages look a bit more presentable too
20:59 PM
aCiD2 makes a task
20:59 PM
ruaok
I'm glad that external search will go in. then I can focus on WS issues. :)
20:59 PM
I'm making good progress on WS stuffs. The core is there, now its just time to pound out every last little nook and cranny.
20:59 PM
ijabz_
acid2 what is 'External Search' ?
20:59 PM
aCiD2
ijabz_: search.mb
21:00 PM
ruaok
ijabz_: what you and I have been working on.
21:00 PM
indexed search.
21:00 PM
we should actually do a nomenclature fix on that.
21:00 PM
aCiD2
yea sorry, I need to stop calling it external
21:00 PM
ruaok
it should be indexed not direct.
21:00 PM
aCiD2
s/direct/external/
21:00 PM
ijabz_
oh
21:02 PM
ruaok
21:02 PM
aCiD2
that was quick!
21:03 PM
ruaok
I nearly have all the base data of the core entities returning. search is esp quick. it ported real nice from the old version.
21:03 PM
that hard part will be getting all the supporting information for these basic requests.
21:04 PM
aCiD2: wrt to the minor ramifications, feel free to fix them outright when you work on direct search. or, send me mail and I'll clean up after b1
21:04 PM
aCiD2
sure thing
21:05 PM
ijabz_
nice to see the webservice working !
21:05 PM
ruaok nods
21:06 PM
ruaok
all the core entities but work are working. search and mbid access.
21:06 PM
oh, and what is cool about this is that the table that defines all of the possible values for each call type can be used to automatically generate the core docs for the WS. :)
21:06 PM
aCiD2
neat!
21:08 PM
ruaok
21:08 PM
see the num column.
21:08 PM
the first value is the track number.
21:08 PM
the value after the / is the medium number.
21:09 PM
how do we best represent this in a sensical mannger.
21:09 PM
ruaok thinks navap or nikki might also have valuable feedback on this.
21:09 PM
aCiD2
do we need to present it?
21:09 PM
at all?
21:09 PM
nikki
hi
21:09 PM
ruaok
x / y is clearly unintutive.
21:09 PM
hi nikki
21:10 PM
aCiD2 would think just pointing to the release is enough
21:10 PM
nikki
<- kaputt
21:10 PM
ruaok
nikki: bed time or are you damaged goods?
21:10 PM
nikki
long day with lots of train delays and cancellations
21:10 PM
ruaok
aCiD2: I think giving an indication as to what medium we're talking about is nice.
21:10 PM
and we have the data at the ready.
21:11 PM
nikki: where did you go?
21:11 PM
aCiD2
ah, the medium number is interesting, true
21:11 PM
ruaok
is the snow still gripping .de?
21:11 PM
aCiD2
i'm not sure you need to go any further than that
21:11 PM
ruaok
aCiD2: "track x on medium y" would be best.
21:11 PM
navap
aCiD2: Oh..lol that's what it is.
21:11 PM
I've been trying to figure out what "5 / 1" mkeans.
21:11 PM
nikki
I'm back at mum's, it's her birthday in a few days
21:11 PM
ruaok
another thought is to leave out medium number when its 1.
21:11 PM
nikki
at my mum's, even
21:11 PM
ruaok
and only display when its non 1.
21:11 PM
aCiD2
ruaok: right, but you don't have space for that
21:11 PM
ruaok
nikki: ah.
21:12 PM
aCiD2: right.
21:12 PM
so, now what. :)
21:12 PM
and navap proves that this is a problem.
21:12 PM
aCiD2
Personally, I think the clearest way to present it is to have a "Medium #" column, and just use the medium position value
21:12 PM
ruaok
oh duh.
21:12 PM
ijabz_
acid2, agreed
21:12 PM
aCiD2
readable, gives me enough information to make a decision 99% of the time
21:12 PM
ruaok
ok, done.
21:12 PM
aCiD2
:)
21:13 PM
ruaok
21:13 PM
navap
Is the number of releases a recording is on important information?
21:14 PM
aCiD2
no, I'd lose the Num and Tracks columns
21:14 PM
this is just me though
21:14 PM
ruaok
navap: not in this context I think.
21:14 PM
the tracks column is critical for tagging.
21:15 PM
aCiD2
does it show the same recording multiple times for each release then?
21:15 PM
ruaok
and the medium is really handy for letting the user pinpoint the track on the release view.
21:15 PM
yes
21:15 PM
aCiD2
Ok, in that case I'd do this:
21:15 PM
Medium # | Track columns
21:15 PM
ruaok
21:15 PM
aCiD2
and have x | y/z
21:16 PM
x = medium number, y/z track number/total tracks on medium
21:16 PM
ruaok
that sounds fine to me.
21:16 PM
navap, ijabz_?
21:16 PM
navap
I like condensing the number and tracks columns.
21:16 PM
ruaok
though I would swap the order of the two columns.
21:16 PM
navap
+1 on swap
21:16 PM
aCiD2
i'm happy either way, swap away!
21:17 PM
ruaok
21:18 PM
aCiD2
s/Tracks/Track/
21:18 PM
ruaok
refresh
21:18 PM
aCiD2
perfect!
21:18 PM
ijabz_
SO youre losing total tracks on release then
21:18 PM
navap
Can we switch from displaying a score out of 100 to a score out of 10?
21:18 PM
And round to whole number.
21:18 PM
ruaok
ijabz_: we never had that, did we?
21:18 PM
ijabz_
navap, the score is calculated out of 100
21:19 PM
ruaok
navap: we;ve been using 100.
21:19 PM
acid should match the direct search to use 100 as well.
21:19 PM
navap
Sort using the 100 calulation, but display using an integer from 1-10
21:19 PM
aCiD2
I don't think I ever know how many tracks on my releases, only mediums
21:19 PM
ruaok
navap: I'd like to keep 100.
21:19 PM
ijabz_
But total was intended to be release total wasn;t it, i it just wasnt
21:19 PM
ruaok
ok, I'm happy.
21:19 PM
navap
ruaok: Okay.
21:20 PM
ruaok
I'm ready to commit this and return to WS work.
21:20 PM
aCiD2
\ยบ/
21:20 PM
ruaok goes to make it so
21:21 PM
ruaok crosses his fingers after doing an svn up
21:21 PM
navap
ruaok: Can you do something about the alignment of the tracks column?
21:21 PM
ruaok
do what?
21:21 PM
navap
1/4 and 13/15 end up aligned very differently.
21:21 PM
Can you align the first number to the left and the second number to the right?
21:21 PM
That will make it easier to read the numbers at a glance.
21:21 PM
aCiD2
that's hell to do
21:22 PM
ruaok
if you give me the CSS/HTML, I'll do it.
21:22 PM
aCiD2
well, not hell, but too much effort
21:22 PM
:)
21:22 PM
navap
heh okay, let me see what I can do.
21:22 PM
ruaok
but sadly, I must return to the WS if I want to have a snowball's chance in hell to fix it by tomorrow afternoon.
21:22 PM
ijabz_
Column needs to be wide enough to fit in xx/yy at least in one line
21:22 PM
aCiD2
ruaok: get it pushed and then navap can patch it and I'll merge it back in :)
21:22 PM
ruaok
aCiD2: good plan
21:24 PM
navap
Yeah the track column needs to be at least 45px wide, maybe 46px.
21:24 PM
To fix the xx/yy values.
21:27 PM
ruaok
code is committed.
21:27 PM
aCiD2
woohoo
21:29 PM
ruaok got this sudden sense of accomplishment
21:29 PM
ruaok
/home/robert/musicbrainz/mb_server-ngs-svn/script/../lib/MusicBrainz/Server/Controller/Search.pm line 107
21:29 PM
I used your suggestion in the codereview, aCiD2
21:30 PM
aCiD2
We don't have Model::Search.pm
21:30 PM
ruaok
yep