[bookbrainz-site] MonkeyDo opened pull request #549 (master…snyk-upgrade-7c07eebc0481fcb385cc7232d7696f2d): [Snyk] Upgrade @babel/runtime from 7.10.1 to 7.12.5 https://github.com/bookbrainz/bookbrainz-site/pul…
if you use "inc=random" you should get an error (for now at least)
2021-01-22 02238, 2021
Mr_Monkey
reosarevok: You just need the right tool that will take an image and export all the icons you can imagine in one go.
2021-01-22 02239, 2021
Mr_Monkey
Couldn't find the one I used for BB, but this one looks neat and even has reporting :) :https://realfavicongenerator.net/favicon_checker?protocol=http&site=www.musicbrainz.org
2021-01-22 02244, 2021
reosarevok
We do have inc=media
2021-01-22 02255, 2021
reosarevok
Documented
2021-01-22 02234, 2021
Mr_Monkey
You'll need to make sure the image you give it is 1) high quality 2) the right —square— format and 3) with the background color you want (I guess white)
2021-01-22 02240, 2021
reosarevok
Funnily, inc=medium seems to give the same results? :D
2021-01-22 02231, 2021
Mr_Monkey
reosarevok: Tell you what, since I'm already checking it out, do you want me to send you a zip file containing all the finished icons?
2021-01-22 02250, 2021
yvanzo
Mr_Monkey: some tools have a small rendering issue with LB logo
So I guess it's not documented because it's deprecated
2021-01-22 02212, 2021
MagnusSvensson[m has quit
2021-01-22 02207, 2021
yvanzo
_lucifer: there are small issues such as making search results to comply with lookup/browse results, medium issues as adding support for collections, large issues such as porting to python 3.
2021-01-22 02228, 2021
reosarevok
Anyway. Re: the counts, we should probably try to be consistent, so either have them on both search and lookup/browse for JSON, or on neither. Removing is more dangerous than adding, but at the same time I'm not sure how useful this data is... :/
2021-01-22 02245, 2021
BrainzGit has quit
2021-01-22 02242, 2021
yvanzo
_lucifer: there are other possible improvements for continuous integration (e.g. by testing sir against mbsssss), and documentation (e.g. adding more details about search fields).
2021-01-22 02222, 2021
yvanzo
reosarevok: So adding a duplicate JSON element named "media-count"?
2021-01-22 02241, 2021
reosarevok
Might be best. The same is probably true of all other count attributes/elements, FWIW
2021-01-22 02254, 2021
BrainzGit joined the channel
2021-01-22 02257, 2021
reosarevok
bitmap: ^ when you're back, I'd like an opinion on that too
2021-01-22 02213, 2021
reosarevok
yvanzo: if you have some smaller stuff that might not require setting up the whole infrastructure locally to test, I could also give a hand
2021-01-22 02219, 2021
reosarevok
With search stuff I mean
2021-01-22 02237, 2021
reosarevok
Documentation might be a thing, if you have a clear idea of what's missing
2021-01-22 02213, 2021
reosarevok
I do have a couple free hours today and no clear ticket to do next
2021-01-22 02229, 2021
yvanzo
reosarevok: I found that https://musicbrainz.org/doc/Indexed_Search_Syntax is incomplete as in: 1) the expected values for search fields is not always documented, 2) some search fields are just missing.
2021-01-22 02243, 2021
reosarevok
Do you have a couple examples?
2021-01-22 02201, 2021
reosarevok
I can look into that. Do we have a good file with all possible search fields on github? :)
2021-01-22 02249, 2021
yvanzo
for example, the "lang" search field for Work should be ISO-639-3
2021-01-22 02204, 2021
yvanzo
there is a script to generate the list of search fields in "sir" repo
Also, is there a specific ticket for updating these docs, or should I make one? :)
2021-01-22 02254, 2021
yvanzo
it looks much better, thanks, there is no ticket afaik
2021-01-22 02235, 2021
yvanzo
I did some research about how to document search fields directly in mbsssss but it seems uneasy to do/maintain.
2021-01-22 02258, 2021
reosarevok
Ok, I'll add a ticket (so I can track the work better) and keep doing other entities
2021-01-22 02259, 2021
reosarevok
Well, doing it directly in mbssssssssssssss I guess would imply having a JSON file where you map a key (probably "entity/field") to a description string.
2021-01-22 02219, 2021
reosarevok
And then have the python script take the description from the file
2021-01-22 02231, 2021
yvanzo
Ideally, I'd like to put that directly in a "summary" attribute for each search field defined in */conf/schema.xml but I did not test if that would break Solr.
2021-01-22 02210, 2021
reosarevok
Well, I'll update the summaries for now and we can always move them to the schema later
2021-01-22 02217, 2021
yvanzo
+1
2021-01-22 02254, 2021
reosarevok
Also, is there a ticket to search more stuff by MBIDs?
2021-01-22 02212, 2021
reosarevok
It would be good to have area search that doesn't depend on English names that might change, at the very least
2021-01-22 02249, 2021
yvanzo
+1
2021-01-22 02200, 2021
yvanzo
there is no ticket about that so far
2021-01-22 02208, 2021
_lucifer
ruaok: i read about that but didn't quite understand what's going on. (elasticsearch)
2021-01-22 02223, 2021
yvanzo
searching by MBID is available for some fields but not for area (but in area search index)
2021-01-22 02225, 2021
ruaok
_lucifer: is it clear to you now?
2021-01-22 02215, 2021
yvanzo
reosarevok: so adding search fields "aid", "beginaid", "endaid", right?
2021-01-22 02222, 2021
reosarevok
Yeah
2021-01-22 02237, 2021
_lucifer
ruaok: not yet but still reading up the link you shared above.
2021-01-22 02247, 2021
yvanzo
in general, there is no problem with adding search fields, but to recording search index (which is not related to area, so it's ok)
2021-01-22 02253, 2021
reosarevok
Eventually probably the same for gender and type for example, but I guess we would need to document those gids properly later :p
2021-01-22 02208, 2021
reosarevok
Why is recording a problem? Just because it's huge and would require a full reindex?
2021-01-22 02210, 2021
yvanzo
(the problem with adding data to recording search index is: it's too large already)
2021-01-22 02223, 2021
reosarevok
Recording seems to have nothing of the sort, so that's ok