Add hash price for orders #495

Merged
paratoxic merged 14 commits from add-hashprice into master 2026-04-09 16:34:40 +00:00
Owner
No description provided.
Add hash price for orders
Some checks failed
CI / test-linux (pull_request) Has been cancelled
CI / lint (pull_request) Has been cancelled
CI / ckpool (pull_request) Has been cancelled
40ad980945
Change
Some checks failed
CI / ckpool (pull_request) Successful in 12s
CI / lint (pull_request) Successful in 1m35s
CI / test-linux (pull_request) Failing after 4m57s
9bc8150f3d
Consolidate
Some checks failed
CI / ckpool (pull_request) Successful in 11s
CI / lint (pull_request) Successful in 1m5s
CI / test-linux (pull_request) Failing after 4m0s
56d1d23d9d
Change
Some checks failed
CI / ckpool (pull_request) Successful in 19s
CI / lint (pull_request) Successful in 1m30s
CI / test-linux (pull_request) Failing after 4m27s
8bedfcb8ca
Fix
Some checks failed
CI / ckpool (pull_request) Successful in 20s
CI / lint (pull_request) Successful in 1m30s
CI / test-linux (pull_request) Failing after 4m22s
6b55bb5044
Consolidate
Some checks failed
CI / ckpool (pull_request) Successful in 9s
CI / lint (pull_request) Successful in 1m16s
CI / test-linux (pull_request) Failing after 3m54s
837b897be9
Tweak
Some checks failed
CI / ckpool (pull_request) Successful in 8s
CI / lint (pull_request) Successful in 1m5s
CI / test-linux (pull_request) Failing after 4m13s
abc8875126
Relevel
Some checks failed
CI / ckpool (pull_request) Successful in 11s
CI / lint (pull_request) Successful in 1m6s
CI / test-linux (pull_request) Failing after 4m5s
ed7f62d0db
Relevel
Some checks failed
CI / ckpool (pull_request) Successful in 14s
CI / lint (pull_request) Successful in 1m11s
CI / test-linux (pull_request) Failing after 3m54s
39d3b3cc83
Consolidate
Some checks failed
CI / ckpool (pull_request) Successful in 16s
CI / lint (pull_request) Successful in 1m37s
CI / test-linux (pull_request) Has been cancelled
d6c80da923
Change
Some checks failed
CI / ckpool (pull_request) Successful in 12s
CI / lint (pull_request) Successful in 1m5s
CI / test-linux (pull_request) Has been cancelled
74d463455a
Change
Some checks failed
CI / ckpool (pull_request) Successful in 12s
CI / lint (pull_request) Successful in 1m9s
CI / test-linux (pull_request) Has been cancelled
d2c95e7a44
Change
Some checks failed
CI / ckpool (pull_request) Successful in 10s
CI / lint (pull_request) Successful in 1m12s
CI / test-linux (pull_request) Failing after 3m59s
b32ca889a7
parabit approved these changes 2026-04-09 16:14:07 +00:00
parabit left a comment
Owner

LGTM

Two nits, no blockers

LGTM Two nits, no blockers
@ -98,2 +98,4 @@
--tick-interval 1 \
--hashprice 5000000 \
--default-order tb1qft5p2uhsdcdc3l2ua4ap5qqfg4pjaqlp250x7us7a8qqhrxrxfsqaqh7jw.chameleon@127.0.0.1:42069 \
--wallet-birthday 13000 \
Owner

Is this just being used as a test value? It seems rather low for a real wallet.

Is this just being used as a test value? It seems rather low for a real wallet.
@ -41,0 +43,4 @@
value_parser = clap::value_parser!(u64).range(1..),
help = "Price per petahash-day in sats."
)]
pub(crate) hash_price: u64,
Owner

Would be better as a non-required field as it is reasonable that someone could want to use the router with only default-orders or with free reassignment of their hashrate.

Would be better as a non-required field as it is reasonable that someone could want to use the router with only default-orders or with free reassignment of their hashrate.
Update
Some checks failed
CI / ckpool (pull_request) Successful in 17s
CI / lint (pull_request) Successful in 1m18s
CI / test-linux (pull_request) Failing after 4m32s
86ebc8b664
paratoxic merged commit 4a27fb96c1 into master 2026-04-09 16:34:40 +00:00
paratoxic deleted branch add-hashprice 2026-04-09 16:34:40 +00:00
paratoxic referenced this pull request from a commit 2026-04-09 16:34:41 +00:00
Sign in to join this conversation.
No description provided.