Add occurrence count in the json output for search engine #1076
Conversation
|
Note that there will be a conflict with #1067 which we will need to resolve (in whichever PR gets merged last). |
Signed-off-by: Paul-Elliot <peada@free.fr>
src/search/json_index/json_search.ml
Outdated
| ("direct", `Float (float_of_int occ.Odoc_occurrences.Table.direct)); | ||
| ("indirect", `Float (float_of_int occ.indirect)); | ||
| ] | ||
| | None -> `Null |
There was a problem hiding this comment.
Nit-picking but it would look more consistent to me if we had 0 for both counters here (see the output in the tests).
There was a problem hiding this comment.
What about "no 'occurrences' field at all"?
I'm not very keen on choosing a wrong value when it was not possible to compute one because the table was not given...
There was a problem hiding this comment.
Well, at least if we don't put this field at all it reduces the diff of this PR!
There was a problem hiding this comment.
With 6205f97, if the occurrence table is not given, the field is not included.
However, if the occurrence table is given, but does not contain the occurrence, a use of 0 is counted.
Indeed, that removed that the diff for search without occurrences.
Thanks for the comment, I think that's better now!
Signed-off-by: Paul-Elliot <peada@free.fr>
This adds the occurrence count to each element of the json index Signed-off-by: Paul-Elliot <peada@free.fr>
Signed-off-by: Paul-Elliot <peada@free.fr>
Signed-off-by: Paul-Elliot <peada@free.fr>
panglesd
force-pushed
the
occurrences-in-index
branch
from
6205f97
to
69306f6
Compare
Signed-off-by: Paul-Elliot <peada@free.fr>
Signed-off-by: Paul-Elliot <peada@free.fr>
Co-authored-by: Guillaume "Liam" Petiot <guillaume@tarides.com>
Since #972 there is a
compile-indexcommand which outputs a json index containing all searchable entries. The intended purpose of this json index is to pass it to a search engine.Since #976 there is also a
count-occurrencescommand which creates a table containing the number of occurrences for each identifier.This PR makes the
compile-indexbe able to use the information fromcount-occurrence. It adds a--occurrencesargument to get the path to the table.The first commit isolate the occurrence code in its own library.
The second commit implements the
--occurrencesargument of ``compile-index`.