dgb: Phase B pool-layer slice 2 — sharechain verify+tracker (share_check, share_tracker) — Scrypt-only - #116
Closed
frstrtr wants to merge 1 commit into
Closed
dgb: Phase B pool-layer slice 2 — sharechain verify+tracker (share_check, share_tracker) — Scrypt-only#116frstrtr wants to merge 1 commit into
frstrtr wants to merge 1 commit into
Conversation
…rypt-only Phase B pool-layer slice 2. Ports ltc share_check.hpp (3307L) and share_tracker.hpp (2814L) into the dgb namespace. Divergence from src/impl/ltc is namespace-only (ltc -> dgb) EXCEPT one consensus constant: parent block-period score window uses DGB BLOCK_PERIOD = 15s (farsider350 digibyte.py:21) vs LTC 150s. SHARE_PERIOD (25), CHAIN_LENGTH/REAL_CHAIN_LENGTH (8640) already match farsider350 via config_pool.hpp PoolConfig accessors — no further divergence. Formatter / wire layout byte-parity with p2pool-merged-v36 preserved. DOGE-aux merged-mining verify paths carried verbatim, dormant in the default (no -DAUX_DOGE) build; parent-target semantics now = DGB-Scrypt. Header-only pillars, not yet wired into node.hpp (later slice), mirroring prior slices. Verified: standalone -fsyntax-only error set is byte-identical to the upstream ltc header (pre-existing consumer-provided include order).
frstrtr
added a commit
that referenced
this pull request
Jun 17, 2026
dgb: RE-LAND sharechain verify+tracker (recover orphaned #116 slice-2)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Phase B pool-layer slice 2, stacked on #115 (base
dgb/pool-layer-port).What
Ports the sharechain verification + tracker pillars into the dgb namespace:
src/impl/dgb/share_check.hpp(3307L) — share init/check, hash-link, merkle-link, gentx/PPLNS verifysrc/impl/dgb/share_tracker.hpp(2814L) — verified-chain tracking, scoring, decayed PPLNS weightsDivergence from src/impl/ltc
Namespace-only, with ONE consensus constant:
ltc→dgb)BLOCK_PERIOD(farsider350digibyte.py:21)SHARE_PERIOD(25),CHAIN_LENGTH/REAL_CHAIN_LENGTH(8640) already match farsider350 viaconfig_pool.hppPoolConfigaccessors — no further divergence.Compat
-DAUX_DOGE) build; parent-target semantics now = DGB-Scrypt.Verification
Header-only pillars, not yet wired into
node.hpp(later slice), mirroring #113/#115. Standalone-fsyntax-onlyerror set is byte-identical to the upstream ltc header (pre-existing consumer-provided include order:<cstdint>,mul128_shift), proving the port adds no compile regression.HOLD merge — integrator merges on operator
push approved. Stack order: #112 → #113 → #115 → this.