Shorter model name params

[bborn]

Fixes #820:

• renames chat_completion_model_name to chat_model
• renames completion_model_name to completion_model
• renames embeddings_model_name to embedding_model

This is a breaking change, and I updated the changelog and bumped the version to reflect that.

[qarol]

What do you think to release new versions following Semantic Versioning https://semver.org? If the release introduces breaking change, then bumping major version should help to catch potential problems earlier.

[bborn]

@qarol you're probably right - a breaking change should be a MAJOR version change according to server. @andreibondarev I'll leave that decision to you though...

[andreibondarev]

@bborn Quick thought -- maybe since the 3 methods are called:

def embed
def complete
def chat

we should call the defaults:

:embed_model
:complete_model
:chat_model

What do you think?

[bborn]

@andreibondarev I think that makes sense. Updated.

My only concern is that complete_model just sounds kind of ... weird to me. But it's consistent with the method names so probably ok.

[andreibondarev]

What do you think to release new versions following Semantic Versioning https://semver.org? If the release introduces breaking change, then bumping major version should help to catch potential problems earlier.

@qarol I love that! Given that we're pre-1.0.0, do you suggest that we keep incrementing minor versions, 0.18.0, 0.19.0, 0.20.0, etc.?

[andreibondarev]

@bborn Great PR! Thank you!

[qarol]

What do you think to release new versions following Semantic Versioning https://semver.org? If the release introduces breaking change, then bumping major version should help to catch potential problems earlier.

@qarol I love that! Given that we're pre-1.0.0, do you suggest that we keep incrementing minor versions, 0.18.0, 0.19.0, 0.20.0, etc.?

@andreibondarev question is, what things blocks from releasing 1.0.0 ;) For pre-1.0.0 it's acceptable to bump just minor version no matter if it's a breaking change or new feature. For bugfixes increasing patch number should be enough IMO