in reply to Please review documentation of my AI::Embedding module
Typos: compatator; tyhe; chargable (should be chargeable [1]); "will random" => "will be random"; ACKNOWLEDGEMENTS section misses a trailing full stop.
Interface: returning the HTTP::Tiny response object on failure of various methods means you always need to call success (or error) to know how to look at the response. If I were using this, I'd rather it returned something more easily testable such as undef - the HTTP::Tiny response could be included in the error() return value, or be provided by a separate http_error method.
Interface: it seems strange to have the comparator be built in to the object. To me it would make more sense to have the comparator method return a subref (and remove it as a new option).
Text: "It requires the 'key' parameter." is not needed, the following sentence already says this (and it is reiterated with the individual parameter listing). Use of the word "homogeneous" is odd: I'm not sure in what way the string form is more "homogeneous" than the array form. The text generally seems to assume that the only use for this interface is to rank search matches, I'm not sure how appropriate that is. More generally I'm unsure about calling the class "Embedding": if I correctly understand what I've read, an instance of this class is not an embedding but an embedder - it is something capable of providing embeddings.
Hope this helps. :)
[1] I use the phrase "city centre; cat cut cot" to remind me that in almost all English words, a 'c' or 'g' is hard when followed by 'i' or 'e', but soft when followed by 'a', 'u' or 'o'. Thus "chargable" can't be right - it would have to be pronounced with a hard 'g', so it gets an extra 'e' inserted to make it soft. Similarly "manageable" and "enforceable"; but "reproducible" is fine.
|
---|
Replies are listed 'Best First'. | |
---|---|
Re^2: RFC - Documentation Review
by Bod (Parson) on Jun 02, 2023 at 23:05 UTC | |
by hv (Prior) on Jun 03, 2023 at 01:38 UTC | |
by Bod (Parson) on Jun 03, 2023 at 11:31 UTC | |
by hv (Prior) on Jun 03, 2023 at 14:12 UTC | |
by afoken (Chancellor) on Jun 03, 2023 at 12:23 UTC | |
Homogeneous (was: Re^2: RFC - Documentation Review)
by Bod (Parson) on Jun 11, 2023 at 13:12 UTC | |
by kcott (Archbishop) on Jun 11, 2023 at 14:37 UTC |