Skip to content

fix(channels): Update channel nominal hashrate from downstream vardiff estimate#245

Merged
plebhash merged 1 commit into
stratum-mining:mainfrom
marafoundation:fix/tproxy-stale-nominal-hashrate
Feb 7, 2026
Merged

fix(channels): Update channel nominal hashrate from downstream vardiff estimate#245
plebhash merged 1 commit into
stratum-mining:mainfrom
marafoundation:fix/tproxy-stale-nominal-hashrate

Conversation

@gimballock

Copy link
Copy Markdown
Contributor

In aggregated mode, update the upstream extended channel's nominal hashrate to match the downstream vardiff estimate before forwarding UpdateChannel. In non-aggregated mode, update the specific extended channel's nominal hashrate. This ensures monitoring reports values consistent with downstream vardiff estimates.

See stratum-mining/sv2-apps#242

@GitGab19 GitGab19 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to get stratum-mining/stratum#2077 merged before proceeding with this one, otherwise CI is not gonna pass.

@plebhash

plebhash commented Feb 7, 2026

Copy link
Copy Markdown
Member

now that stratum-mining/stratum#2077 was merged, we need to update the following so they point to the new version:

  • integration-tests/Cargo.lock
  • miner-apps/Cargo.lock
  • pool-apps/Cargo.lock

@Shourya742 Shourya742 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK.

…f estimate

In aggregated mode, update the upstream extended channel's nominal hashrate to match the downstream vardiff estimate before forwarding UpdateChannel. In non-aggregated mode, update the specific extended channel's nominal hashrate. This ensures monitoring reports values consistent with downstream vardiff estimates.
@gimballock
gimballock force-pushed the fix/tproxy-stale-nominal-hashrate branch from 81735e1 to ea5fa6e Compare February 7, 2026 21:02
@plebhash
plebhash merged commit b6a8f73 into stratum-mining:main Feb 7, 2026
11 checks passed
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.

4 participants