Page MenuHomePhabricator
Feed All Stories

Thu, Mar 20

PiRK accepted D17837: [avalanche] Add some missing headers.

lgtm (if the bitcoin-chainstate / kernel build does not fail)

Thu, Mar 20, 09:34
Fabien planned changes to D17836: [avalanche] Improve lock annotations for AvalancheFinalizeBlock.
Thu, Mar 20, 09:30
Fabien requested review of D17838: [avalanche] Don't include validation.h via proof.h.
Thu, Mar 20, 09:27
Fabien requested review of D17837: [avalanche] Add some missing headers.
Thu, Mar 20, 09:23
teamcity added a comment to D17836: [avalanche] Improve lock annotations for AvalancheFinalizeBlock.
Thu, Mar 20, 09:11
Fabien requested review of D17836: [avalanche] Improve lock annotations for AvalancheFinalizeBlock.
Thu, Mar 20, 09:08
PiRK added inline comments to D17831: [avalanche] Limit the finalized txs radix tree size so it always fits in the next block.
Thu, Mar 20, 08:57
PiRK added inline comments to D17831: [avalanche] Limit the finalized txs radix tree size so it always fits in the next block.
Thu, Mar 20, 08:56
PiRK added inline comments to D17831: [avalanche] Limit the finalized txs radix tree size so it always fits in the next block.
Thu, Mar 20, 08:52
PiRK added inline comments to D17835: test_framework: improve the syntax of serialize() methods.
Thu, Mar 20, 08:40
PiRK added a comment to D17835: test_framework: improve the syntax of serialize() methods.

This whole diff is a test on how Grok v3 can helpful to do semi-automated changes on a large amount of code.
It worked quite well, I just had to revert a few changes msg_avaproofs.serialize and msg_avaproofsreq.serialize, which imo were not trivial enough to justify removing the intermadiate var and all the comments to make it a one-liner.

Thu, Mar 20, 08:38
PiRK updated the diff for D17835: test_framework: improve the syntax of serialize() methods.

back-off unrelated changes

Thu, Mar 20, 08:33
PiRK planned changes to D17835: test_framework: improve the syntax of serialize() methods.
Thu, Mar 20, 08:30
PiRK requested review of D17835: test_framework: improve the syntax of serialize() methods.
Thu, Mar 20, 08:27
Fabien updated the diff for D17832: [avalanche] Poll for the transactions that didn't make it into the last block.

Fix another lock issue (cs_main => cs_peermanager)

Thu, Mar 20, 08:21
PiRK accepted D17834: [avalanche] Refactor helpers to build avaproofs messages.
Thu, Mar 20, 07:37
alitayin updated the diff for D17815: [Cashtab] Introduce CSS variables to make styling easier.

Switch to my branch and commit again.

Thu, Mar 20, 01:34
bytesofman requested changes to D17815: [Cashtab] Introduce CSS variables to make styling easier.

this diff was created on the master branch, which makes it difficult to rebase onto the latest master

Thu, Mar 20, 01:15

Wed, Mar 19

teamcity added a comment to D17832: [avalanche] Poll for the transactions that didn't make it into the last block.
Wed, Mar 19, 23:31
Fabien closed D17829: [avalanche] Add to the recently finalized items filter at callsite.
Wed, Mar 19, 23:24
Fabien committed rABCd0d90118ccea: [avalanche] Add to the recently finalized items filter at callsite (authored by Fabien).
[avalanche] Add to the recently finalized items filter at callsite
Wed, Mar 19, 23:24
Fabien updated the diff for D17832: [avalanche] Poll for the transactions that didn't make it into the last block.

Fix mempool lock

Wed, Mar 19, 23:23
Fabien accepted D17833: [avalanche] Cleanup unused prefilled proof index set.
Wed, Mar 19, 23:07
roqqit requested changes to D17832: [avalanche] Poll for the transactions that didn't make it into the last block.

back to you

Wed, Mar 19, 22:56
roqqit accepted D17829: [avalanche] Add to the recently finalized items filter at callsite.

This looks a bit odd to move it to the callsite but since it is moving in the right direction with regards to the processor FIXME, it is probably a small step in the right direction.

Wed, Mar 19, 22:53
roqqit requested review of D17834: [avalanche] Refactor helpers to build avaproofs messages.
Wed, Mar 19, 22:48
roqqit requested review of D17833: [avalanche] Cleanup unused prefilled proof index set.
Wed, Mar 19, 21:44
teamcity added a comment to D17832: [avalanche] Poll for the transactions that didn't make it into the last block.
Wed, Mar 19, 21:03
teamcity closed T3772: Build Bitcoin-ABC / Master / Native tests / Test Native OSX is broken. as Resolved.
Wed, Mar 19, 21:02
Fabien closed D17830: Bump the log crawling timeout after node restart in rpc_setban.py.
Wed, Mar 19, 20:53
Fabien committed rABC9b7827b53bdd: Bump the log crawling timeout after node restart in rpc_setban.py (authored by Fabien).
Bump the log crawling timeout after node restart in rpc_setban.py
Wed, Mar 19, 20:53
bytesofman accepted D17830: Bump the log crawling timeout after node restart in rpc_setban.py.
Wed, Mar 19, 20:51
Fabien requested review of D17832: [avalanche] Poll for the transactions that didn't make it into the last block.
Wed, Mar 19, 20:50
Fabien requested review of D17831: [avalanche] Limit the finalized txs radix tree size so it always fits in the next block.
Wed, Mar 19, 20:48
Fabien requested review of D17830: Bump the log crawling timeout after node restart in rpc_setban.py.
Wed, Mar 19, 20:15
Fabien requested review of D17829: [avalanche] Add to the recently finalized items filter at callsite.
Wed, Mar 19, 19:54
alitayin updated the diff for D17815: [Cashtab] Introduce CSS variables to make styling easier.

