-
Notifications
You must be signed in to change notification settings - Fork 24
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
[simulate] Support for multi txn in simulated #136
base: main
Are you sure you want to change the base?
Conversation
…Authenticator implementation
… and update related documentation
…ensuring proper formatting in multiple files
…d organization and consistency
…ding comments in example
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.
Looks generally fine, could we get a unit test for it?
Sure, let me add to that. |
…tion simulation test
… multi-signer test
It's a bit strange that importing
|
This is, I think an issue involving not limiting the number of concurrent threads. This can cause it to run out of threads, or if you have other things going on, it can cause other issues. Can be very difficult to debug |
Description
Support for multi in simulated trading
Test Plan
examples/simulate_multi_transaction
Related Links