08:37 <PaperSword> Why does 0xfffffffffffffffffffffffffffffffebaaedce6af48a03bbfd25e8cd0364140 and 0x0000000000000000000000000000000000000000000000000000000000000001 generate the same public key under SECP256K1? both of those values exist at the end of the valid input range for the function.

09:13 <real_or_random> PaperSword you mean *xonly* pubkey?

09:14 <real_or_random> elliptic curves such as secp256k1 are cyclic groups. so the input is taken modulo the group order, which is 0xfffffffffffffffffffffffffffffffebaaedce6af48a03bbfd25e8cd0364140.... your first value is -1 mod the group order and 1 is 1 mod the group order.

09:17 <real_or_random> So these generate different elliptic curve points P1 = -G and and P2 = G, where G is the generator. Since P1 = -P2, that means that P1 and P2 happen to have the same x-coordinate but their y-coordinate add up to 0 and have different signs. (You negate a point simply by negating its y-coordinate.)

09:18 <real_or_random> If you look at the *xonly* pubkeys (which have only the x-coordinate), then yes, those two generate the same pubkey

09:19 <PaperSword> Correct xonly

09:19 <PaperSword> Thanks so much for the explanation

12:38 <bitcoin-git> [bitcoin] furszy opened pull request #26780: rpc: simplify scan blocks (master...2022_simplify_scan_blocks) https://github.com/bitcoin/bitcoin/pull/26780

12:44 <bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/65de8eeeca29...8575d5d84265

12:44 <bitcoin-git> bitcoin/master 090ad51 Joshua Kelly: rpc: Remove duplicate field in RPCHelpMan for gettransactions

12:44 <bitcoin-git> bitcoin/master 8575d5d MarcoFalke: Merge bitcoin/bitcoin#26777: rpc: Remove duplicate field in RPCHelpMan for...

12:44 <bitcoin-git> [bitcoin] MarcoFalke merged pull request #26777: rpc: Remove duplicate field in RPCHelpMan for gettransactions (master...rpc/get-transactions-typo) https://github.com/bitcoin/bitcoin/pull/26777

15:38 <bitcoin-git> [bitcoin] hebasto opened pull request #26781: Release `LockData::dd_mutex` before calling `*_detected` functions (master...221231-lock) https://github.com/bitcoin/bitcoin/pull/26781

18:25 <bitcoin-git> [bitcoin] andrewtoth opened pull request #26782: p2p: cache block hash in ProcessGetBlockData and CMPCTBLOCK processing (master...cache-block-header-hash) https://github.com/bitcoin/bitcoin/pull/26782

19:58 <bitcoin-git> [bitcoin] SmashedFrenzy16 opened pull request #26784: Update key_io.cpp (master...patch-1) https://github.com/bitcoin/bitcoin/pull/26784

20:11 <bitcoin-git> [bitcoin] hebasto closed pull request #26784: Update key_io.cpp (master...patch-1) https://github.com/bitcoin/bitcoin/pull/26784

20:47 <bitcoin-git> [bitcoin] CodeMaster7000 opened pull request #26785: Update README.md to improve grammar (master...patch-1) https://github.com/bitcoin/bitcoin/pull/26785

20:58 <bitcoin-git> [bitcoin] andrewtoth closed pull request #26782: p2p: cache block hash in ProcessGetBlockData and CMPCTBLOCK processing (master...cache-block-header-hash) https://github.com/bitcoin/bitcoin/pull/26782