Page MenuHomePhabricator

PiRK (Pierre K)
User

Projects

User does not belong to any projects.

User Details

User Since
Aug 5 2020, 16:14 (238 w, 4 d)

Recent Activity

Today

PiRK closed D17730: tests: use noban_tx_relay in more tests.
Mon, Mar 3, 08:11
PiRK committed rABCc71468342225: tests: use noban_tx_relay in more tests (authored by PiRK).
tests: use noban_tx_relay in more tests
Mon, Mar 3, 08:11

Fri, Feb 28

PiRK accepted D17729: [avalanche] Fix a race in contenders voting test.
Fri, Feb 28, 21:21
PiRK requested review of D17730: tests: use noban_tx_relay in more tests.
Fri, Feb 28, 21:19
PiRK closed D17722: indexes, refactor: Remove CBlockIndex* uses in index Init methods.
Fri, Feb 28, 20:13
PiRK committed rABC3f303c2e897e: indexes, refactor: Remove CBlockIndex* uses in index Init methods (authored by Ryan Ofsky <ryan@ofsky.org>).
indexes, refactor: Remove CBlockIndex* uses in index Init methods
Fri, Feb 28, 20:13
PiRK closed D17721: indexes, refactor: Remove CBlockIndex* uses in coinstatsindex LookUpOne function.
Fri, Feb 28, 20:13
PiRK committed rABCef4a1052f5f0: indexes, refactor: Remove CBlockIndex* uses in coinstatsindex LookUpOne function (authored by Ryan Ofsky <ryan@ofsky.org>).
indexes, refactor: Remove CBlockIndex* uses in coinstatsindex LookUpOne function
Fri, Feb 28, 20:13
PiRK accepted D17725: [ecash-herald] Send daily summaries to the main chat.
Fri, Feb 28, 20:11
PiRK accepted D17724: [ecash-herald] Do not use polling for the telegram bot.
Fri, Feb 28, 20:10
PiRK updated the diff for D17723: [kernel 2c/n] Introduce kernel::Context, encapsulate global init/teardown.

rm Makefile.am

Fri, Feb 28, 15:26
PiRK requested review of D17723: [kernel 2c/n] Introduce kernel::Context, encapsulate global init/teardown.
Fri, Feb 28, 15:25
PiRK accepted D17717: [avalanche] Also finalize the ancestors of a transaction.
Fri, Feb 28, 13:44
PiRK requested review of D17722: indexes, refactor: Remove CBlockIndex* uses in index Init methods.
Fri, Feb 28, 13:08
PiRK requested review of D17721: indexes, refactor: Remove CBlockIndex* uses in coinstatsindex LookUpOne function.
Fri, Feb 28, 13:07
PiRK closed D17714: index: fix reindex-chainstate with active indexes.
Fri, Feb 28, 12:10
PiRK committed rABC1583f9118eba: index: fix reindex-chainstate with active indexes (authored by Martin Zumsande <mzumsande@gmail.com>).
index: fix reindex-chainstate with active indexes
Fri, Feb 28, 12:10
PiRK updated the diff for D17714: index: fix reindex-chainstate with active indexes.

use -noparkdeepreorg and mine 5 blocks

Fri, Feb 28, 07:47

Thu, Feb 27

PiRK added inline comments to D17714: index: fix reindex-chainstate with active indexes.
Thu, Feb 27, 16:04
PiRK updated the diff for D17714: index: fix reindex-chainstate with active indexes.

fix clang-tidy style error, fix test flakiness (mine enough new block to trigger automatic unparking on the other node, 5 was not enough)

