< adulrunaredviva>
good morning everybody. What is the money transfer fee variable? It depends on what? I will be happy if you give me information. Thank you.
< sipa>
adulrunaredviva: https://bitcoin.stackexchange.com is a better place for these questions; in short: the price is set by supply and demand
< adulrunaredviva>
What does the "settxfee" command do? Is there any connection with the global transaction fee?
< sipa>
adulrunaredviva: not here, this is a development channel
< bitcoin-git>
[bitcoin] jnewbery opened pull request #20282: Wallet: Change upgradewallet return type to be an object (master...2020-11-upgrade-wallet-return) https://github.com/bitcoin/bitcoin/pull/20282
< bitcoin-git>
[bitcoin] MarcoFalke opened pull request #20283: test: Only try witness deser when checking for witness deser failure (master...2011-testWitnessFail) https://github.com/bitcoin/bitcoin/pull/20283
< bitcoin-git>
bitcoin/master c5ec036 MarcoFalke: Merge #20165: Only relay Taproot spends if next block has it active
< bitcoin-git>
[bitcoin] MarcoFalke merged pull request #20165: Only relay Taproot spends if next block has it active (master...202010_taproot_policy) https://github.com/bitcoin/bitcoin/pull/20165
< bitcoin-git>
[bitcoin] vasild opened pull request #20284: addrman: ensure old versions don't parse peers.dat (master...peers_dat_format) https://github.com/bitcoin/bitcoin/pull/20284
< bitcoin-git>
[bitcoin] fanquake opened pull request #20285: Remove references to CreateWalletFromFile (master...createwalletfromfilenomore) https://github.com/bitcoin/bitcoin/pull/20285
< bitcoin-git>
[gui] hebasto opened pull request #127: Replace QMetaObject::invokeMethod with Mutex locking (master...201102-queued) https://github.com/bitcoin-core/gui/pull/127
< shesek>
I was not able to reliably reproduce this yet, but I got `"scanning":{"duration":19,"progress":}}` (with no value for `progress`) in response to `getwalletinfo` when rescanning is in progress
< shesek>
I'm still trying to reproduce and will report back if I have more information, but perhaps someone familiar with the wallet internals have some ideas on potential causes?
< shesek>
it just happened again, but I'm still not sure why exactly. it didn't happen before for like 15-20 attempts in a row. seems like something racey is going on
< shesek>
another thing I noticed is that getnetworkinfo and getwalletinfo are very slow to reply when a rescan is taking place, is that to be expected?
< shesek>
I'm running v0.20.0 and testing against a regtest instance. this happened while doing a batched `importmulti` of 2000 addresses, none of which have any history.
< shesek>
I also noticed during my attempts to reproduce this that the `progress` field is always either 0 or missing, it never reports any progress
< vasild>
jnewbery: how to propose a topic for the P2P meeting? I guess #proposedmeetingtopic will do so for the general dev meeting on Thu.
< wumpus>
vasild: I don't know the exact rules but yes, C/C++ really likes to promote things to 'int'
< jonatack>
vasild: am reviewing 20284, and a propos, was looking at why the cast was needed: s << static_cast<uint8_t>(m_incompatibility_base + lowest_compatible);
< jonatack>
i was surprised, but it is
< vasild>
yes, poor me expected that typeX + typeX will be of typeX
< jonatack>
yes
< jonatack>
same
< jonatack>
i wonder if the cast can still be avoided though
< wumpus>
one of the things I like about rust is that it does not have silent promotions or truncations like that; want to cast between sized integer types? you always need to be explicit about it
< sipa>
jonatack, wumpus: there is a very annoying rule in C++ (and C) that any operations on a type smaller than int automatically get upcasted to int
< jonatack>
https://en.cppreference.com/w/cpp/language/implicit_conversion "In particular, arithmetic operators do not accept types smaller than int as arguments, and integral promotions are automatically applied after lvalue-to-rvalue conversion, if applicable."
< bitcoin-git>
bitcoin/master a357111 Andrew Chow: Update wallet_importprunedfunds to avoid dumpprivkey
< bitcoin-git>
bitcoin/master dc81418 Andrew Chow: Use a separate watchonly wallet in rpc_fundrawtransaction.py
< bitcoin-git>
bitcoin/master 553dbf9 Andrew Chow: Make import tests in wallet_listtransactions.py legacy wallet only
< bitcoin-git>
[bitcoin] laanwj merged pull request #18788: tests: Update more tests to work with descriptor wallets (master...desc-wallet-tests) https://github.com/bitcoin/bitcoin/pull/18788
< bitcoin-git>
[bitcoin] practicalswift opened pull request #20289: fuzz: Check for addrv1 compatibility before using addrv1 serializer/deserializer on CService (master...fuzzers-service_deserialize-addrv2) https://github.com/bitcoin/bitcoin/pull/20289