1
0
mirror of https://github.com/bitcoinbook/bitcoinbook synced 2024-12-23 23:18:42 +00:00
Commit Graph

3913 Commits

Author SHA1 Message Date
Andreas M. Antonopoulos
e3695afbf6
Merge pull request #742 from syncom/syncom/ch07-2-of-5-multisig
ch07: 2-of-5 instead of 2-of-3 for Mohammed's example
2021-01-19 11:41:58 -06:00
Andreas M. Antonopoulos
bf56c59a85
Merge pull request #746 from jerzybrzoska/patch-3
ch.02: Let us not be too assuming about the gender of "each miner"
2021-01-19 11:20:38 -06:00
Hennadii Stepanov
5d50a93ee6
ch10: Align target description with the actual implementation
See: 43f3ada27b/src/pow.cpp (L86-L88)
2021-01-19 15:10:02 +02:00
jerzybrzoska
65dd6b8be0
ch.02: Let us not be too assuming about the gender of "each miner"
It is too assuming to suppose that the gender of "each miner" is masculine. Hence I propose to use [singular they](https://en.wikipedia.org/wiki/Singular_they). More on singular they [here](https://www.grammarly.com/blog/use-the-singular-they/).
'He' was substituted with 'they' only when it refers to "each miner", i.e. miners in general, and not when it refers to Jing. In the latter case I have left he unchanged.
2021-01-19 13:05:10 +01:00
Ning Shang
7284634837
Nit: exponentially -> significantly
In the dice rolling example, because there are only two dice, the phrase 'exponentially more' could cause confusion. It feels better to be a bit handwavy here and just say 'significantly more'.
2021-01-18 20:51:21 -08:00
Ning Shang
c942d694b6
ch07: Typo fixes
Fix minor typos related to 'equal to' and 'less/greater than'.
2021-01-18 20:34:24 -08:00
Ning Shang
06f1a947e9
chp04: xref <<vanity_minor_code>> seems better for note on std:random_device
Because 'std::random_device' appears in xref vanity_miner_code as opposed to xref vanity_miner_run, it feels more appropriate to use the former in this note.
2021-01-18 20:25:31 -08:00
Ning Shang
555be41b2b
ch07: 2-of-5 instead of 2-of-3 for Mohammed's example
The example is a 2-of-5 multisig, according to the Pay-to-Script-Hash example description earlier, and the P2SH example a few lines below.
2021-01-18 20:00:19 -08:00
marco zandonadi
57f0d77d40
Update ch03.asciidoc
Indefinite article "a" should precede a noun starting with a consonant
2021-01-18 19:40:45 -08:00
rating89us
f467afe028
ch08: delete extra space 2021-01-18 20:43:05 +01:00
Andreas M. Antonopoulos
b926626c1a
Merge pull request #739 from hebasto/210118-dns
ch08: Update number of DNS seeds
2021-01-18 09:08:42 -06:00
Hennadii Stepanov
b5a7b5df3e
ch08: Update number of DNS seeds
There are 9 seeds since 0.21: https://github.com/bitcoin/bitcoin/blob/0.21/src/chainparams.cpp#L121-L129

And there were 8 seeds since v0.18: https://github.com/bitcoin/bitcoin/blob/0.18/src/chainparams.cpp#L123-L130
2021-01-18 16:52:48 +02:00
Andreas M. Antonopoulos
d1e61889bb
Merge pull request #738 from rating89us/patch-2
ch10: make it clear that we talk about hard fork in bitcoin
2021-01-17 09:40:43 -06:00
rating89us
1d58378edf ch10: make it clear that hard fork is in bitcoin 2021-01-16 21:06:43 +01:00
rating89us
21597f5cf9
ch01: add space after comma and before i.e. 2021-01-15 17:23:43 +01:00
Andreas M. Antonopoulos
0eb5d5eb30
Merge pull request #733 from hebasto/210113-trap
ch04: Fix mathematical terms
2021-01-15 06:46:52 -06:00
Hennadii Stepanov
a8c672ffb5
ch04: Fix mathematical terms
1. EC multiplication is a "one-way" function, not a "trapdoor" one.
There is no information that makes reverse function easy.
See:
  - https://en.wikipedia.org/wiki/Trapdoor_function
  - https://en.wikipedia.org/wiki/One-way_function

2. The "discrete logarithm" is more usual term for the reverse operation
for EC multiplication.
2021-01-15 09:23:51 +02:00
Andreas M. Antonopoulos
387e9b1a31
Merge pull request #735 from jerzybrzoska/patch-1
Choice of words: possible but pointless (introduction)
2021-01-14 19:59:00 -06:00
Andreas M. Antonopoulos
ca5d0e6485
Merge pull request #731 from rating89us/patch-9
ch02: better wording of transaction outputs explanation
2021-01-14 19:53:33 -06:00
Andreas M. Antonopoulos
784351ad69
Merge pull request #708 from rating89us/patch-5
ch10: delete 'no longer'; clarify 'forward' word
2021-01-14 19:34:49 -06:00
Andreas M. Antonopoulos
5d923833a4
Merge pull request #707 from rating89us/patch-4
ch10: follows -> follow
2021-01-14 19:34:11 -06:00
Andreas M. Antonopoulos
c7b30db76d
Merge pull request #706 from rating89us/patch-3
ch10: add "'s input"
2021-01-14 19:33:26 -06:00
Andreas M. Antonopoulos
8393fa275d
Merge pull request #705 from rating89us/patch-2
ch10: paralellism (the...the)
2021-01-14 19:32:39 -06:00
Andreas M. Antonopoulos
71b4e52dcb
Merge pull request #704 from rating89us/patch-1
ch10: candidate node -> candidate block
2021-01-14 19:32:04 -06:00
Andreas M. Antonopoulos
3a76f89a46
Merge pull request #700 from satsie/fix-ch10-typo
Fix Minor Typo in Chapter 10
2021-01-14 19:31:41 -06:00
Andreas M. Antonopoulos
86d59b0045
Merge pull request #699 from Xekyo/transactionSizes
Correct transaction costs in segwit example
2021-01-14 19:30:55 -06:00
Andreas M. Antonopoulos
22a09e0658
Merge pull request #736 from jerzybrzoska/patch-2
450 MiB is now the default cache size (ch. 3)
2021-01-14 19:28:50 -06:00
Andreas M. Antonopoulos
43faf7e7a4
Merge pull request #734 from hebasto/210113-hash
ch04: Fix terms of ScriptSig
2021-01-14 19:28:27 -06:00
jerzybrzoska
39118cdd21
450 MiB is now the default cache size
The default cache size is now 450 MiB on the most recent release (0.20.1)
2021-01-14 12:14:55 +01:00
jerzybrzoska
ccc684d19a
Wording: possible but pointless (introduction)
It is possible to make a specific recommendation but it would be rather futile to do so.
2021-01-14 10:51:50 +01:00
Hennadii Stepanov
3d34e80ef5
ch04: Fix terms of ScriptSig
For P2PKH outputs the ScriptSig contains a public key, but not its hash.
2021-01-14 02:08:19 +02:00
Andreas M. Antonopoulos
d4ee007cbb
Merge pull request #732 from jerzybrzoska/patch-1
"Bitcoin are ..." is not correct.
2021-01-11 14:45:36 -06:00
jerzybrzoska
d652c952cf
Update ch01.asciidoc 2021-01-11 20:53:50 +01:00
jerzybrzoska
f6ba290d4b
"Bitcoin are ..." is not correct.
I also added a parenthetical clause that explained what bitcoins are. This version seems less ambiguous to me.
2021-01-11 20:40:09 +01:00
rating89us
472136434b
ch02: better wording of transaction outputs explanation 2021-01-11 15:07:32 +01:00
Andreas M. Antonopoulos
77186aa3b1
Merge pull request #729 from gyousuf/patch-1
Update full node disk space requirement
2021-01-07 08:46:27 -06:00
Guled Yousuf
2a7d860f58
Update full node disk space requirement
Updating the disk space requirement to run a full node from 160GB to 360GB.
2021-01-05 19:47:40 -08:00
rating89us
dd17f8a422
ch10: 'support'->'provide(s) support for'; add 'string'; add 'finally,' 2020-11-25 21:18:39 +01:00
rating89us
fadbe5d4f4
ch10: delete 'no longer'; clarify that 'forward' means in relation to the hard fork event 2020-11-25 20:05:26 +01:00
rating89us
1af3d97de5
ch10: follows -> follow 2020-11-25 08:47:22 +01:00
rating89us
1fc2e16334
ch10: add "'s input" 2020-11-25 00:34:14 +01:00
rating89us
095bf26c9a
ch10: paralellism (the...the) 2020-11-24 13:15:07 +01:00
rating89us
7a97ea0ee3
ch10: candidate node -> candidate block 2020-11-24 12:56:34 +01:00
Stacie Waleyko
eb4ec4f0b7
Fix minor typo in Chapter 10 2020-10-24 21:57:40 -04:00
Murch
8072effb93
Correct transaction costs in segwit example
The transaction fees in the example were off.
2-of-3 P2SH inputs are between 293 and 297 bytes (293 bytes if both
signatures have low-R, 296 bytes if one signature has high-R, 297 bytes
if both signatures have high-R) and P2SH outputs are 32 bytes.

2-of-3 P2WSH (native segwit) inputs are between 104 and 104.5 vbytes
(416–418 WU, depending on low-R/high-R as above) and P2WSH outputs are
43 bytes.

The transaction header has 10 bytes for non-segwit transactions and 42
WU for segwit transactions.

Source:
https://github.com/BitGo/unspents/blob/master/src/dimensions.ts#L69

I have updated the numbers in the example and amended the conclusion
which did not match the new numbers. Calculation details follow.

-----

Let's assume that the wallet is using signature grinding and will always
produce signatures with low-R:

•Without Segregated Witness:
Transaction A using 2-of-3 P2SH inputs and P2SH outputs:
3*293 + 10 + 2*32 = 953
953 * 30 = 28,590

Transaction B using 2-of-3 P2SH inputs and P2SH outputs:
2*293 + 10 + 3*32 = 692
692 * 30 = 20,760

•With Segregated Witness:
Transaction A using 2-of-3 P2WSH (native segwit) inputs and outputs:
3*104 + 10.5 + 2*43 = 408.5
408.5 * 30 = 12,255

Transaction B using 2-of-3 P2WSH (native segwit) inputs and P2WSH
outputs:
2*104 + 10.5 + 3*43 = 347.5
347.5 * 30 = 10,425
2020-10-13 23:11:13 -04:00
Andreas M. Antonopoulos
f40ccd601f
Merge pull request #698 from reachjason/patch-1
Update ch01.asciidoc
2020-10-13 17:12:30 -05:00
Andreas M. Antonopoulos
535c9a0653
Merge pull request #696 from ivigamberdiev/patch-1
Update links on resources
2020-10-13 17:11:43 -05:00
Andreas M. Antonopoulos
5b720c8380
Merge pull request #695 from 64H-1/patch-1
Change 1.8 ZH/s to 1.8 ZH
2020-10-13 17:10:28 -05:00
Jason Rodrigues
3e39371a1f
Update ch01.asciidoc
Fixed some grammatical errors.
2020-10-12 11:55:45 +05:30
ivigamberdiev
ba72ea5cf5
Update links on resources
- Update link to new libbitcoin repo
- https for bcoin
- Remove Bits of Proof, whose site was purchased
- Update the link to the fork of pybitcointools that was maintained by the community after Vitalik ended support
2020-08-07 19:29:49 +03:00