Feat/twitter client #18
Conversation
to job's client instead of plugin's client
| @@ -1,4 +1,11 @@ | |||
| WHITELISTED_WALLET_PRIVATE_KEY=<whitelisted-wallet-private-key> | |||
| BUYER_WALLET_PRIVATE_KEY=<buyer-wallet-private-key> | |||
There was a problem hiding this comment.
can keep the current envs? we can just use the same whitelisted dev key for all agents
| @@ -1,3 +1,7 @@ | |||
| WHITELISTED_WALLET_PRIVATE_KEY=<whitelisted-wallet-private-key> | |||
| BUYER_WALLET_PRIVATE_KEY=<buyer-wallet-private-key> | |||
There was a problem hiding this comment.
can keep the current one? we just use the same whitelisted dev key
| @@ -32,10 +32,11 @@ def on_new_task(job: ACPJob): | |||
|
|
|||
| # Initialize the ACP client | |||
| acp_client = VirtualsACP( | |||
| wallet_private_key=env.WHITELISTED_WALLET_PRIVATE_KEY, | |||
| wallet_private_key=env.SELLER_WALLET_PRIVATE_KEY, | |||
There was a problem hiding this comment.
don't think this is needed
| if memo.next_phase == ACPJobPhase.COMPLETED: | ||
| print("Evaluating deliverable", job.id) | ||
| job.evaluate(True) | ||
| break | ||
|
|
||
| # Initialize the ACP client | ||
| acp_client = VirtualsACP( | ||
| wallet_private_key=env.WHITELISTED_WALLET_PRIVATE_KEY, | ||
| wallet_private_key=env.EVALUATOR_WALLET_PRIVATE_KEY, |
There was a problem hiding this comment.
don't think this is needed
| acp = VirtualsACP( | ||
| wallet_private_key=env.WHITELISTED_WALLET_PRIVATE_KEY, | ||
| wallet_private_key=env.BUYER_WALLET_PRIVATE_KEY, |
There was a problem hiding this comment.
don't think this is needed
| return memo.content if memo else None | ||
|
|
||
| @property | ||
| def deliverable(self) -> Optional[str]: |
There was a problem hiding this comment.
perhaps better to rename as get_deliverable(self)?
| return memo.content if memo else None | ||
|
|
||
| @property | ||
| def provider_agent(self) -> Optional["IACPAgent"]: |
There was a problem hiding this comment.
perhaps better to rename as get_provider_agent(self)?
| return self.acp_client.get_agent(self.provider_address) | ||
|
|
||
| @property | ||
| def client_agent(self) -> Optional["IACPAgent"]: |
There was a problem hiding this comment.
perhaps better to rename as get_client_agent(self)?
| return self.acp_client.get_agent(self.client_address) | ||
|
|
||
| @property | ||
| def evaluator_agent(self) -> Optional["IACPAgent"]: |
There was a problem hiding this comment.
perhaps better to rename as get_evaluator_agent(self)?
| @property | ||
| def evaluator_agent(self) -> Optional["IACPAgent"]: | ||
| """Get the evaluator agent details""" | ||
| return self.acp_client.get_agent(self.client_address) |
There was a problem hiding this comment.
shouldn't it be "evaluator_address"?
No description provided.