pablomartin4btc

Results 123 comments of pablomartin4btc

Updates: - I think [lint CI failure](https://github.com/bitcoin/bitcoin/pull/28670/checks?check_run_id=20626357772) is unrelated with the code change, it was running successful before started failing with no code change, run `./test/lint/lint-python.py` locally with no failure.

Updates: - Addressed [feedback](https://github.com/bitcoin/bitcoin/pull/28670#discussion_r1566860922) from @fjahr by amending the logging info description. - Addressed [feedback](https://github.com/bitcoin/bitcoin/pull/28670#discussion_r1597147718) from @furszy by using a better RPC error code (`RPC_DESERIALIZATION_ERROR` instead of `RPC_INTERNAL_ERROR`).

> Hm, unfortunately, I think this PR was made redundant by #29612 recently. After the latest feedback there which I addressed just 3 weeks ago, we are enhancing the assumeutxo...

I've also tested the fix above provided by @hebasto and it works as expected, thanks! Tested it running all tests successfully on Windows 11 Pro after a cross-build on Ubuntu...

I've addressed feedback from reviewers, thanks to all!

> In addition it would be good to test if you're able to load the snapshot and complete the background sync. I've already checked IBD completed fine with`[background validation]`finished, but...

> As discussed offline, the `utxo_snapshot.sh` script should have a sanity check to make the node isn't pruned below the snapshot height. On a pruned node you can only make...

Same for me but you can't perform other actions either as reported in #1231, seems it has the same root problem. Thanks to @Anudorannador, his [workaround](https://github.com/phw/peek/issues/365#issuecomment-432828444) above has worked for...