Error in Pinecone batch selection logic

[jamescalam]

Current implementation of pinecone vec db finds the batches using:

# set end position of batch
i_end = min(i + batch_size, len(texts))

link

But the following lines then go on to use a mix of [i : i + batch_size] and [i:i_end] to create batches:

# get batch of texts and ids
lines_batch = texts[i : i + batch_size]
# create ids if not provided
if ids:
    ids_batch = ids[i : i + batch_size]
else:
    ids_batch = [str(uuid.uuid4()) for n in range(i, i_end)]

Fortunately, there is a zip function a few lines down that cuts the potentially longer chunks, preventing an error from being raised — yet I don't think think [i: i+batch_size] should be maintained as it's confusing and not explicit

Raised a PR here #907