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

Refactor to support async trades #1244

Open
slundqui opened this issue Jan 10, 2024 · 5 comments
Open

Refactor to support async trades #1244

slundqui opened this issue Jan 10, 2024 · 5 comments
Labels
enhancement A feature or refactor request

Comments

@slundqui
Copy link

Interactive hyperdrive was written assuming synchronous, one block per transaction. Ideally, we would add a feature allowing sending multiple transactions on the same block.

@dpaiton dpaiton added the enhancement A feature or refactor request label Jan 11, 2024
@dpaiton dpaiton changed the title Interactive hyperdrive allows for multiple transactions on the same block Interactive hyperdrive should allow for multiple transactions on the same block Mar 11, 2024
@dpaiton
Copy link
Contributor

dpaiton commented May 1, 2024

This might be best achieved "under the hood" via executing policy actions from interactive hyperdrive

@slundqui
Copy link
Author

slundqui commented May 7, 2024

Moving priority to high, as this currently is possible through policies. Need to implement async single calls to allow this generally, which is a huge refactor, and doesn't bring much to the table.

@dpaiton
Copy link
Contributor

dpaiton commented Jun 5, 2024

renaming to the actual task that needs to be performed in order to support multiple transactions on the same block

@dpaiton dpaiton changed the title Interactive hyperdrive should allow for multiple transactions on the same block Refactor to support async trades Jun 5, 2024
@dpaiton
Copy link
Contributor

dpaiton commented Jun 5, 2024

This is a "nice to have" for RL bots but there are workarounds (per @slundqui ) . Moving the priority to medium.

@slundqui
Copy link
Author

slundqui commented Jun 5, 2024

Here's the snippit for running sync functions async.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement A feature or refactor request
Projects
None yet
Development

No branches or pull requests

2 participants