Thu, Feb 27, 15:58
PiRK planned changes to D17714: index: fix reindex-chainstate with active indexes.
Thu, Feb 27, 15:17
PiRK requested review of D17714: index: fix reindex-chainstate with active indexes.
Thu, Feb 27, 15:10
PiRK closed D17706: refactor: use std::string for thread names.
Thu, Feb 27, 10:21
PiRK committed rABC2d464809a97d: refactor: use std::string for thread names (authored by stickies-v <stickies-v@protonmail.com>).
refactor: use std::string for thread names
Thu, Feb 27, 10:21
PiRK committed rABC3926deb8fa05: refactor: use std::string for index names (authored by stickies-v <stickies-v@protonmail.com>).
refactor: use std::string for index names
Thu, Feb 27, 10:21
PiRK closed D17707: refactor: use std::string for index names.
Thu, Feb 27, 10:21
PiRK closed D17705: indexes, refactor: Pass Chain interface instead of CChainState class to indexes.
Thu, Feb 27, 10:21
PiRK committed rABC94e4f949ce51: indexes, refactor: Pass Chain interface instead of CChainState class to indexes (authored by Ryan Ofsky <ryan@ofsky.org>).
indexes, refactor: Pass Chain interface instead of CChainState class to indexes
Thu, Feb 27, 10:21
PiRK accepted D17713: Prevent occasional timeout in feature_assumevalid.
Thu, Feb 27, 10:15
PiRK closed D17686: [chronik] add Electrum RPC server.donation_address.
Thu, Feb 27, 07:28
PiRK committed rABC1e96a8544247: [chronik] add Electrum RPC server.donation_address (authored by PiRK).
[chronik] add Electrum RPC server.donation_address
Thu, Feb 27, 07:28
PiRK updated the diff for D17706: refactor: use std::string for thread names.

remove unused include

Thu, Feb 27, 07:23
PiRK updated the diff for D17705: indexes, refactor: Pass Chain interface instead of CChainState class to indexes.

remove unnecessary forward declaration

Thu, Feb 27, 07:22

Wed, Feb 26

PiRK added a comment to D17705: indexes, refactor: Pass Chain interface instead of CChainState class to indexes.

I'll keep looking what's wrong with feature_assumevalid, but it is not related to this diff.

