💬 maflcko commented on pull request "wallet: Disable creating and loading legacy wallets":
(https://github.com/bitcoin/bitcoin/pull/31250#discussion_r2060029160)
forgot to remove this?
(https://github.com/bitcoin/bitcoin/pull/31250#discussion_r2060029160)
forgot to remove this?
💬 maflcko commented on pull request "wallet: Disable creating and loading legacy wallets":
(https://github.com/bitcoin/bitcoin/pull/31250#discussion_r2059977759)
unrelated in c847dee1488a294c9a9632a00ba1134b21e41947: However, while touching, it could make sense to force named args for literal args, especially integral (boolean) ones:
```py
def __init__(self, rpc, *, cli=False):
(https://github.com/bitcoin/bitcoin/pull/31250#discussion_r2059977759)
unrelated in c847dee1488a294c9a9632a00ba1134b21e41947: However, while touching, it could make sense to force named args for literal args, especially integral (boolean) ones:
```py
def __init__(self, rpc, *, cli=False):
💬 maflcko commented on pull request "wallet: Disable creating and loading legacy wallets":
(https://github.com/bitcoin/bitcoin/pull/31250#discussion_r2060139068)
Also, you forgot to remove `REQUIRE_WALLET_TYPE_SET` in this commit.
(https://github.com/bitcoin/bitcoin/pull/31250#discussion_r2060139068)
Also, you forgot to remove `REQUIRE_WALLET_TYPE_SET` in this commit.
💬 maflcko commented on pull request "wallet: Disable creating and loading legacy wallets":
(https://github.com/bitcoin/bitcoin/pull/31250#discussion_r2060010792)
why remove bdbro and swap-bdb-endian? The options remain, so either they should be tested, or fully removed.
(https://github.com/bitcoin/bitcoin/pull/31250#discussion_r2060010792)
why remove bdbro and swap-bdb-endian? The options remain, so either they should be tested, or fully removed.
💬 maflcko commented on pull request "wallet: Disable creating and loading legacy wallets":
(https://github.com/bitcoin/bitcoin/pull/31250#discussion_r2060088046)
forgot to remove LEGACY_WALLET_MSG?
(https://github.com/bitcoin/bitcoin/pull/31250#discussion_r2060088046)
forgot to remove LEGACY_WALLET_MSG?
💬 vasild commented on pull request "p2p: Advance pindexLastCommonBlock early after connecting blocks":
(https://github.com/bitcoin/bitcoin/pull/32180#discussion_r2059993955)
This change assumes their tip is higher than our tip. What if their tip is lower? Maybe worth advancing `pindexLastCommonBlock` in that case as well. Something like this:
```cpp
// If our tip has advanced beyond pindexLastCommonBlock, move it ahead to the tip. We don't need to
// download any blocks in between, and skipping ahead here allows us to determine nWindowEnd better.
const auto our_tip{m_chainman.ActiveTip()};
const auto peer_best_known{state->pindexBestKnownBlock};
if (our_tip-
...
(https://github.com/bitcoin/bitcoin/pull/32180#discussion_r2059993955)
This change assumes their tip is higher than our tip. What if their tip is lower? Maybe worth advancing `pindexLastCommonBlock` in that case as well. Something like this:
```cpp
// If our tip has advanced beyond pindexLastCommonBlock, move it ahead to the tip. We don't need to
// download any blocks in between, and skipping ahead here allows us to determine nWindowEnd better.
const auto our_tip{m_chainman.ActiveTip()};
const auto peer_best_known{state->pindexBestKnownBlock};
if (our_tip-
...
💬 vasild commented on pull request "p2p: Advance pindexLastCommonBlock early after connecting blocks":
(https://github.com/bitcoin/bitcoin/pull/32180#discussion_r2060107128)
nit: could use an array here:
```py
stall_blocks = [blocks[0].sha256, blocks[500].sha256]
```
which would make it possible to have the following code like:
```diff
- peers.append(node.add_outbound_p2p_connection(P2PStaller([stall_block, second_stall]), p2p_idx=id, connection_type="outbound-full-relay"))
+ peers.append(node.add_outbound_p2p_connection(P2PStaller(stall_blocks), p2p_idx=id, connection_type="outbound-full-relay"))
- self.wait_until(lambda: sum(len(peer['inflight']) f
...
(https://github.com/bitcoin/bitcoin/pull/32180#discussion_r2060107128)
nit: could use an array here:
```py
stall_blocks = [blocks[0].sha256, blocks[500].sha256]
```
which would make it possible to have the following code like:
```diff
- peers.append(node.add_outbound_p2p_connection(P2PStaller([stall_block, second_stall]), p2p_idx=id, connection_type="outbound-full-relay"))
+ peers.append(node.add_outbound_p2p_connection(P2PStaller(stall_blocks), p2p_idx=id, connection_type="outbound-full-relay"))
- self.wait_until(lambda: sum(len(peer['inflight']) f
...
👍 vasild approved a pull request: "p2p: Advance pindexLastCommonBlock early after connecting blocks"
(https://github.com/bitcoin/bitcoin/pull/32180#pullrequestreview-2793701909)
ACK c69ee2d5b93296d3008d6978182b2bc29bbeb457
The code changes look reasonable. I tested that `p2p_ibd_stalling.py` fails without commit 1d8504c5e6 `p2p: During block download, adjust pindexLastCommonBlock right away`. It failed 7 times and passed 1 time. Is it expected to pass sometimes?
I also checked that the code inside the modified `if`s is executed during `p2p_ibd_stalling.py`.
(https://github.com/bitcoin/bitcoin/pull/32180#pullrequestreview-2793701909)
ACK c69ee2d5b93296d3008d6978182b2bc29bbeb457
The code changes look reasonable. I tested that `p2p_ibd_stalling.py` fails without commit 1d8504c5e6 `p2p: During block download, adjust pindexLastCommonBlock right away`. It failed 7 times and passed 1 time. Is it expected to pass sometimes?
I also checked that the code inside the modified `if`s is executed during `p2p_ibd_stalling.py`.
💬 vasild commented on pull request "p2p: Advance pindexLastCommonBlock early after connecting blocks":
(https://github.com/bitcoin/bitcoin/pull/32180#discussion_r2060110804)
nit: could use a variable `second_stall_block_index = 500` or something like that to avoid repeating the number `500`.
(https://github.com/bitcoin/bitcoin/pull/32180#discussion_r2060110804)
nit: could use a variable `second_stall_block_index = 500` or something like that to avoid repeating the number `500`.
⚠️ Pimpim11 opened an issue: "Double Poisson Sum: Enhancing Energy Distribution in Bitcoin Proof of Work with Main Formula γ^(i/R) + ∑(S[0:i]·τ·φ^j)"
(https://github.com/bitcoin/bitcoin/issues/32348)
Title: Double Poisson Sum: Enhancing Energy Distribution in Bitcoin Proof of Work with Main Formula γ^(i/R) + ∑(S[0:i]·τ·φ^j)
Author: Farrel Al Feshal
Created: April 25, 2025
Abstract
Introduces a novel mathematical framework, Double Poisson Sum, as a potential enhancement to Bitcoin’s Proof of Work (PoW) consensus mechanism. The proposed main formula, γ^(i/R) + ∑(S[0:i]·τ·φ^j), aims to achieve a more equitable distribution of computational energy across the network while providing robust crypt
...
(https://github.com/bitcoin/bitcoin/issues/32348)
Title: Double Poisson Sum: Enhancing Energy Distribution in Bitcoin Proof of Work with Main Formula γ^(i/R) + ∑(S[0:i]·τ·φ^j)
Author: Farrel Al Feshal
Created: April 25, 2025
Abstract
Introduces a novel mathematical framework, Double Poisson Sum, as a potential enhancement to Bitcoin’s Proof of Work (PoW) consensus mechanism. The proposed main formula, γ^(i/R) + ∑(S[0:i]·τ·φ^j), aims to achieve a more equitable distribution of computational energy across the network while providing robust crypt
...
✅ fanquake closed an issue: "Double Poisson Sum: Enhancing Energy Distribution in Bitcoin Proof of Work with Main Formula γ^(i/R) + ∑(S[0:i]·τ·φ^j)"
(https://github.com/bitcoin/bitcoin/issues/32348)
(https://github.com/bitcoin/bitcoin/issues/32348)
💬 willcl-ark commented on issue "Double Poisson Sum: Enhancing Energy Distribution in Bitcoin Proof of Work with Main Formula γ^(i/R) + ∑(S[0:i]·τ·φ^j)":
(https://github.com/bitcoin/bitcoin/issues/32348#issuecomment-2830289998)
For proposed protocol changes you can post to the [bitcoindev mailing list](https://groups.google.com/g/bitcoindev) or the [Delving Bitcoin](https://delvingbitcoin.org/) Discorse forum. However as this appears to be largely about "Syamailcoin" I am not sure it's particularly appropriate for there either.
For general bitcoin discussion you can try [bitcointalk](https://bitcointalk.org) or [https://reddit.com/r/bitcoin](reddit.com/r/bitcoin).
(https://github.com/bitcoin/bitcoin/issues/32348#issuecomment-2830289998)
For proposed protocol changes you can post to the [bitcoindev mailing list](https://groups.google.com/g/bitcoindev) or the [Delving Bitcoin](https://delvingbitcoin.org/) Discorse forum. However as this appears to be largely about "Syamailcoin" I am not sure it's particularly appropriate for there either.
For general bitcoin discussion you can try [bitcointalk](https://bitcointalk.org) or [https://reddit.com/r/bitcoin](reddit.com/r/bitcoin).
💬 Sjors commented on pull request "doc: warn that CheckBlock() underestimates sigops":
(https://github.com/bitcoin/bitcoin/pull/31624#discussion_r2060159606)
As explained in https://github.com/bitcoin/bitcoin/pull/31981#issuecomment-2830264751 this is fine. `TestBlockValidity` also calls the more thorough method.
I don't know why this check is here, perhaps just a way to quickly reject a block with lots of sigops?
(https://github.com/bitcoin/bitcoin/pull/31624#discussion_r2060159606)
As explained in https://github.com/bitcoin/bitcoin/pull/31981#issuecomment-2830264751 this is fine. `TestBlockValidity` also calls the more thorough method.
I don't know why this check is here, perhaps just a way to quickly reject a block with lots of sigops?
📝 hebasto opened a pull request: "test: Increase stack size for "Debug" builds with MSVC"
(https://github.com/bitcoin/bitcoin/pull/32349)
This PR accommodates the deep recursion in the `FindChallenges()` function used in `test/miniscript_tests.cpp`.
Fixes https://github.com/bitcoin/bitcoin/issues/32341#issuecomment-2829441596.
CI log: https://github.com/hebasto/bitcoin/actions/runs/14664806617/job/41156972137
(https://github.com/bitcoin/bitcoin/pull/32349)
This PR accommodates the deep recursion in the `FindChallenges()` function used in `test/miniscript_tests.cpp`.
Fixes https://github.com/bitcoin/bitcoin/issues/32341#issuecomment-2829441596.
CI log: https://github.com/hebasto/bitcoin/actions/runs/14664806617/job/41156972137
💬 hebasto commented on issue "ctest -C Debug fails on vs2022 (miniscript_tests (SEGFAULT))":
(https://github.com/bitcoin/bitcoin/issues/32341#issuecomment-2830328335)
> Thx. Though, now it says:
>
> ```
> The following tests FAILED:
> 60 - miniscript_tests (SEGFAULT)
> Errors while running CTest
> ```
>
> * I guess this is not really a segfault/bug in the code, but rather a stack overflow, due to the debug config eating more memory?
Fixed in https://github.com/bitcoin/bitcoin/pull/32349.
(https://github.com/bitcoin/bitcoin/issues/32341#issuecomment-2830328335)
> Thx. Though, now it says:
>
> ```
> The following tests FAILED:
> 60 - miniscript_tests (SEGFAULT)
> Errors while running CTest
> ```
>
> * I guess this is not really a segfault/bug in the code, but rather a stack overflow, due to the debug config eating more memory?
Fixed in https://github.com/bitcoin/bitcoin/pull/32349.
💬 fanquake commented on pull request "wallet: Disable creating and loading legacy wallets":
(https://github.com/bitcoin/bitcoin/pull/31250#issuecomment-2830337961)
Looks like `is_bdb_compiled` & `skip_if_no_bdb` should have been dropped.
(https://github.com/bitcoin/bitcoin/pull/31250#issuecomment-2830337961)
Looks like `is_bdb_compiled` & `skip_if_no_bdb` should have been dropped.
💬 maflcko commented on pull request "Add checkBlock() to Mining interface":
(https://github.com/bitcoin/bitcoin/pull/31981#issuecomment-2830342567)
I've implemented a typo check in DrahtBot and there are more, see https://github.com/bitcoin/bitcoin/pull/31981#issuecomment-2697111800
Also, the CI is failing.
(https://github.com/bitcoin/bitcoin/pull/31981#issuecomment-2830342567)
I've implemented a typo check in DrahtBot and there are more, see https://github.com/bitcoin/bitcoin/pull/31981#issuecomment-2697111800
Also, the CI is failing.
💬 Pimpim11 commented on issue "Double Poisson Sum: Enhancing Energy Distribution in Bitcoin Proof of Work with Main Formula γ^(i/R) + ∑(S[0:i]·τ·φ^j)":
(https://github.com/bitcoin/bitcoin/issues/32348#issuecomment-2830347260)
@willcl-ark Thanks a lot, I did Muscle Memory in Math in 2016 for Syamailcoin, and I never know if it is not possible to market under 21 Million, CoinMarketCap say I must have Big Community to Enter the Crypto Market, in September 2024 I managed to run consensus but because I didn't think about the community from the beginning, i reset all and on 13 April 2025 i did the last revision in the whitepaper.
Maybe I'm a little tired, not a day goes by without thinking about this and whether the laws
...
(https://github.com/bitcoin/bitcoin/issues/32348#issuecomment-2830347260)
@willcl-ark Thanks a lot, I did Muscle Memory in Math in 2016 for Syamailcoin, and I never know if it is not possible to market under 21 Million, CoinMarketCap say I must have Big Community to Enter the Crypto Market, in September 2024 I managed to run consensus but because I didn't think about the community from the beginning, i reset all and on 13 April 2025 i did the last revision in the whitepaper.
Maybe I'm a little tired, not a day goes by without thinking about this and whether the laws
...
💬 Pimpim11 commented on issue "Double Poisson Sum: Enhancing Energy Distribution in Bitcoin Proof of Work with Main Formula γ^(i/R) + ∑(S[0:i]·τ·φ^j)":
(https://github.com/bitcoin/bitcoin/issues/32348#issuecomment-2830349113)
@willcl-ark
I am Farrel Al Feshal who is currently active as a Cryptographer, and I want to realize this Syamailcoin, a Layer-1 that I know has never existed in our country, despite the trust because of artistic tokens and exchange tokens not being obstacles for me. This is not just referential reading, but I did Muscle Memory in Math in 2016 where I felt Mathematics is not just numbers but I can travel within Mathematics itself.
In short, I have successfully launched it but according to CMC,
...
(https://github.com/bitcoin/bitcoin/issues/32348#issuecomment-2830349113)
@willcl-ark
I am Farrel Al Feshal who is currently active as a Cryptographer, and I want to realize this Syamailcoin, a Layer-1 that I know has never existed in our country, despite the trust because of artistic tokens and exchange tokens not being obstacles for me. This is not just referential reading, but I did Muscle Memory in Math in 2016 where I felt Mathematics is not just numbers but I can travel within Mathematics itself.
In short, I have successfully launched it but according to CMC,
...
:lock: fanquake locked an issue: "Double Poisson Sum: Enhancing Energy Distribution in Bitcoin Proof of Work with Main Formula γ^(i/R) + ∑(S[0:i]·τ·φ^j)"
(https://github.com/bitcoin/bitcoin/issues/32348)
(https://github.com/bitcoin/bitcoin/issues/32348)
💬 fanquake commented on pull request "test: Increase stack size for "Debug" builds with MSVC":
(https://github.com/bitcoin/bitcoin/pull/32349#discussion_r2060189048)
Where does this reserve value come from?
(https://github.com/bitcoin/bitcoin/pull/32349#discussion_r2060189048)
Where does this reserve value come from?