[14:31:39] \o [14:39:00] doh, broke deepcat :( [14:59:34] hmm, somehow i'm not even constructing a manual query that does the appropriate terms query...might need some coffee first [15:08:02] yea...when doing an `s/_/ /g` to test i removed the underscore from `lowercase_keyword`, so the system works fine we just constructed the wrong terms query. [15:37:41] ebernhardson: was that related to any of the deepcat tickets? [16:01:41] pfischer: yea there is a ticket with a patch attached, but the patch is a partial revert [16:01:52] https://phabricator.wikimedia.org/T414859 [16:02:10] the revert also wouldn't quite work right as it has the expanded 5k limit, just working up some tests now the fix is easy enough [17:21:19] not sure whos arround, could use a review. Cindy should vote in a few minutes: https://gerrit.wikimedia.org/r/c/mediawiki/extensions/CirrusSearch/+/1228158 [17:22:18] doh, actually i have it backwards :( but easy fix... [19:08:06] * ebernhardson has wired up neural queries and semantic profiles to cirrus...except somehow the returned query is completely empty :( But getting somewhere [19:33:04] * ebernhardson is separately unsure how to enfore no offsets and a restricted limit...right now the ft query builder doesn't get that info [21:29:19] so many questions...will have to ponder :P I now have it dispatching to CONTEXT_SEMANTIC, but that still uses the fulltext query builder profiles. The dispatcher that chooses CONTEXT_SEMANTIC requires engine_autoselect, but might have to ponder how to not have that ft profile available in normal search [21:29:50] pondering if it would be appropriate to have separate query building profiles for fulltext vs semantic, intead of just different defaults, but not sure yet. [21:33:45] also have to ponder what to do about the default routing...the profile needs a model_id, it's not clear if cirrus should ship with a semantic search profile. In MLR we avoided that by only adding the rescore profiles, maybe we can only add semantic routes if profiles exist or some such [21:34:07] only adding rescore profiles in operations/mediawiki-config, no defualts [22:25:54] ryankemper patch up for adding DC labels to the probe alerts: https://gerrit.wikimedia.org/r/c/operations/alerts/+/1229203