Wed, Feb 26, 16:59
PiRK closed D17704: index: Move last_locator_write_time and logging to end of threadsync loop.
Wed, Feb 26, 16:54
PiRK closed D17703: doc: BaseIndex sync behavior with empty datadir.
Wed, Feb 26, 16:54
PiRK committed rABCa23a2ae0eda8: index: Move last_locator_write_time and logging to end of threadsync loop (authored by Fabian Jahr <fjahr@protonmail.com>).
index: Move last_locator_write_time and logging to end of threadsync loop
Wed, Feb 26, 16:54
PiRK committed rABC8e4b9a1ff950: doc: BaseIndex sync behavior with empty datadir (authored by James O'Beirne <james.obeirne@pm.me>).
doc: BaseIndex sync behavior with empty datadir
Wed, Feb 26, 16:54
PiRK committed rABCef3b321b2815: index: Don't commit a best block before indexing it during sync (authored by Martin Zumsande <mzumsande@gmail.com>).
index: Don't commit a best block before indexing it during sync
Wed, Feb 26, 16:54
PiRK closed D17702: index: Don't commit a best block before indexing it during sync.
Wed, Feb 26, 16:54
PiRK updated the diff for D17686: [chronik] add Electrum RPC server.donation_address.

use the constant in strings

Wed, Feb 26, 15:57
PiRK requested review of D17707: refactor: use std::string for index names.
Wed, Feb 26, 15:47
PiRK requested review of D17706: refactor: use std::string for thread names.
Wed, Feb 26, 15:45
PiRK requested review of D17705: indexes, refactor: Pass Chain interface instead of CChainState class to indexes.
Wed, Feb 26, 15:44
PiRK requested review of D17704: index: Move last_locator_write_time and logging to end of threadsync loop.
Wed, Feb 26, 15:32
PiRK updated the summary of D17703: doc: BaseIndex sync behavior with empty datadir.
Wed, Feb 26, 15:32
PiRK requested review of D17703: doc: BaseIndex sync behavior with empty datadir.
Wed, Feb 26, 15:31
PiRK requested review of D17702: index: Don't commit a best block before indexing it during sync.
Wed, Feb 26, 15:30
PiRK updated the diff for D17686: [chronik] add Electrum RPC server.donation_address.

make max length a constexpr

Wed, Feb 26, 10:52
PiRK updated the summary of D17686: [chronik] add Electrum RPC server.donation_address.
Wed, Feb 26, 10:40
PiRK updated the diff for D17686: [chronik] add Electrum RPC server.donation_address.

chech the donation address length

Wed, Feb 26, 09:10

Tue, Feb 25

PiRK committed rABC40402b2ac954: wallet: Ignore chainStateFlushed notifications while attaching chain (authored by Martin Zumsande <mzumsande@gmail.com>).
wallet: Ignore chainStateFlushed notifications while attaching chain
Tue, Feb 25, 11:32
PiRK closed D17689: wallet: Ignore chainStateFlushed notifications while attaching chain.
Tue, Feb 25, 11:32
PiRK updated the diff for D17689: wallet: Ignore chainStateFlushed notifications while attaching chain.

remove first comment which would mess with formatting if left inline and is already explained by the previous block of comments
add a comment as suggested to first call m_attaching_chain = false

Tue, Feb 25, 10:37
PiRK updated the diff for D17689: wallet: Ignore chainStateFlushed notifications while attaching chain.

rebase

Tue, Feb 25, 08:34
PiRK accepted D17695: [avalanche] Make payout scripts unique in contenders test.
Tue, Feb 25, 08:32
PiRK closed D17688: doc: Improve ZMQ documentation.
Tue, Feb 25, 08:31
PiRK committed rABCc52393d22e61: doc: Improve ZMQ documentation (authored by node01 <mbildwic@protonmail.com>).
doc: Improve ZMQ documentation
Tue, Feb 25, 08:31
PiRK accepted D17693: [avalanche] Cleanup payout address optionality.
Tue, Feb 25, 08:23
PiRK accepted D17694: test: Ignore UTF-8 errors in assert_debug_log.
Tue, Feb 25, 08:22
PiRK closed D17691: test: refactor: use `create_block` parameters.
Tue, Feb 25, 08:12
PiRK committed rABCf4d4a586a65b: test: refactor: use `create_block` parameters (authored by Sebastian Falbesoner <sebastian.falbesoner@gmail.com>).
test: refactor: use `create_block` parameters
Tue, Feb 25, 08:12

Mon, Feb 24

PiRK committed rABC33c0dc6359a1: test: remove unnecessary block rehashing prior to solving (authored by Sebastian Falbesoner <sebastian.falbesoner@gmail.com>).
test: remove unnecessary block rehashing prior to solving
Mon, Feb 24, 17:08
PiRK closed D17690: test: remove unnecessary block rehashing prior to solving.
Mon, Feb 24, 17:08
PiRK added inline comments to D17691: test: refactor: use `create_block` parameters.
Mon, Feb 24, 16:31
PiRK updated the diff for D17691: test: refactor: use `create_block` parameters.

don't shadow j either

Mon, Feb 24, 16:28
PiRK requested review of D17691: test: refactor: use `create_block` parameters.
Mon, Feb 24, 16:26
PiRK requested review of D17690: test: remove unnecessary block rehashing prior to solving.
Mon, Feb 24, 13:21

Sun, Feb 23

PiRK planned changes to D17689: wallet: Ignore chainStateFlushed notifications while attaching chain.

CI failure most likely unrelated, but will have to investigate anyway (tomorrow)

Sun, Feb 23, 09:25
PiRK requested review of D17689: wallet: Ignore chainStateFlushed notifications while attaching chain.
Sun, Feb 23, 09:12

Sat, Feb 22

PiRK added a comment to D17688: doc: Improve ZMQ documentation.

Backported as a dependency of https://github.com/bitcoin/bitcoin/pull/27596/commits/f073917a9e7ba423643dcae0339776470b628f65

Sat, Feb 22, 15:19
PiRK requested review of D17688: doc: Improve ZMQ documentation.
Sat, Feb 22, 15:19
PiRK updated the diff for D17686: [chronik] add Electrum RPC server.donation_address.

fix comment layout

Sat, Feb 22, 09:21

Fri, Feb 21

PiRK requested review of D17686: [chronik] add Electrum RPC server.donation_address.
Fri, Feb 21, 16:57
PiRK accepted D17650: [chronik, chronik-client, ecash-agora, ecash-lib, cashtab, ecash-herald, token-server, faucet, mock-chronik-client] Improve names of keys and params that expect satoshis and atoms (aka base tokens).

Still lgtm

Fri, Feb 21, 14:20
PiRK accepted D17684: [chronik] Fix the chronik_plugins test.
Fri, Feb 21, 14:05
PiRK accepted D17682: Fix nullptr clientModel access during shutdown.
Fri, Feb 21, 13:42
PiRK closed D17681: [chronik] make the maximum history length for electrum RPCs an init param.
Fri, Feb 21, 10:35
PiRK committed rABCb760220009df: [chronik] make the maximum history length for electrum RPCs an init param (authored by PiRK).
[chronik] make the maximum history length for electrum RPCs an init param
Fri, Feb 21, 10:35

Thu, Feb 20

PiRK retitled D17681: [chronik] make the maximum history length for electrum RPCs an init param from [chronik] make the maximum history length an init param to [chronik] make the maximum history length for electrum RPCs an init param.
Thu, Feb 20, 19:36
PiRK requested review of D17681: [chronik] make the maximum history length for electrum RPCs an init param.
Thu, Feb 20, 19:21
PiRK closed D17676: [chronik] implement server.version electrum method.
Thu, Feb 20, 07:38
PiRK committed rABC91ef9971ad45: [chronik] implement server.version electrum method (authored by PiRK).
[chronik] implement server.version electrum method
Thu, Feb 20, 07:38
PiRK accepted D17650: [chronik, chronik-client, ecash-agora, ecash-lib, cashtab, ecash-herald, token-server, faucet, mock-chronik-client] Improve names of keys and params that expect satoshis and atoms (aka base tokens).
Thu, Feb 20, 07:24

Wed, Feb 19

PiRK accepted D17677: Improve reliability of abc_rpc_getavalancheproofs.
Wed, Feb 19, 11:50
PiRK accepted D17675: [chronik] Consistently sync the validation queue in functional tests.
Wed, Feb 19, 11:49
PiRK updated the diff for D17676: [chronik] implement server.version electrum method.

remove done TODO

Wed, Feb 19, 09:35
PiRK added inline comments to D17676: [chronik] implement server.version electrum method.
Wed, Feb 19, 09:34
PiRK updated the summary of D17676: [chronik] implement server.version electrum method.
Wed, Feb 19, 09:33
PiRK requested review of D17676: [chronik] implement server.version electrum method.
Wed, Feb 19, 09:33

Tue, Feb 18

PiRK accepted D17674: [avalanche] Deprecate the node availability score in the peerinfo RPC.
Tue, Feb 18, 17:12
PiRK committed rABC9205e1004401: test: Add explicit onion bind to p2p_permissions (authored by Ava Chow <github@achow101.com>).
test: Add explicit onion bind to p2p_permissions
Tue, Feb 18, 14:46
PiRK closed D17673: test: Add explicit onion bind to p2p_permissions.
Tue, Feb 18, 14:46
PiRK requested review of D17673: test: Add explicit onion bind to p2p_permissions.
Tue, Feb 18, 13:19
PiRK closed D17672: [chronik] Implement blockchain.scripthash.listunspent electrum method.
Tue, Feb 18, 12:12
PiRK committed rABCe09df7e9e3ab: [chronik] Implement blockchain.scripthash.listunspent electrum method (authored by PiRK).
[chronik] Implement blockchain.scripthash.listunspent electrum method
Tue, Feb 18, 12:12
PiRK closed D17670: [electrum] support newer protobuf library versions.
Tue, Feb 18, 12:12
PiRK committed rABC36570956a320: [electrum] support newer protobuf library versions (authored by PiRK).
[electrum] support newer protobuf library versions
Tue, Feb 18, 12:12
PiRK accepted D17671: Re-enable the getavaaddr_interval_test.

let's see if it pops up on CI again

Tue, Feb 18, 09:41
PiRK requested review of D17672: [chronik] Implement blockchain.scripthash.listunspent electrum method.
Tue, Feb 18, 09:22