-
Notifications
You must be signed in to change notification settings - Fork 8
Release: AWS SDK for Python v0.1.1 #23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 3 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,6 +2,18 @@ | |
|
|
||
| ## Unreleased | ||
|
|
||
| ## v0.1.1 | ||
|
|
||
| ### API Changes | ||
| * New stop reason (`MODEL_CONTEXT_WINDOW_EXCEEDED`) for Converse and ConverseStream. | ||
|
|
||
| ### Enhancements | ||
| * Improvements to the underlying AWS CRT HTTP client result in a signifigant decrease in CPU usage. Addresses [aws-sdk-python#11](https://github.com/awslabs/aws-sdk-python/issues/11). | ||
|
|
||
| ### Dependencies | ||
|
|
||
| * **Updated**: `smithy_http[awscrt]` from `~=0.1.0` to `~=0.2.0`. | ||
|
|
||
| ### Breaking | ||
| - Removed unused `serialize.py` and `deserialize.py` modules. | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Were these being surfaced publicly through an
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. No, the modules exist in v0.1.0 but aren't used by anything. I mainly called it out since someone could technically be doing
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yeah that's fine. I'm asking at a higher level about our public contract with this. It's implying we may have been expecting someone to do this. We handle these cases in boto3 because we have a very fuzzy contract. We don't want to carry that forward in this product. It should be extremely clear what is a public interface and then we treat everything else as private.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Oh I see, I think you saying these should have originally been |
||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,3 @@ | ||
| # Code generated by smithy-python-codegen DO NOT EDIT. | ||
|
|
||
| __version__: str = "0.1.0" | ||
| __version__: str = "0.1.1" |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,6 @@ | ||
| # Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved. | ||
| # SPDX-License-Identifier: Apache-2.0 | ||
| __version__ = "0.1.0" | ||
| __version__ = "0.1.1" | ||
|
|
||
|
|
||
| # TODO: Consider adding relative imports for services from the top level namespace? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit; we won't be getting these kinds of updates from the normal model releases. We should consider if we want to keep this to using the "official" service team release notes rather than writing our own.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The "official" service team release note is
New stop reason for Converse and ConverseStream. I added theMODEL_CONTEXT_WINDOW_EXCEEDEDpart to provide clarity. I'll update to use the exact service note thoughReference: https://github.com/boto/botocore/blob/develop/CHANGELOG.rst#L252
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Addressed in d0f4286