志宇
志宇
@storopoli sorry can you do a rebase on master? thanks
> > > This can be fixed with -- --test-threads=2. Shall we add? > > > > > > Why not increase [bitcoind's rpc threads and work queue](https://github.com/fedimint/fedimint/blob/32bac957b23c0facfaff32baaa427d4898bce06d/devimint/src/cfg/bitcoin.conf#L9C10-L10C10) from the...
> > 64 threads is way too much. I think the ideal is to have 1 thread per CPU core. I think a good compromise is 4 (between desktop and...
> @storopoli does this PR substitute #1257? Shall we close that or do we need to leave it open? Yes it does. Closing #1257 Edit: > This is a simpler...
Thanks for creating a ticket. Everyone was thinking this for a long while.
Concept ACK, approach NACK. This will be a great feature to have! However, I think we are limited by the Esplora/Electrum APIs (let me know if I have missed something)....
> Oh got it! That is trickier to implement than I thought. Am I correct in that the `last_seen` field is only intended for mempool transactions, and would be appropriate...
> > Currently, the nature of `TxGraph` is fully monotone. Nothing can be added or removed. > > I think things can be added! Sorry *nothing can be removed 😅
Also, what do you think about naming the crate `bdk_sqlite`? The term `store` doesn't add any more meaning to the name.
I'm very happy with this codebase, thanks for spearheading this effort @notmandatory. Just the things I've mentioned above and I'll be happy to ACK.