Biconomy integration #277

nerfZael · 2024-06-24T19:10:39Z

No description provided.

dOrgJelli

Incredible!

…o nerfzael/biconomy

…xecute

nerfZael · 2024-06-28T16:58:15Z

/workflows/benchmarks agents/token/send,agents/token/test_swap.py 3

github-actions · 2024-06-28T16:58:27Z

Finished benchmarks
Download artifacts

Test Run Summary

Run from: ./autotx/tests/agents/token/send,./autotx/tests/agents/token/test_swap.py
Base path: autotx/tests/agents/token/
Iterations: 3
Total Cost: $0.26
Total Success Rate (%): ${\color{red} \LARGE \texttt {97.22} \large \texttt { (97/-3)} }$

Detailed Results

Test Name	Success Rate (%)	Passes	Fails	Avg Time	Avg Cost
`send/test_send.py::test_send_erc20`	${\color{none} \large \texttt {100} \normalsize \texttt {} }$	${\color{none} \large \texttt {3}}$	${\color{none} \large \texttt {0}}$	20s	$0.00
`send/test_send.py::test_send_erc20_parallel`	${\color{none} \large \texttt {100} \normalsize \texttt {} }$	${\color{none} \large \texttt {3}}$	${\color{none} \large \texttt {0}}$	23s	$0.01
`send/test_send.py::test_send_eth_multiple`	${\color{none} \large \texttt {100} \normalsize \texttt {} }$	${\color{none} \large \texttt {3}}$	${\color{none} \large \texttt {0}}$	30s	$0.01
`send/test_send.py::test_send_native`	${\color{none} \large \texttt {100} \normalsize \texttt {} }$	${\color{none} \large \texttt {3}}$	${\color{none} \large \texttt {0}}$	14s	$0.00
`send/test_send.py::test_send_native_sequential`	${\color{none} \large \texttt {100} \normalsize \texttt {} }$	${\color{none} \large \texttt {3}}$	${\color{none} \large \texttt {0}}$	18s	$0.01
`test_swap.py::test_swap_complex_1`	${\color{red} \large \texttt {67} \normalsize \texttt {(-33)} }$	${\color{red} \large \texttt {2}}$	${\color{red} \large \texttt {1}}$	34s	$0.01
`test_swap.py::test_swap_complex_2`	${\color{none} \large \texttt {100} \normalsize \texttt {} }$	${\color{none} \large \texttt {3}}$	${\color{none} \large \texttt {0}}$	34s	$0.01
`test_swap.py::test_swap_multiple_1`	${\color{none} \large \texttt {100} \normalsize \texttt {} }$	${\color{none} \large \texttt {3}}$	${\color{none} \large \texttt {0}}$	33s	$0.03
`test_swap.py::test_swap_multiple_2`	${\color{none} \large \texttt {100} \normalsize \texttt {} }$	${\color{none} \large \texttt {3}}$	${\color{none} \large \texttt {0}}$	27s	$0.00
`test_swap.py::test_swap_native`	${\color{none} \large \texttt {100} \normalsize \texttt {} }$	${\color{none} \large \texttt {3}}$	${\color{none} \large \texttt {0}}$	22s	$0.00
`test_swap.py::test_swap_triple`	${\color{none} \large \texttt {100} \normalsize \texttt {} }$	${\color{none} \large \texttt {3}}$	${\color{none} \large \texttt {0}}$	41s	$0.00
`test_swap.py::test_swap_with_non_default_token`	${\color{none} \large \texttt {100} \normalsize \texttt {} }$	${\color{none} \large \texttt {3}}$	${\color{none} \large \texttt {0}}$	26s	$0.00

Total run time: 16.15 minutes

nerfZael · 2024-06-28T17:17:08Z

/workflows/benchmarks agents/token/test_swap_and_send.py 3

github-actions · 2024-06-28T17:17:19Z

Finished benchmarks
Download artifacts

Test Run Summary

Run from: ./autotx/tests/agents/token/test_swap_and_send.py
Base path: autotx/tests/agents/token/test_swap_and_send.py::
Iterations: 3
Total Cost: $0.54
Total Success Rate (%): ${\color{red} \LARGE \texttt {66.67} \large \texttt { (67/-33)} }$

Detailed Results

Test Name	Success Rate (%)	Passes	Fails	Avg Time	Avg Cost
`test_send_and_swap_complex`	${\color{red} \large \texttt {0} \normalsize \texttt {(-100)} }$	${\color{red} \large \texttt {0}}$	${\color{red} \large \texttt {3}}$	59s	$0.10
`test_send_and_swap_simple`	${\color{red} \large \texttt {67} \normalsize \texttt {(-33)} }$	${\color{red} \large \texttt {2}}$	${\color{red} \large \texttt {1}}$	39s	$0.05
`test_swap_and_send_complex`	${\color{none} \large \texttt {100} \normalsize \texttt {} }$	${\color{none} \large \texttt {3}}$	${\color{none} \large \texttt {0}}$	42s	$0.02
`test_swap_and_send_simple`	${\color{none} \large \texttt {100} \normalsize \texttt {} }$	${\color{none} \large \texttt {3}}$	${\color{none} \large \texttt {0}}$	29s	$0.01

