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

feat: Refactor get_auction API to return historic entries #2844

Open
sunce86 opened this issue Jul 29, 2024 · 2 comments
Open

feat: Refactor get_auction API to return historic entries #2844

sunce86 opened this issue Jul 29, 2024 · 2 comments

Comments

@sunce86
Copy link
Contributor

sunce86 commented Jul 29, 2024

Problem

Currently get_auction API function returns only the latest auction built by autopilot.

We need to add possibility for this endpoint to return auction by given auction_id/block_number.

Suggested solution

Most of the data can be fetched from database even now. Tables of interest are orders, auction_orders, auction_prices, fee_policies, surplus_capturing_jit_order_owners. Things missing in the database are: auction.block, auction. latest_settlement_block and order.executed field. These fields should be saved and then all the data required to reconstruct the auction will exist.

@fleupold commented (todo check):
Can’t auction.lastest_settlement_block and executed be reconstructed from the existing trades/settlements table assuming auction.block is saved?

Additional context

Solvers, solver-team and other parties are very interested in this endpoint and this could have a significant impact on those parties making their work easier, faster and cleaner.

sunce86 added a commit that referenced this issue Sep 18, 2024
# Description
Currently fee policies are saved only for winning solution.

This PR saves fee policies for all auction orders. This is needed for at
least two reasons:

1. As discussed [in the
PR](#2980 (comment)),
fee policies will be needed for all proposed solutions during a
competition so that the score could be reconstructed in circuit breaker.
2. [For historical
get_auction](#2844).
Copy link

This issue has been marked as stale because it has been inactive a while. Please update this issue or it will be automatically closed.

Copy link

github-actions bot commented Dec 5, 2024

This issue has been marked as stale because it has been inactive a while. Please update this issue or it will be automatically closed.

@github-actions github-actions bot added the stale label Dec 5, 2024
@sunce86 sunce86 removed the stale label Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant