(much less likely to exist than comment:unknown, which already gives 500 results for artists9
2012-02-02 03315, 2012
reosarevok
*)
2012-02-02 03312, 2012
ijabz
null is very techy though isn't it
2012-02-02 03344, 2012
ijabz
What about the fact that we are already using unknown for searching some fields, can I change that or do I have to leave that
2012-02-02 03324, 2012
reosarevok
It is a bit techy, yes
2012-02-02 03336, 2012
reosarevok
But it's not really an "unknown" comment, it's no comment
2012-02-02 03359, 2012
reosarevok
I mean, unknown makes sense for say, country, because everything has one, so if unset, it's unknown
2012-02-02 03301, 2012
ijabz
Good point, important distinction
2012-02-02 03330, 2012
reosarevok
comment:none is less techy
2012-02-02 03337, 2012
reosarevok
Although it gives 4 results
2012-02-02 03338, 2012
ijabz
okay so we could keep 'unknown' for all fields that are lists, indicating there is a value but we don't know it
2012-02-02 03359, 2012
reosarevok
Yeah
2012-02-02 03300, 2012
ijabz
and use a different value for when there is no value, and there may not be one
2012-02-02 03304, 2012
reosarevok
I'd do that
2012-02-02 03330, 2012
reosarevok
and there, I think null or none are the best options. null being techy doesn't matter too much, because these are techy searches
2012-02-02 03348, 2012
ijabz
is 'null' any better than the existing partial solution of '-' that we use in the canto example
2012-02-02 03352, 2012
reosarevok
(I mean, searching for no comment is something you won't probably do unless you know how stuff works)
2012-02-02 03324, 2012
reosarevok
Not sure
2012-02-02 03335, 2012
reosarevok
Let's see what other people think
2012-02-02 03346, 2012
reosarevok
nikki? hrglgrmpf? ocharles? warp?
2012-02-02 03334, 2012
nikki
the value could be "qwerty" for all I care if it means we can actually use it :P
2012-02-02 03338, 2012
nikki
a dash seems to make sense
2012-02-02 03313, 2012
ocharles
warp: I suggest getting yourself a copy of 'Jailer'
2012-02-02 03320, 2012
ocharles
it's basically your web service dumper tool, but polished :)
2012-02-02 03337, 2012
ijabz
I think now have the d distinction I prefer using -, seeing as we can already use that for some of the values
2012-02-02 03315, 2012
ijabz
okay thanks guys
2012-02-02 03331, 2012
nikki
all the others are words and could be things someone wants to search for
2012-02-02 03341, 2012
nikki
whereas a dash is just a symbol that doesn't really have a meaning...
2012-02-02 03356, 2012
nikki
of course, someone might still want to search for a dash on its own, but since it doesn't mean anything, it's less likely
2012-02-02 03352, 2012
ocharles
navap: jailer is something you might also be interested in. As people wanted a 'test dataset' - I'd be ok with generating a dump and putting it on the ftp
2012-02-02 03301, 2012
ocharles
(I think that's the correct way to solve that problem)
so may be ---- (entered as \-\-\-\-) would be better
2012-02-02 03348, 2012
nikki
by that logic, ____ would be even better, since _ doesn't seem to be used at all and certainly not four of them :P
2012-02-02 03326, 2012
nikki
I think I'd prefer just a single one though, multiple ones means you also need to remember exactly how many to enter
2012-02-02 03315, 2012
ijabz
okay I'll leave as is then
2012-02-02 03336, 2012
nikki
oh, and the stats use country:unknown, etc, so before removing those the server should be updated
2012-02-02 03325, 2012
ijabz
sort, what do you mean ?
2012-02-02 03331, 2012
ijabz
s/sort/sorry
2012-02-02 03334, 2012
ijabz
Thinking about reosarevok distinction if we do not know the release group for a release it still exists, but its a freeform field so thinking I will use '-' for that one, although 'unknown' could be deemed correct
2012-02-02 03343, 2012
ijabz
and just keep 'unknown' for fixed lists
2012-02-02 03352, 2012
reosarevok
Hmm
2012-02-02 03359, 2012
reosarevok
You mean the release group type?
2012-02-02 03315, 2012
reosarevok
Or what? I mean, all releases have RGs
2012-02-02 03318, 2012
ijabz
Your right, silly example
2012-02-02 03329, 2012
ijabz
won't happen
2012-02-02 03356, 2012
kurtjx joined the channel
2012-02-02 03352, 2012
nikki joined the channel
2012-02-02 03345, 2012
ijabz
nikki, i didn't understand what you meant 'the stats use country:unknown, etc, so before removing those the server should be updated'
2012-02-02 03311, 2012
reosarevok
I guess that means the script used to generate the stats searches for it
2012-02-02 03328, 2012
reosarevok
So if something is changed, the query made by the stats code needs to change too
2012-02-02 03325, 2012
ijabz
Im not changing the unknown stuff, but would what is this script, never seen it
wellI expect it queries the database, so I don't see the search stuff affects this
2012-02-02 03341, 2012
wpl joined the channel
2012-02-02 03304, 2012
ocharles
ijabz: nikki means that the stats page links to country:unknown, that's all
2012-02-02 03313, 2012
ocharles
it doesn't actually perform that search
2012-02-02 03325, 2012
ocharles
it just helps users see what the 864 releases with 'unknown' country are
2012-02-02 03332, 2012
ocharles
at least, that's how I understand it
2012-02-02 03340, 2012
nikki
yes
2012-02-02 03349, 2012
nikki
and we don't want to break our own links :P
2012-02-02 03350, 2012
ijabz
ok, that won't change
2012-02-02 03342, 2012
ijabz
I don't think… but I don't know how that is done
2012-02-02 03359, 2012
reosarevok
heh
2012-02-02 03302, 2012
reosarevok
864
2012-02-02 03305, 2012
reosarevok
More like 216,085
2012-02-02 03340, 2012
reosarevok
But yeah
2012-02-02 03351, 2012
nikki
how do I reset my github master?
2012-02-02 03305, 2012
Mineo
"reset"?
2012-02-02 03323, 2012
nikki
I edited a file on github so it made a commit to master
2012-02-02 03337, 2012
nikki
but I don't want that, I want master to match mb's master :/
2012-02-02 03320, 2012
Mineo
I think you need to pull from your github respository, git reset --hard HEAD^, git push origin master -f
2012-02-02 03344, 2012
Mineo
(that's not really a nice solution, but I know of no other)
2012-02-02 03359, 2012
ocharles
that is the correct solution
2012-02-02 03304, 2012
ocharles
rewriting history is not meant to be nice :)
2012-02-02 03339, 2012
nikki
what do I need to do to pull from it?
2012-02-02 03339, 2012
Mineo
do you have a clone of your github repository on your computer?
2012-02-02 03343, 2012
nikki
yes
2012-02-02 03357, 2012
Mineo
then change to that directory and git pull origin master
2012-02-02 03316, 2012
Mineo
(if you're on the master branch)
2012-02-02 03314, 2012
nikki
and I guess "origin" is "github" for me
2012-02-02 03326, 2012
nikki
at least I think "origin" is mb's git thing
2012-02-02 03322, 2012
Mineo
indeed
2012-02-02 03315, 2012
Mineo
yikes, internet without dns sucks :(
2012-02-02 03358, 2012
warp
hello!
2012-02-02 03325, 2012
warp
Mineo: luckily google has made dns servers easy to remember.
2012-02-02 03308, 2012
warp
(well, atleast their ipv4 addresses. I don't know their ipv6 addresses yet ;)
2012-02-02 03327, 2012
nikki already uses the opendns ones
2012-02-02 03359, 2012
Mineo
if only their wasn't this firewall between my university and them :/
2012-02-02 03312, 2012
nikki
:/
2012-02-02 03312, 2012
voiceinsideyou joined the channel
2012-02-02 03312, 2012
kurtjx joined the channel
2012-02-02 03342, 2012
kurtjx_ joined the channel
2012-02-02 03345, 2012
reoafk joined the channel
2012-02-02 03305, 2012
nikki
I'm getting errors locally in the release editor
2012-02-02 03316, 2012
nikki
(running master)
2012-02-02 03357, 2012
nikki
"Caught exception in MusicBrainz::Server::Controller::ReleaseEditor::Add->add "Can't use an undefined value as an ARRAY reference at /home/nikki/dev/mbserver/script/../lib/MusicBrainz/Server/Edit/Utils.pm line 152.""
2012-02-02 03320, 2012
nikki
all I did was open the release editor, enter "foo" as the title, "bar" as the artist and click the release duplicates tab :/
2012-02-02 03301, 2012
nikki
but I can't reproduce it on beta or test
2012-02-02 03336, 2012
the_metalgamer joined the channel
2012-02-02 03324, 2012
voiceinsideyou joined the channel
2012-02-02 03339, 2012
MBJenkins
ijabz: #SEARCH-174:Allow searching for when release barcode is not known and when barcode is known to be none.
