Skip to content
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

Fix Update API Key Bug #272

Merged
merged 1 commit into from
Oct 10, 2024
Merged

Fix Update API Key Bug #272

merged 1 commit into from
Oct 10, 2024

Conversation

thiago-aixplain
Copy link
Collaborator

Make the update API Key functionality to work.

@ikxplain ikxplain merged commit b13c21d into development Oct 10, 2024
@thiago-aixplain thiago-aixplain deleted the BUG-183-updateAPIKey branch October 10, 2024 15:28
thiago-aixplain added a commit that referenced this pull request Oct 10, 2024
* Fix Update API Key Bug (#272)
thiago-aixplain added a commit that referenced this pull request Oct 29, 2024
Co-authored-by: xainaz <zaina.abushaban@gmail.com>
Co-authored-by: Thiago Castro Ferreira <thiago@aixplain.com>

* BUG-177: Fixed pipeline validation (#262)

* BUG-177: Fixed pipeline validation

* BUG-177 rephrased exception message

* Get usage limit (#264)

* Update SDK version (#266)

* Eng 739 get api key (#268)

* Get usage limit

* Get usage limit of an api key service

* Fix asset limits

* Fixing usage limit services

* Fix Update API Key Bug (#272)

* Eng 735 ai xplain sdk improve error log messages (#271)

* Improve error log for: Benchmark, Corpus, Dataset, Model, Pipeline

* Fixed issue + Added test

* Added required changes to error prompts

* Small improvements

---------

Co-authored-by: xainaz <zaina.abushaban@gmail.com>
Co-authored-by: Thiago Castro Ferreira <thiago@aixplain.com>

* Max tokens and iterations in agents/teams (#276)

* Update model running endpoints from v1 to v2 (#275)

* Update model running endpoints from v1 to v2

* Support v1/v2 model serving endpoints

* Adding the details in case of validation error

* Adding one more test for asynchronous method in new endpoint

* Setting error details and treating when data is null

* Adding missing import in unit tests for models

* Eng 711 new model endpoints (#278)

* Update model running endpoints from v1 to v2

* Support v1/v2 model serving endpoints

* Adding the details in case of validation error

* Adding one more test for asynchronous method in new endpoint

* Setting error details and treating when data is null

* Adding missing import in unit tests for models

* Having endpoint v1 by default and setting v2 on the fly

* Group of Improvements in API Key CRUD (#277)

* assetLimits -> assetsLimits

* Update api key usage limit service

* Remove option for asset ID filtering in the URLs for getting usage

* Update version

* Full response on synchronous model execution

* Get API Key service

* Change the name to daily count limits

* Explicit point to the error field when it exists

* Validate api key before updating it

* Create Setters for token and request limits

* Bug 149 - Path validation removed and decision node output param handling revisited (#260)

* Changed function to required field (#283)

* Changed function to required feild

* added function assert to list

* fixed list function errors

* Fixing the listing tests

---------

Co-authored-by: Thiago Castro Ferreira <thiago@aixplain.com>

* BUG-206: Fixed passthrough parameter reflection to next node (#284)

* Get model description (#286)

---------

Signed-off-by: mikelam-us-aixplain <michael.lam@aixplain.com>
Signed-off-by: mikelam-us <michael.lam@aixplain.com>
Signed-off-by: Michael Lam <michael.lam@aixplain.com>
Signed-off-by: root <root@ip-172-31-53-203.ec2.internal>
Co-authored-by: Lucas Pavanelli <86805709+lucas-aixplain@users.noreply.github.com>
Co-authored-by: Thiago Castro Ferreira <thiagocastroferreira@ip-192-168-0-4.ec2.internal>
Co-authored-by: mikelam-us-aixplain <131073216+mikelam-us-aixplain@users.noreply.github.com>
Co-authored-by: ikxplain <88332269+ikxplain@users.noreply.github.com>
Co-authored-by: Thiago Castro Ferreira <thiagocastroferreira@ip-192-168-0-3.ec2.internal>
Co-authored-by: Shreyas Sharma <85180538+shreyasXplain@users.noreply.github.com>
Co-authored-by: Thiago Castro Ferreira <thiagocastroferreira@Thiagos-MacBook-Pro-2.local>
Co-authored-by: Thiago Castro Ferreira <thiagocastroferreira@ip-192-168-0-126.ec2.internal>
Co-authored-by: Lucas Pavanelli <lucas.pavanelli@aixplain.com>
Co-authored-by: kadirpekel <kadirpekel@gmail.com>
Co-authored-by: kadir pekel <kadirpekel@ip-192-168-178-35.ec2.internal>
Co-authored-by: root <root@ip-172-31-53-203.ec2.internal>
Co-authored-by: Zaina Abu Shaban <zaina.abushaban@aixplain.com>
Co-authored-by: xainaz <zainaschaban@gmail.com>
Co-authored-by: xainaz <zaina.abushaban@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants