-
Notifications
You must be signed in to change notification settings - Fork 5
Agent Mode #116
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
base: main
Are you sure you want to change the base?
Agent Mode #116
Conversation
Create service
"home_chain": "gnosis", | ||
"configurations": { | ||
"gnosis": { | ||
"agent_id": 37, |
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.
This should be updated with a different agent_id.
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.
Do agent_id matter here? If so, what should it be?
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.
To properly associate the services with the real usage via events, ie mech-client agent, it is necessary to to mint a new agent on L1.
Before doing that, we need:
- A Package hash (e.g. https://gateway.autonolas.tech/ipfs/bafybeihae4lqokbjzcu5reb4w7cf5xy6fuibkejjqw24chdraqqlpbvpwu)
- a basic component (e.g. https://marketplace.olas.network/ethereum/components/134)
Please check also with Jose.
For the time being, can we avoid to use mech agent id? Please consider to temporely use agent id = 1.
config/mech_client.json
Outdated
@@ -0,0 +1,25 @@ | |||
{ | |||
"name": "Mech Client (agent mode)", | |||
"hash": "bafybeigb5a6ptwtfx5wpnvqv6mp5jg6dpjt5scgjgnq74d4mri2i7vntbm", |
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.
This should be updated with a different hash.
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.
Yes, as per the spec doc, we still have to figure out a dummy agent and service for mech client
"threshold": 1, | ||
"use_mech_marketplace": false, | ||
"cost_of_bond": 1, | ||
"fund_requirements": { |
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.
In the future we should we fund the wallet based on the kind of mech they are willing to interact.
What I mean is, assuming that we want to interact with an olas mech, we fund the safe with olas other then some small native amount
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.
created a ticket for the above comments
mech_client/cli.py
Outdated
ctx.ensure_object(dict) | ||
ctx.obj["agent_mode"] = agent_mode | ||
|
||
if agent_mode: |
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.
as we already discussed this will be the default mode
we also need gas estimation |
{ | ||
"name": "Mech Client (agent mode)", | ||
"hash": "bafybeiczb3blzzvwjnfqmyphve3qudad5mnfjrv7yvrthsiofw7alj437q", | ||
"description": "The mech_client service to allow interacting with on-chain mechs", |
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.
I copied the description from the service.
config/mech_client.json
Outdated
@@ -0,0 +1,25 @@ | |||
{ | |||
"name": "Mech Client (agent mode)", | |||
"hash": "bafybeiczb3blzzvwjnfqmyphve3qudad5mnfjrv7yvrthsiofw7alj437q", |
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.
updated with new service - it won't work until the agent is minted.
No description provided.