Total run time: 8.44 minutes

nerfZael · 2024-06-28T20:07:15Z

/workflows/benchmarks agents/token/research 3

github-actions · 2024-06-28T20:07:26Z

Finished benchmarks
Download artifacts

Test Run Summary

Run from: ./autotx/tests/agents/token/research
Base path: autotx/tests/agents/token/research/test_
Iterations: 3
Total Cost: $7.45
Total Success Rate (%): ${\color{red} \LARGE \texttt {69.44} \large \texttt { (69/-21)} }$

Detailed Results

Test Name	Success Rate (%)	Passes	Fails	Avg Time	Avg Cost
`advanced.py::test_research_and_swap_many_tokens_subjective_complex`	${\color{red} \large \texttt {0} \normalsize \texttt {(-10)} }$	${\color{red} \large \texttt {0}}$	${\color{red} \large \texttt {3}}$	8.39m	$0.00
`advanced.py::test_research_and_swap_many_tokens_subjective_simple`	${\color{red} \large \texttt {33} \normalsize \texttt {(-57)} }$	${\color{red} \large \texttt {1}}$	${\color{red} \large \texttt {2}}$	1.20m	$0.42
`research.py::test_get_top_5_memecoins`	${\color{lightgreen} \large \texttt {100} \normalsize \texttt {(+10)} }$	${\color{lightgreen} \large \texttt {3}}$	${\color{lightgreen} \large \texttt {0}}$	38s	$0.28
`research.py::test_get_top_5_memecoins_in_optimism`	${\color{none} \large \texttt {100} \normalsize \texttt {} }$	${\color{none} \large \texttt {3}}$	${\color{none} \large \texttt {0}}$	36s	$0.21
`research.py::test_get_top_5_most_traded_tokens_from_l1`	${\color{none} \large \texttt {100} \normalsize \texttt {} }$	${\color{none} \large \texttt {3}}$	${\color{none} \large \texttt {0}}$	46s	$0.21
`research.py::test_get_top_5_tokens_from_base`	${\color{none} \large \texttt {100} \normalsize \texttt {} }$	${\color{none} \large \texttt {3}}$	${\color{none} \large \texttt {0}}$	32s	$0.18
`research.py::test_price_change_information`	${\color{none} \large \texttt {100} \normalsize \texttt {} }$	${\color{none} \large \texttt {3}}$	${\color{none} \large \texttt {0}}$	19s	$0.03
`research_and_swap.py::test_research_and_buy_multiple`	${\color{none} \large \texttt {100} \normalsize \texttt {} }$	${\color{none} \large \texttt {3}}$	${\color{none} \large \texttt {0}}$	1.29m	$0.43
`research_and_swap.py::test_research_and_buy_one`	${\color{none} \large \texttt {100} \normalsize \texttt {} }$	${\color{none} \large \texttt {3}}$	${\color{none} \large \texttt {0}}$	32s	$0.13
`research_swap_and_send.py::test_research_buy_multiple_send_multiple`	${\color{red} \large \texttt {0} \normalsize \texttt {(-100)} }$	${\color{red} \large \texttt {0}}$	${\color{red} \large \texttt {3}}$	1.14m	$0.24
`research_swap_and_send.py::test_research_buy_one_send_multiple`	${\color{none} \large \texttt {100} \normalsize \texttt {} }$	${\color{none} \large \texttt {3}}$	${\color{none} \large \texttt {0}}$	48s	$0.18
`research_swap_and_send.py::test_research_buy_one_send_one`	${\color{red} \large \texttt {0} \normalsize \texttt {(-100)} }$	${\color{red} \large \texttt {0}}$	${\color{red} \large \texttt {3}}$	41s	$0.16

Total run time: 50.72 minutes

cbrzn · 2024-07-02T11:48:52Z

autotx/load_tokens.py

@@ -41,5 +42,5 @@ def fetch_tokens_list() -> None:
        f.write(content)


-def run() -> None:
-    fetch_tokens_list()
+async def run() -> None:


is there any reason to make this async? this is expected to be run through the script poetry run load-token; I just tried it and got the error

autotx-py3.10cesar@dandy:~/dev/polywrap/auto-tx$ poetry run load-tokens <coroutine object run at 0x7e0b30163d10> sys:1: RuntimeWarning: coroutine 'run' was never awaited RuntimeWarning: Enable tracemalloc to get the object allocation traceback

