Skip to content

Conversation

@jlopp
Copy link
Contributor

@jlopp jlopp commented Aug 4, 2016

It looks like this value isn't documented in the RPC help. If it would be helpful, we could be even more specific and mention that it uses the default trailing 120 blocks parameter to calculate the hashrate.

@paveljanik
Copy link
Contributor

ACK 205eb35

As you are touching the helptext, can you please align "errors" and "chain" helptexts by adding two spaces before "(string)"?

  "errors": "..."          (string) Current errors
  "networkhashps": nnn,      (numeric) The network hashes per second
  "pooledtx": n              (numeric) The size of the mem pool
  "testnet": true|false      (boolean) If using testnet or not
  "chain": "xxxx",         (string) current network name as defined in BIP70 (main, test, regtest)

@jonasschnelli
Copy link
Contributor

ACK 205eb35 agree with @paveljanik about the PR related nits.

@maflcko
Copy link
Member

maflcko commented Aug 5, 2016

ACK 65f4532

@jlopp
Copy link
Contributor Author

jlopp commented Aug 5, 2016

Updated to align "chain" and "errors" output with rest of help text.

@paveljanik
Copy link
Contributor

Thank you!

ACK 65f4532

@btcdrak
Copy link
Contributor

btcdrak commented Aug 5, 2016

ACK 65f4532

1 similar comment
@fanquake
Copy link
Member

fanquake commented Aug 6, 2016

ACK 65f4532

@laanwj laanwj merged commit 65f4532 into bitcoin:master Aug 19, 2016
laanwj added a commit that referenced this pull request Aug 19, 2016
… RPC endpoint

65f4532 document return value of networkhashps for getmininginfo RPC endpoint (Jameson Lopp)
@luke-jr
Copy link
Member

luke-jr commented Sep 10, 2016

Backport needed, I guess. (all the way back to 0.9.x, but we don't maintain that far back)

@maflcko
Copy link
Member

maflcko commented Sep 11, 2016

@luke-jr According to current https://bitcoincore.org/en/lifecycle/ 0.11 only receives critical security fixes and this is only a documentation fix. (Though, I am not against backports for 0.12 and 0.13)

luke-jr pushed a commit to luke-jr/bitcoin that referenced this pull request Sep 21, 2016
codablock pushed a commit to codablock/dash that referenced this pull request Jan 8, 2018
…inginfo RPC endpoint

65f4532 document return value of networkhashps for getmininginfo RPC endpoint (Jameson Lopp)
andvgal pushed a commit to energicryptocurrency/gen2-energi that referenced this pull request Jan 6, 2019
…inginfo RPC endpoint

65f4532 document return value of networkhashps for getmininginfo RPC endpoint (Jameson Lopp)
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants