Re: [PR] Pass custom similarity function to similarityToQueryVector API [lucene]

2024-04-16 Thread via GitHub
github-actions[bot] commented on PR #13187: URL: https://github.com/apache/lucene/pull/13187#issuecomment-2060105283 This PR has not had activity in the past 2 weeks, labeling it as stale. If the PR is waiting for review, notify the d...@lucene.apache.org list. Thank you for your contributi

Re: [PR] Pass custom similarity function to similarityToQueryVector API [lucene]

2024-04-02 Thread via GitHub
benwtrent commented on PR #13187: URL: https://github.com/apache/lucene/pull/13187#issuecomment-2031759399 I am sorry this work has stalled. But I have been iterating on https://github.com/apache/lucene/pull/13200 for a week now. Its getting to a palatable place. -- This is an automated

Re: [PR] Pass custom similarity function to similarityToQueryVector API [lucene]

2024-04-01 Thread via GitHub
github-actions[bot] commented on PR #13187: URL: https://github.com/apache/lucene/pull/13187#issuecomment-2030835653 This PR has not had activity in the past 2 weeks, labeling it as stale. If the PR is waiting for review, notify the d...@lucene.apache.org list. Thank you for your contributi

Re: [PR] Pass custom similarity function to similarityToQueryVector API [lucene]

2024-03-18 Thread via GitHub
shubhamvishu commented on PR #13187: URL: https://github.com/apache/lucene/pull/13187#issuecomment-2004568222 @benwtrent So should we instead wait for the pluggability support and discard this for now? or Is it possible to go forward with this? > What makes this PR doubly worrying is

Re: [PR] Pass custom similarity function to similarityToQueryVector API [lucene]

2024-03-18 Thread via GitHub
benwtrent commented on PR #13187: URL: https://github.com/apache/lucene/pull/13187#issuecomment-2004052748 > Though I'm not sure if this change conflicts with or makes things difficult for the ongoing efforts to have pluggability (maybe @benwtrent would be interested in sharing his thoughts

Re: [PR] Pass custom similarity function to similarityToQueryVector API [lucene]

2024-03-18 Thread via GitHub
shubhamvishu commented on PR #13187: URL: https://github.com/apache/lucene/pull/13187#issuecomment-2003991803 Thanks for the review @msokolov! The idea to make it pluggable seems relevant and interesting. Currently it was not possible to use any custom vector similarity function other than

Re: [PR] Pass custom similarity function to similarityToQueryVector API [lucene]

2024-03-18 Thread via GitHub
msokolov commented on PR #13187: URL: https://github.com/apache/lucene/pull/13187#issuecomment-2003799323 There is some discussion how to make similarities more pluggable https://github.com/apache/lucene/issues/13182 that seems relevant. Part of the idea there is to accept ordinal values ra

[PR] Pass custom similarity function to similarityToQueryVector API [lucene]

2024-03-17 Thread via GitHub
shubhamvishu opened a new pull request, #13187: URL: https://github.com/apache/lucene/pull/13187 ### Description This PR allows passing a custom vector similarity function to DVS implementations of `VectorSimilarityValuesSource` as opposed to current behaviour which by default only a