Skip to content

staking: tell a market that is empty apart from a batch too small to price - #11

Merged
GracedEternalKingCabbageMan merged 1 commit into
sequentiafrom
fix/too-small-to-price
Aug 24, 2026
Merged

GracedEternalKingCabbageMan merged 1 commit into
sequentiafrom
fix/too-small-to-price

Conversation

@GracedEternalKingCabbageMan

Copy link
Copy Markdown
Collaborator

Run against the live testnet's cross book, a wallet holding 682 atoms of OILX was told "no market for this pair". There were twelve resting offers buying OILX for Bitcoin. What was actually true is that 682 atoms price to zero satoshis.

Saying "no market" there sends a staker looking for liquidity that is already on the book — and invites someone to go and seed a market that already exists.

The reference price tells the two apart (it is only set when there were offers), so a quote of nothing against a non-zero reference is now its own answer: there is a market, this is just worth less than one unit of it yet.

19 tests pass, including one that pins both readings.

…price

Run against the live testnet's cross book, a wallet holding 682 atoms of
OILX was told 'no market for this pair'. There were twelve resting offers
buying OILX for Bitcoin. What was actually true is that 682 atoms price to
zero satoshis.

Saying 'no market' there sends a staker looking for liquidity that is
already on the book, and invites someone to go and seed a market that
exists. The reference price tells the two apart -- it is only set when
there were offers -- so a quote of nothing against a non-zero reference is
now its own answer: there is a market, this is just worth less than one
unit of it yet.
@GracedEternalKingCabbageMan
GracedEternalKingCabbageMan merged commit b3cb4e8 into sequentia Aug 24, 2026
11 of 16 checks passed
@GracedEternalKingCabbageMan
GracedEternalKingCabbageMan deleted the fix/too-small-to-price branch August 24, 2026 02:32
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

Successfully merging this pull request may close these issues.

1 participant