Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a proof of concept to serialize a vector without indices in the query string
user_ids[]=100&user_ids[]=101
. The default is using indicesuser_ids[0]=100&user_ids[1]=101
. The structure without the indices is used by ie. the OpenAI API.I added a new method
to_string_config(input, config)
to be able to serialize with a custom configuration. The existing methodto_string(input)
will work the same as before and will use the default config (using indices).The new config currently only has the field
use_indices
of type boolean - but can be expanded later if needed.I'm not really a fan of using two different methods, do you have any suggestion on how you would like this to work? @samscott89
As there is no standard for this type of serialization - if you do not want to support this, feel free to close it. Thanks.