Completed the above modifications

Wed, Mar 19, 17:25
alitayin abandoned D17828: [Cashtab] Introduce CSS variables to make styling easier.
Wed, Mar 19, 17:22
alitayin requested review of D17828: [Cashtab] Introduce CSS variables to make styling easier.
Wed, Mar 19, 16:58
teamcity triaged T3772: Build Bitcoin-ABC / Master / Native tests / Test Native OSX is broken. as Unbreak Now! priority.
Wed, Mar 19, 15:09
bytesofman added a comment to D17815: [Cashtab] Introduce CSS variables to make styling easier.
  • If we are switching from px to rem, switch; remove all the px comments, mb keep one to show what the base-level px is
  • do not use calc. if some ratios seem unreadable, create a variable for them and use the variable
  • remove all unused variables
Wed, Mar 19, 14:46
bytesofman closed D17826: [install readme] Add deps needed to build with chronik and plugins.
Wed, Mar 19, 14:42
bytesofman committed rABC0ba12c26bb6d: [install readme] Add deps needed to build with chronik and plugins (authored by bytesofman).
[install readme] Add deps needed to build with chronik and plugins
Wed, Mar 19, 14:42
Fabien accepted D17826: [install readme] Add deps needed to build with chronik and plugins.
Wed, Mar 19, 14:28
bytesofman updated the diff for D17826: [install readme] Add deps needed to build with chronik and plugins.

back out the pythonpath stuff, add dep

Wed, Mar 19, 14:25
bytesofman updated the diff for D17826: [install readme] Add deps needed to build with chronik and plugins.

debian python note

Wed, Mar 19, 13:58
bytesofman published D17826: [install readme] Add deps needed to build with chronik and plugins for review.
Wed, Mar 19, 13:56
teamcity closed T3771: Build Bitcoin-ABC / Nightly Builds / GUIX Builds / GUIX Linux is broken. as Resolved.
Wed, Mar 19, 12:45
bytesofman closed D17821: [Cashtab] Patch bug in rendering of managed NFTs.
Wed, Mar 19, 12:26
bytesofman committed rABCb50acc96032b: [Cashtab] Patch bug in rendering of managed NFTs (authored by bytesofman).
[Cashtab] Patch bug in rendering of managed NFTs
Wed, Mar 19, 12:26
PiRK closed D17816: [electrum] bump the version to 5.4.1 and add release notes.
Wed, Mar 19, 11:46
PiRK committed rABCe5e31cf6c69a: [electrum] bump the version to 5.4.1 and add release notes (authored by PiRK).
[electrum] bump the version to 5.4.1 and add release notes
Wed, Mar 19, 11:46
Fabien closed D17824: [electrum] No python 3.12 syntax.
Wed, Mar 19, 11:44
Fabien committed rABCaf031872bf01: [electrum] No python 3.12 syntax (authored by Fabien).
[electrum] No python 3.12 syntax
Wed, Mar 19, 11:44
Fabien closed D17825: [electrum] Pin version of shamir-mnemonic.
Wed, Mar 19, 11:44
Fabien committed rABCe948318adde9: [electrum] Pin version of shamir-mnemonic (authored by Fabien).
[electrum] Pin version of shamir-mnemonic
Wed, Mar 19, 11:44
PiRK closed D17823: [electrum] fix pyopenssl version.
Wed, Mar 19, 10:52
PiRK committed rABC79a912688729: [electrum] fix pyopenssl version (authored by Jonas Lundqvist <jonas@gannon.se>).
[electrum] fix pyopenssl version
Wed, Mar 19, 10:52
PiRK accepted D17825: [electrum] Pin version of shamir-mnemonic.
Wed, Mar 19, 10:44
PiRK accepted D17824: [electrum] No python 3.12 syntax.
Wed, Mar 19, 10:43
Fabien requested review of D17825: [electrum] Pin version of shamir-mnemonic.
Wed, Mar 19, 10:27
Fabien accepted D17821: [Cashtab] Patch bug in rendering of managed NFTs.
Wed, Mar 19, 10:26
Fabien accepted D17823: [electrum] fix pyopenssl version.
Wed, Mar 19, 10:25
Fabien requested review of D17824: [electrum] No python 3.12 syntax.
Wed, Mar 19, 10:24
PiRK updated the summary of D17823: [electrum] fix pyopenssl version.
Wed, Mar 19, 09:10
PiRK requested review of D17823: [electrum] fix pyopenssl version.
Wed, Mar 19, 09:09
Fabien closed D17820: [CI] Fix the chronik-plugins docker image.
Wed, Mar 19, 08:31
Fabien committed rABC844e4063c18b: [CI] Fix the chronik-plugins docker image (authored by Fabien).
[CI] Fix the chronik-plugins docker image
Wed, Mar 19, 08:31
PiRK closed D17819: [electrum] include metadata for slip10 for the windows and osx releases.
Wed, Mar 19, 08:17
PiRK committed rABCa370271feb5e: [electrum] include metadata for slip10 for the windows and osx releases (authored by PiRK).
[electrum] include metadata for slip10 for the windows and osx releases
Wed, Mar 19, 08:17
Fabien accepted D17819: [electrum] include metadata for slip10 for the windows and osx releases.
Wed, Mar 19, 08:15
PiRK published D17819: [electrum] include metadata for slip10 for the windows and osx releases for review.
Wed, Mar 19, 07:46
alitayin added inline comments to D17815: [Cashtab] Introduce CSS variables to make styling easier.
Wed, Mar 19, 04:45
bytesofman added inline comments to D17815: [Cashtab] Introduce CSS variables to make styling easier.
Wed, Mar 19, 03:28
bytesofman published D17821: [Cashtab] Patch bug in rendering of managed NFTs for review.
Wed, Mar 19, 03:18
alitayin added a comment to D17815: [Cashtab] Introduce CSS variables to make styling easier.

would be a good change and help to get some of the css more standardized in Cashtab

Wed, Mar 19, 03:02
alitayin added a comment to D17815: [Cashtab] Introduce CSS variables to make styling easier.
Wed, Mar 19, 02:51
alitayin added inline comments to D17815: [Cashtab] Introduce CSS variables to make styling easier.
Wed, Mar 19, 02:44

Tue, Mar 18

bytesofman accepted D17820: [CI] Fix the chronik-plugins docker image.
Tue, Mar 18, 23:29
Fabien requested review of D17820: [CI] Fix the chronik-plugins docker image.
Tue, Mar 18, 22:07
bytesofman requested changes to D17815: [Cashtab] Introduce CSS variables to make styling easier.

would be a good change and help to get some of the css more standardized in Cashtab

Tue, Mar 18, 20:09
bytesofman closed D17814: [chronik-client] Patch failoverProxy to recognize another type of server failure.
Tue, Mar 18, 19:05
bytesofman committed rABCcdbb66df256c: [chronik-client] Patch failoverProxy to recognize another type of server failure (authored by bytesofman).
[chronik-client] Patch failoverProxy to recognize another type of server failure
Tue, Mar 18, 19:05
bytesofman updated the diff for D17814: [chronik-client] Patch failoverProxy to recognize another type of server failure.

better comments

Tue, Mar 18, 18:34
Fabien closed D17817: [avalanche] Improve performance of inventory filtering.
Tue, Mar 18, 17:14
Fabien committed rABC88dc412d3367: [avalanche] Improve performance of inventory filtering (authored by Fabien).
[avalanche] Improve performance of inventory filtering
Tue, Mar 18, 17:14
roqqit committed rABCf7b0bdf7c762: [electrum] Fix typo in trezor firmware warning text (authored by roqqit).
[electrum] Fix typo in trezor firmware warning text
Tue, Mar 18, 16:31
roqqit closed D17818: [electrum] Fix typo in trezor firmware warning text.
Tue, Mar 18, 16:31
Mengerian accepted D17818: [electrum] Fix typo in trezor firmware warning text.
Tue, Mar 18, 16:07
roqqit requested review of D17818: [electrum] Fix typo in trezor firmware warning text.
Tue, Mar 18, 15:52
PiRK accepted D17817: [avalanche] Improve performance of inventory filtering.
Tue, Mar 18, 15:50
roqqit accepted D17817: [avalanche] Improve performance of inventory filtering.
Tue, Mar 18, 15:46
Fabien accepted D17816: [electrum] bump the version to 5.4.1 and add release notes.
Tue, Mar 18, 15:39
PiRK updated the diff for D17816: [electrum] bump the version to 5.4.1 and add release notes.

mention which devices have a custom Trezor firmware

Tue, Mar 18, 15:38
Fabien requested review of D17817: [avalanche] Improve performance of inventory filtering.
Tue, Mar 18, 15:32
PiRK updated the diff for D17816: [electrum] bump the version to 5.4.1 and add release notes.

reflow release note

Tue, Mar 18, 15:28
PiRK requested review of D17816: [electrum] bump the version to 5.4.1 and add release notes.
Tue, Mar 18, 15:25
alitayin updated the diff for D17815: [Cashtab] Introduce CSS variables to make styling easier.

[Cashtab] Introduce CSS variables to make styling easier

Tue, Mar 18, 15:16
alitayin requested review of D17815: [Cashtab] Introduce CSS variables to make styling easier.
Tue, Mar 18, 15:00
PiRK closed D17785: [electrum] install a custom version of trezorlib.
Tue, Mar 18, 14:49