fix: use client.multiplier to avoid KeyError crash (#7766)#7768
fix: use client.multiplier to avoid KeyError crash (#7766)#7768lequangsang01 wants to merge 2 commits into
Conversation
FakerHideInBush
left a comment
There was a problem hiding this comment.
This PR makes the exact same single-line change as the already-open PR #7767 — both modify vintage_miner/vintage_miner_client.py line 433 from evidence['profile']['multiplier'] to client.multiplier.
The diff is byte-for-byte identical to PR #7767:
- print(f"Multiplier: {evidence['profile']['multiplier']}x")
+ print(f"Multiplier: {client.multiplier}x")Merging both will either produce a merge conflict (if the branch is not rebased after #7767 merges) or result in a no-op second merge. There should only be one PR for this fix.
Recommendation: Close this PR in favor of #7767 (or vice versa), or add the issue reference (Closes #7766) to PR #7767's description if that PR was filed without it — then close this one. The fix itself (client.multiplier instead of evidence['profile']['multiplier']) is correct; it just needs to land via one PR, not two.
Closes #7766
RTC wallet: RTCfe13452d122263caf633ab1876bd9631133b68b1
Changes
evidence["profile"]["multiplier"]withclient.multiplierTesting