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

ENG-1332: Started implementing new aixplain programmatic api #356

Merged
merged 40 commits into from
Feb 4, 2025

Conversation

kadirpekel
Copy link
Collaborator

No description provided.

Copy link
Collaborator

@thiago-aixplain thiago-aixplain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

General comment: Please extend the functional tests for version 2 of the SDK.

def __init__(
self,
api_key: str,
backend_url: str = "https://dev-platform-api.aixplain.com",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Set production env by default

@@ -353,3 +353,6 @@ def deploy(self) -> None:
assert self.status != AssetStatus.ONBOARDED, "Agent is already deployed."
self.status = AssetStatus.ONBOARDED
self.update()

def __repr__(self):
return f"Agent(id={self.id}, name={self.name})"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add the status as well

return f"<Model: {self.name} by {self.supplier['name']}>"
except Exception:
return f"<Model: {self.name} by {self.supplier}>"
return f"Model(id={self.id}, name={self.name}, supplier={self.supplier})"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add the function

aixplain/v2/agent.py Show resolved Hide resolved
aixplain/v2/model.py Show resolved Hide resolved
@@ -353,3 +353,6 @@ def deploy(self) -> None:
assert self.status != AssetStatus.ONBOARDED, "Agent is already deployed."
self.status = AssetStatus.ONBOARDED
self.update()

def __repr__(self):
return f"Agent(id={self.id}, name={self.name}, function={self.function})"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agent has no function properly saud. Please set the status instead

kadirpekel and others added 5 commits January 21, 2025 15:20
* Adding Wallet on v2

* Adding API Key

* Data asset in v2

* Fixed create in api key and dataset

* Fixes in corpus, dataset and api key new factories

---------

Co-authored-by: xainaz <[email protected]>

return PipelineFactory.init(name, api_key=api_key)

@classmethod
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The 3 following methods can be removed from the Pipeline class

@thiago-aixplain thiago-aixplain merged commit c546351 into development Feb 4, 2025
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.

3 participants