{"action":"created","issue":{"url":"https://api.github.com/repos/castorini/Anserini/issues/470","repository_url":"https://api.github.com/repos/castorini/Anserini","labels_url":"https://api.github.com/repos/castorini/Anserini/issues/470/labels{/name}","comments_url":"https://api.github.com/repos/castorini/Anserini/issues/470/comments","events_url":"https://api.github.com/repos/castorini/Anserini/issues/470/events","html_url":"https://github.com/castorini/Anserini/pull/470","id":377238342,"node_id":"MDExOlB1bGxSZXF1ZXN0MjI4MjMxOTYw","number":470,"title":"Adding multi-threads support with multiple params to SearchCollection ","user":{"login":"Peilin-Yang","id":3334391,"node_id":"MDQ6VXNlcjMzMzQzOTE=","avatar_url":"https://avatars3.githubusercontent.com/u/3334391?v=4","gravatar_id":"","url":"https://api.github.com/users/Peilin-Yang","html_url":"https://github.com/Peilin-Yang","followers_url":"https://api.github.com/users/Peilin-Yang/followers","following_url":"https://api.github.com/users/Peilin-Yang/following{/other_user}","gists_url":"https://api.github.com/users/Peilin-Yang/gists{/gist_id}","starred_url":"https://api.github.com/users/Peilin-Yang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Peilin-Yang/subscriptions","organizations_url":"https://api.github.com/users/Peilin-Yang/orgs","repos_url":"https://api.github.com/users/Peilin-Yang/repos","events_url":"https://api.github.com/users/Peilin-Yang/events{/privacy}","received_events_url":"https://api.github.com/users/Peilin-Yang/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-11-05T03:31:43Z","updated_at":"2018-11-07T16:17:38Z","closed_at":null,"author_association":"COLLABORATOR","pull_request":{"url":"https://api.github.com/repos/castorini/Anserini/pulls/470","html_url":"https://github.com/castorini/Anserini/pull/470","diff_url":"https://github.com/castorini/Anserini/pull/470.diff","patch_url":"https://github.com/castorini/Anserini/pull/470.patch"},"body":"In this PR I fundamentally change how `SearchCollection` works.\r\n1. With this PR one can provide multiple params for base models, for example `-b 0.2 0.75` for BM25 or `-mu 200 2000` for QL. One can also provide multiple params for reranking models, at the same time, for example `-rm3.fbDocs 10 20 -rm3.fbTerms 50 100` for RM3. As a result, there are N1*N2*N3*... run files being generated where N1..Nn are the values of the params.\r\n2. Now the `SearchCollection` spawns new threads for all retrievals by default. This, together with the newly introduced `-inmem` option, make a bunch of retrieval less expensive since multiple retrievals need to load the index (potentially in the memory for better multi-threading) once."},"comment":{"url":"https://api.github.com/repos/castorini/Anserini/issues/comments/436681074","html_url":"https://github.com/castorini/Anserini/pull/470#issuecomment-436681074","issue_url":"https://api.github.com/repos/castorini/Anserini/issues/470","id":436681074,"node_id":"MDEyOklzc3VlQ29tbWVudDQzNjY4MTA3NA==","user":{"login":"lintool","id":313837,"node_id":"MDQ6VXNlcjMxMzgzNw==","avatar_url":"https://avatars1.githubusercontent.com/u/313837?v=4","gravatar_id":"","url":"https://api.github.com/users/lintool","html_url":"https://github.com/lintool","followers_url":"https://api.github.com/users/lintool/followers","following_url":"https://api.github.com/users/lintool/following{/other_user}","gists_url":"https://api.github.com/users/lintool/gists{/gist_id}","starred_url":"https://api.github.com/users/lintool/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lintool/subscriptions","organizations_url":"https://api.github.com/users/lintool/orgs","repos_url":"https://api.github.com/users/lintool/repos","events_url":"https://api.github.com/users/lintool/events{/privacy}","received_events_url":"https://api.github.com/users/lintool/received_events","type":"User","site_admin":false},"created_at":"2018-11-07T16:17:38Z","updated_at":"2018-11-07T16:17:38Z","author_association":"MEMBER","body":"High-level comments, for discussion:\r\n\r\n1. Instead of depending on `toString`, maybe use explicit `tag` method?\r\n2. Would it make sense to introduce a new abstraction called `RankingModel` that is a combination of the similarity and the rerankers?\r\n\r\n"}}
{"actor":{"display_login":"lintool"}}