Feat/sagemaker llms dev #257
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* latest * remove mistral 7b standard
…#249) * refactor no functional changes * refactor * change names to follow guidance * formatting * further renaming: * wip * latest wip - composite alarms not compatible with scaling policies * latest wip * latest wip * likely all working * formatting
* extend to 1hr * current status * latest * cleanup unecessary commentary * remove commented out code
* Deactivating the code to move logs to S3 buckets * Retaining the module functionality as it may be revived in future to help us get user data associated with the log (perhaps?) which can be used for log retrieval for ensuring we get latest logs specific to user-accounts
* fix: remove all endpoints except phi * feat: put all sagemaker resources behind feature flag * feat: put all sagemaker resources behind feature flag * fix: formatting * fix: remove unnccessary files * correct missing parts * correct 1 to true * resolve issues discovered on terraform apply --------- Co-authored-by: Isobel Daley <[email protected]> Co-authored-by: Isobel Daley <[email protected]>
* remove all references to unused code * remove more code
* refactor no functional changes * refactor * change names to follow guidance * formatting * further renaming: * wip * fix: remove all endpoints except phi * latest wip - composite alarms not compatible with scaling policies * latest wip * latest wip * likely all working * formatting * wip * basic functionality present - just needs to look nicer * formatting * feat: put all sagemaker resources behind feature flag * feat: put all sagemaker resources behind feature flag * fix: formatting * fix: remove unnccessary files * aesthetic teams post * final state * resolve * correct missing parts * remove cloudwatch logs moving * remove all references to unused code * correct 1 to true * correct 1 to true * resolve issues with permissions * resolve issues discovered on terraform apply * remove logs code * remove more code * remove mistaken addition of variables --------- Co-authored-by: Isobel Daley <[email protected]> Co-authored-by: Isobel Daley <[email protected]>
Feat/sagemaker dev merge conflicts
fix: address python linting errors
fix: address python linting errors
fix: address python linting errors
fix: address python linting errors
fix: address python linting errors
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Bringing latest SageMaker development changes into the main SageMaker branch for review.
Includes: