Page MenuHomePhabricator
Feed All Stories

Thu, Feb 27

teamcity added a comment to D17713: Prevent occasional timeout in feature_assumevalid.
Thu, Feb 27, 10:16
Fabien updated the diff for D17713: Prevent occasional timeout in feature_assumevalid.

Remove stupid timeout

Thu, Feb 27, 10:15
PiRK accepted D17713: Prevent occasional timeout in feature_assumevalid.
Thu, Feb 27, 10:15
teamcity added a comment to D17713: Prevent occasional timeout in feature_assumevalid.
Thu, Feb 27, 10:14
Fabien updated the diff for D17713: Prevent occasional timeout in feature_assumevalid.

Remove debug statements

Thu, Feb 27, 10:07
Fabien planned changes to D17713: Prevent occasional timeout in feature_assumevalid.
Thu, Feb 27, 10:05
Fabien requested review of D17713: Prevent occasional timeout in feature_assumevalid.
Thu, Feb 27, 10:05
Fabien accepted D17711: [avalanche] Return proofids in addition to payout scripts in getWinners.
Thu, Feb 27, 08:22
Fabien requested changes to D17710: [avalanche] Simplify contenders by treating rejection and invalidation the same.
Thu, Feb 27, 08:19
Fabien accepted D17706: refactor: use std::string for thread names.
Thu, Feb 27, 08:15
teamcity added a comment to D17706: refactor: use std::string for thread names.
Thu, Feb 27, 07:34
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
bytesofman accepted D17712: [e.cash] Update copy on Build page.

some suggestions trying to tighten up the copy, go with whatever you think is best. I don't think anyone reads something longer than a full sentence on the internet anymore. But we really only need a handful of people who "get it" to see this stuff, so I don't think we need to worry about general consumption too much here.

Thu, Feb 27, 04:07
teamcity added a comment to D17712: [e.cash] Update copy on Build page.

Build Bitcoin ABC Diffs / Diff Testing (preview-e.cash) passed.
Preview is available at http://51.68.37.192:41502 for the next 60 minutes.

Thu, Feb 27, 03:59
teamcity awarded D17712: [e.cash] Update copy on Build page a Doubloon token.
Thu, Feb 27, 03:58
bytesofman added a comment to D17712: [e.cash] Update copy on Build page.

@bot preview-e.cash

Thu, Feb 27, 03:58
teamcity added a comment to D17712: [e.cash] Update copy on Build page.

Build Bitcoin ABC Diffs / Diff Testing (preview-e.cash) passed.
Preview is available at http://51.68.37.192:41146 for the next 60 minutes.

Thu, Feb 27, 01:11
teamcity awarded D17712: [e.cash] Update copy on Build page a Piece of Eight token.
Thu, Feb 27, 01:10
bytesofman added a comment to D17712: [e.cash] Update copy on Build page.

@bot preview-e.cash

Thu, Feb 27, 01:10
Mengerian published D17712: [e.cash] Update copy on Build page for review.
Thu, Feb 27, 01:09
teamcity awarded D17712: [e.cash] Update copy on Build page a Haypence token.
Thu, Feb 27, 00:40

Wed, Feb 26

bytesofman updated the diff for 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).

update proto script, patch lingering "value:" keys in READMEs, downstream impacts of proto script update (other bigints, type handling)

Wed, Feb 26, 22:54
roqqit requested review of D17711: [avalanche] Return proofids in addition to payout scripts in getWinners.
Wed, Feb 26, 22:51
roqqit requested review of D17710: [avalanche] Simplify contenders by treating rejection and invalidation the same.
Wed, Feb 26, 22:34
tobias_ruck requested changes to 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).
Wed, Feb 26, 20:57
Fabien accepted D17705: indexes, refactor: Pass Chain interface instead of CChainState class to indexes.
Wed, Feb 26, 20:34
Fabien accepted D17709: [Cashtab] Parse firma yield txs.
Wed, Feb 26, 20:20
Fabien accepted D17708: [Cashtab] Update token scam prevention list.
Wed, Feb 26, 20:16
Fabien accepted D17707: refactor: use std::string for index names.
Wed, Feb 26, 20:16
Fabien requested changes to D17706: refactor: use std::string for thread names.
Wed, Feb 26, 20:13
Fabien added a comment to D17701: [Cashtab] Support one-click redemptions.

Since the button will only work based on the external api, it's better to not show the button if the api is unavailable for some reason.
I suggest you do this:

  1. Attempt to retrieve the price from that endpoint
  2. If not successful, don't show the button. Otherwise show the button with the actual price, like "Redeem FIRMA for 1234 XEC"
  3. Not sure if it's worth refreshing periodically (in the even the price changes quickly)
Wed, Feb 26, 20:10
Fabien accepted D17686: [chronik] add Electrum RPC server.donation_address.
Wed, Feb 26, 19:44
bytesofman published D17708: [Cashtab] Update token scam prevention list for review.
Wed, Feb 26, 18:31
bytesofman published D17709: [Cashtab] Parse firma yield txs for review.
Wed, Feb 26, 18:30
bytesofman published D17701: [Cashtab] Support one-click redemptions for review.
Wed, Feb 26, 17:08
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
roqqit accepted D17704: index: Move last_locator_write_time and logging to end of threadsync loop.
Wed, Feb 26, 16:43
roqqit accepted D17703: doc: BaseIndex sync behavior with empty datadir.
Wed, Feb 26, 16:40
roqqit accepted D17702: index: Don't commit a best block before indexing it during sync.
Wed, Feb 26, 16:39
PiRK updated the diff for D17686: [chronik] add Electrum RPC server.donation_address.

use the constant in strings

Wed, Feb 26, 15:57
teamcity added a comment to D17705: indexes, refactor: Pass Chain interface instead of CChainState class to indexes.
Wed, Feb 26, 15:54
teamcity updated the summary of D17707: refactor: use std::string for index names.
Wed, Feb 26, 15:47
PiRK requested review of D17707: refactor: use std::string for index names.
Wed, Feb 26, 15:47
teamcity updated the summary of D17706: refactor: use std::string for thread names.
Wed, Feb 26, 15:46
PiRK requested review of D17706: refactor: use std::string for thread names.
Wed, Feb 26, 15:45
teamcity updated the summary of D17705: indexes, refactor: Pass Chain interface instead of CChainState class to indexes.
Wed, Feb 26, 15:44
PiRK requested review of D17705: indexes, refactor: Pass Chain interface instead of CChainState class to indexes.
Wed, Feb 26, 15:44
teamcity updated the summary of D17704: index: Move last_locator_write_time and logging to end of threadsync loop.
Wed, Feb 26, 15:32
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
teamcity 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
teamcity updated the summary of D17702: index: Don't commit a best block before indexing it during sync.
Wed, Feb 26, 15:30
PiRK requested review of D17702: index: Don't commit a best block before indexing it during sync.
Wed, Feb 26, 15:30
bytesofman abandoned D17328: [token-server] Bump free xec from 42 to 88.

i think will just forget about this for now. we have good demand for the 42-xec reward and no one has complained about it being insufficient.

Wed, Feb 26, 15:08
bytesofman updated the diff for 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).

patch implementation of reduce in cashtab transactions (do not mix bigint and number)

Wed, Feb 26, 13:29
teamcity added a comment to 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).
Wed, Feb 26, 13:23
Fabien requested changes to D17686: [chronik] add Electrum RPC server.donation_address.
Wed, Feb 26, 12:59
bytesofman updated the diff for 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).

rebase, bigint in reduce, bigint math for token rewards in herald, camelCase patches in ecash-agora, var name patches in cashtab constants, hex instead of decimal for hex-defined constants

Wed, Feb 26, 12:57
bytesofman added inline comments to 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).
Wed, Feb 26, 12:52
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
Fabien requested changes to D17686: [chronik] add Electrum RPC server.donation_address.
Wed, Feb 26, 10:30
Fabien requested changes to D17700: [avalanche] Process responses to stake contender polls.
Wed, Feb 26, 10:23
tobias_ruck requested changes to 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).
Wed, Feb 26, 09:52
PiRK updated the diff for D17686: [chronik] add Electrum RPC server.donation_address.

chech the donation address length

Wed, Feb 26, 09:10
teamcity awarded D17700: [avalanche] Process responses to stake contender polls a Haypence token.
Wed, Feb 26, 09:08
Fabien added a comment to D17700: [avalanche] Process responses to stake contender polls.

@bot build-tsan

Wed, Feb 26, 09:08

Tue, Feb 25

roqqit updated the diff for D17700: [avalanche] Process responses to stake contender polls.

bump -avastalevotethreshold a bit because some rounds get voted while finalizing the first block

Tue, Feb 25, 23:49
teamcity added a comment to D17700: [avalanche] Process responses to stake contender polls.
Tue, Feb 25, 22:14
roqqit updated the diff for D17700: [avalanche] Process responses to stake contender polls.

add test cases for all invalidated and all stalled contenders

Tue, Feb 25, 22:04
teamcity added a comment to D17700: [avalanche] Process responses to stake contender polls.
Tue, Feb 25, 21:38
roqqit planned changes to D17700: [avalanche] Process responses to stake contender polls.
Tue, Feb 25, 21:30
roqqit updated the diff for D17700: [avalanche] Process responses to stake contender polls.

fixing locking issues

Tue, Feb 25, 21:30
teamcity added a comment to D17700: [avalanche] Process responses to stake contender polls.
Tue, Feb 25, 20:31
roqqit planned changes to D17700: [avalanche] Process responses to stake contender polls.

Early feedback indicates some stuff is missing:

  1. Comment on the outcomes of a stale vote.
  2. Test case if all contenders are rejected.
  3. Test case if all contenders are stalled.
Tue, Feb 25, 20:30
teamcity added a comment to D17700: [avalanche] Process responses to stake contender polls.
Tue, Feb 25, 20:30
roqqit closed D17699: [avalanche] Clarify and simplify checking polled contenders.
Tue, Feb 25, 20:23
roqqit committed rABC251c3b16b672: [avalanche] Clarify and simplify checking polled contenders (authored by roqqit).
[avalanche] Clarify and simplify checking polled contenders
Tue, Feb 25, 20:23
roqqit requested review of D17700: [avalanche] Process responses to stake contender polls.
Tue, Feb 25, 20:20
Fabien accepted D17699: [avalanche] Clarify and simplify checking polled contenders.
Tue, Feb 25, 20:16
Fabien accepted D17697: [Cashtab] Make sure whitelisted agora tokens load cache first.
Tue, Feb 25, 20:15
Fabien closed D17698: Avoid inflight polls in getavaaddr_interval_test.
Tue, Feb 25, 20:13
Fabien committed rABC7cab3cee14ab: Avoid inflight polls in getavaaddr_interval_test (authored by Fabien).
Avoid inflight polls in getavaaddr_interval_test
Tue, Feb 25, 20:13
roqqit requested review of D17699: [avalanche] Clarify and simplify checking polled contenders.
Tue, Feb 25, 20:09
teamcity closed T3759: Build Bitcoin-ABC / Master / Native tests / Test Native OSX is broken. as Resolved.
Tue, Feb 25, 16:32
roqqit accepted D17698: Avoid inflight polls in getavaaddr_interval_test.
Tue, Feb 25, 16:31
roqqit closed D17696: [avalanche] Make contender test more comprehensive.
Tue, Feb 25, 16:29
roqqit committed rABCdc86c1e90e2a: [avalanche] Make contender test more comprehensive (authored by roqqit).
[avalanche] Make contender test more comprehensive
Tue, Feb 25, 16:29
roqqit closed D17695: [avalanche] Make payout scripts unique in contenders test.
Tue, Feb 25, 16:28
roqqit committed rABC80581e12b70b: [avalanche] Make payout scripts unique in contenders test (authored by roqqit).
[avalanche] Make payout scripts unique in contenders test
Tue, Feb 25, 16:28