cbrzn · 2024-07-02T11:54:48Z

autotx/smart_accounts/smart_account.py

+    def on_intents_prepared(self, intents: list[Intent]) -> None:
+        pass
+
+    @abstractmethod


nit: for abstract methods, I would recommend to raise an exception so if it is not implemented it will throw (even tho i think the build-check script will complain about this)

cbrzn · 2024-07-02T12:02:35Z

smart_account_api/src/index.ts

+}));
+
+app.post("/api/v1/account/transactions", handleError(async (req: Request, res: Response, next: NextFunction) => {
+  const chainId = parseInt(req.query.chainId as string);


nit: should we validate that chainId is indeed present in the query object?

nerfZael added 10 commits June 20, 2024 19:16

implemented starting smart account api

4cf7925

removed the reliance on manager.balance_of

3b66585

decoupling safe manager from smart wallet

cf5e542

Merge remote-tracking branch 'origin/main' into nerfzael/biconomy

b0f9076

Merge remote-tracking branch 'origin/dev' into nerfzael/biconomy

ca52a5d

smart-account-api docker implementation

eca4e24

sending transactions with local biconomy account

56cff6d

initial biconomy api implementation

e352897

updated readme with biconomy instructions

27d67ef

added network with bundler url

0f45fbb

nerfZael requested review from dOrgJelli and cbrzn June 24, 2024 19:10

typing fixes

d03dc70

dOrgJelli approved these changes Jun 25, 2024

View reviewed changes

nerfZael added 2 commits June 27, 2024 11:11

Merge remote-tracking branch 'origin/nerfzael/logging-and-errors' int…

a0b62cc

…o nerfzael/biconomy

swapping is now async

8197f6b

agentcoinorg deleted a comment from github-actions bot Jun 27, 2024

nerfZael added 4 commits June 27, 2024 20:52

lowered max rounds to 20 when testing

52c0bd1

added execution logs

224fb06

fixed inverted bool

7649dce

catching all exceptions for swap

e95d49d

agentcoinorg deleted a comment from github-actions bot Jun 28, 2024

nerfZael marked this pull request as ready for review June 28, 2024 00:46

added pytest timeout

0fa32df

agentcoinorg deleted a comment from github-actions bot Jun 28, 2024

nerfZael added 7 commits June 28, 2024 12:32

fixes issue where benchmarks break if first test fails

0214491

added timeouts to tests

625acef

using aiohttp directly to not dispose of session

906aef9

added constants for test timeouts

305acc4

executing transactions one by one in safe if the multisend fails to e…

e7ab90c

…xecute

raising exception where appropriate

0749613

Merge remote-tracking branch 'origin/dev' into nerfzael/biconomy

4c2d438

agentcoinorg deleted a comment from github-actions bot Jun 28, 2024

nerfZael added 2 commits June 28, 2024 20:14

increased test timeout of research tests

6532b06

updated tests with intents and using more gaming categories as valid

fd61f5c

agentcoinorg deleted a comment from github-actions bot Jun 28, 2024

nerfZael merged commit d6e34d6 into dev Jul 2, 2024
1 check passed

cbrzn reviewed Jul 2, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Biconomy integration #277

Biconomy integration #277

nerfZael commented Jun 24, 2024

dOrgJelli left a comment

nerfZael commented Jun 28, 2024

github-actions bot commented Jun 28, 2024 •

edited

Loading

nerfZael commented Jun 28, 2024

github-actions bot commented Jun 28, 2024 •

edited

Loading

nerfZael commented Jun 28, 2024

github-actions bot commented Jun 28, 2024 •

edited

Loading

cbrzn Jul 2, 2024

cbrzn Jul 2, 2024

cbrzn Jul 2, 2024

Biconomy integration #277

Biconomy integration #277

Conversation

nerfZael commented Jun 24, 2024

dOrgJelli left a comment

Choose a reason for hiding this comment

nerfZael commented Jun 28, 2024

github-actions bot commented Jun 28, 2024 • edited Loading

Test Run Summary

Detailed Results

nerfZael commented Jun 28, 2024

github-actions bot commented Jun 28, 2024 • edited Loading

Test Run Summary

Detailed Results

nerfZael commented Jun 28, 2024

github-actions bot commented Jun 28, 2024 • edited Loading

Test Run Summary

Detailed Results

cbrzn Jul 2, 2024

Choose a reason for hiding this comment

cbrzn Jul 2, 2024

Choose a reason for hiding this comment

cbrzn Jul 2, 2024

Choose a reason for hiding this comment

github-actions bot commented Jun 28, 2024 •

edited

Loading

github-actions bot commented Jun 28, 2024 •

edited

Loading

github-actions bot commented Jun 28, 2024 •

edited

Loading