Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Read error: Connection reset by peer]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 268 seconds]
_apex2_ has joined #bitcoin-core-dev
Guest34 has joined #bitcoin-core-dev
Guest34 has quit [Client Quit]
_apex2_ has quit [Ping timeout: 268 seconds]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Cory has quit [Ping timeout: 268 seconds]
Pasha has joined #bitcoin-core-dev
Kaizen_K_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 268 seconds]
Kaizen_K_ has quit [Ping timeout: 252 seconds]
vysn has quit [Ping timeout: 268 seconds]
Guest26 has joined #bitcoin-core-dev
Guest26 has quit [Client Quit]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Kaizen_K_ has joined #bitcoin-core-dev
<ariard> laanwj: many thanks for the ~10y of project maintenance and the standard set in terms of dedication and commitment in that space, I wish you the really best for your personal projects and I hope you'll keep contributing to the repo
Kaizen_Kintsugi_ has quit [Ping timeout: 268 seconds]
AaronvanW has quit [Remote host closed the connection]
_apex2_ has joined #bitcoin-core-dev
Kaizen_K_ has quit [Remote host closed the connection]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
_apex2_ has quit [Ping timeout: 268 seconds]
AaronvanW has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 268 seconds]
Kaizen_Kintsugi_ has quit [Remote host closed the connection]
bomb-on has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 252 seconds]
yanmaani has quit [Remote host closed the connection]
yanmaani has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
AaronvanW has quit [Ping timeout: 252 seconds]
brunoerg has quit [Ping timeout: 268 seconds]
_apex2_ has joined #bitcoin-core-dev
_apex2_ has quit [Ping timeout: 268 seconds]
brunoerg has joined #bitcoin-core-dev
_apex2_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 252 seconds]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
_apex2_ has quit [Ping timeout: 268 seconds]
Kaizen_Kintsugi_ has quit [Ping timeout: 268 seconds]
bomb-on has quit [Quit: aллилѹіа!]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
PaperSword has joined #bitcoin-core-dev
_apex2_ has joined #bitcoin-core-dev
bitdex has joined #bitcoin-core-dev
_apex2_ has quit [Ping timeout: 268 seconds]
cmirror has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 268 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 244 seconds]
brunoerg has joined #bitcoin-core-dev
AaronvanW has joined #bitcoin-core-dev
vasild has quit [Remote host closed the connection]
vasild has joined #bitcoin-core-dev
SpellChecker has quit [Remote host closed the connection]
SpellChecker has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 268 seconds]
AaronvanW has quit [Ping timeout: 268 seconds]
Evel-Knievel has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
vasild has quit [Remote host closed the connection]
Kaizen_Kintsugi_ has quit [Ping timeout: 244 seconds]
vasild has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 268 seconds]
brunoerg has quit [Ping timeout: 268 seconds]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 255 seconds]
brunoerg has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] w0xlt opened pull request #25827: descriptor: check if `rawtr` has only one key. (master...expr_rawtr) https://github.com/bitcoin/bitcoin/pull/25827
_apex2_ has joined #bitcoin-core-dev
_apex2_ has quit [Ping timeout: 268 seconds]
AaronvanW has joined #bitcoin-core-dev
bitdex has quit [Remote host closed the connection]
bitdex has joined #bitcoin-core-dev
gossie_ has quit [Ping timeout: 268 seconds]
gossie_ has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 268 seconds]
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/5d294bccbae5...27724c23f72b
<bitcoin-git> bitcoin/master 9376a6d Aurèle Oulès: refactor: make active_chain_tip a reference
<bitcoin-git> bitcoin/master 27724c2 MacroFake: Merge bitcoin/bitcoin#25677: refactor: make active_chain_tip a reference
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #25677: refactor: make active_chain_tip a reference (master...2022-07-make-active-chain-ref) https://github.com/bitcoin/bitcoin/pull/25677
yanmaani has quit [Remote host closed the connection]
SpellChecker has quit [Remote host closed the connection]
SpellChecker has joined #bitcoin-core-dev
yanmaani has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 252 seconds]
SpellChecker has quit [Remote host closed the connection]
SpellChecker has joined #bitcoin-core-dev
SpellChecker has quit [Remote host closed the connection]
SpellChecker has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Read error: Connection reset by peer]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 268 seconds]
MacroFake has quit [Quit: ZNC 1.7.5+deb4 - https://znc.in]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
MacroFake has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
otoburb has quit [Quit: leaving]
_apex2_ has joined #bitcoin-core-dev
_apex2_ has quit [Ping timeout: 268 seconds]
vysn has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 268 seconds]
brunoerg has quit [Ping timeout: 268 seconds]
bomb-on has joined #bitcoin-core-dev
bomb-on has quit [Client Quit]
bomb-on has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
bomb-on has quit [Read error: Connection reset by peer]
bomb-on has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 268 seconds]
ghost43 has quit [Remote host closed the connection]
ghost43 has joined #bitcoin-core-dev
jonatack has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
ghost43 has quit [Remote host closed the connection]
ghost43 has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 255 seconds]
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/27724c23f72b...89b219412495
<bitcoin-git> bitcoin/master daabd41 Vasil Dimov: net: simplify GetLocalAddress()
<bitcoin-git> bitcoin/master 89b2194 MacroFake: Merge bitcoin/bitcoin#25814: net: simplify GetLocalAddress()
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #25814: net: simplify GetLocalAddress() (master...simplify_GetLocalAddress) https://github.com/bitcoin/bitcoin/pull/25814
vysn has quit [Ping timeout: 244 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 268 seconds]
<bitcoin-git> [bitcoin] hebasto opened pull request #25828: [23.x] GUI backports (23.x...220812-23.x-gui) https://github.com/bitcoin/bitcoin/pull/25828
yanmaani has quit [Remote host closed the connection]
yanmaani has joined #bitcoin-core-dev
otoburb has joined #bitcoin-core-dev
otoburb has quit [Client Quit]
brunoerg has joined #bitcoin-core-dev
_apex2_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 244 seconds]
_apex2_ has quit [Ping timeout: 268 seconds]
ghost43 has quit [Remote host closed the connection]
ghost43 has joined #bitcoin-core-dev
dviola has quit [Ping timeout: 268 seconds]
AaronvanW has quit [Remote host closed the connection]
otoburb has joined #bitcoin-core-dev
otoburb has quit [Remote host closed the connection]
otoburb has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake opened pull request #25829: build: include share/rpcauth in release tarball (master...fixup_19081) https://github.com/bitcoin/bitcoin/pull/25829
dviola has joined #bitcoin-core-dev
AaronvanW has joined #bitcoin-core-dev
vysn has joined #bitcoin-core-dev
yanmaani has quit [Remote host closed the connection]
yanmaani has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 268 seconds]
SpellChecker has quit [Remote host closed the connection]
SpellChecker has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/89b219412495...bf3f05f41d27
<bitcoin-git> bitcoin/master 4786959 Ben Woosley: bench: Add a benchmark for descriptor expansion
<bitcoin-git> bitcoin/master bf3f05f MacroFake: Merge bitcoin/bitcoin#25785: bench: Add a benchmark for descriptor expansi...
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #25785: bench: Add a benchmark for descriptor expansion (master...2208-bench-desc-🍉) https://github.com/bitcoin/bitcoin/pull/25785
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 255 seconds]
Kaizen_Kintsugi_ has quit [Ping timeout: 268 seconds]
brunoerg has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
vasild has quit [Remote host closed the connection]
Kaizen_Kintsugi_ has quit [Ping timeout: 268 seconds]
vasild has joined #bitcoin-core-dev
_apex2_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
_apex2_ has quit [Ping timeout: 252 seconds]
Kaizen_Kintsugi_ has quit [Ping timeout: 268 seconds]
vasild has quit [Ping timeout: 268 seconds]
<bitcoin-git> [bitcoin] aureleoules opened pull request #25830: refactor: Replace m_params with chainman.GetParams() (master...2022-08-replace-m_params) https://github.com/bitcoin/bitcoin/pull/25830
sdaftuar has quit [Quit: WeeChat 2.8]
sdaftuar4 is now known as sdaftuar
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 255 seconds]
<bitcoin-git> [bitcoin] aureleoules opened pull request #25831: refactor: Remove trailing semicolon from LOCK2 macro (master...2022-08-macro-trailing-semicolon) https://github.com/bitcoin/bitcoin/pull/25831
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 268 seconds]
Kaizen_Kintsugi_ has quit [Ping timeout: 268 seconds]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
yanmaani has quit [Remote host closed the connection]
yanmaani has joined #bitcoin-core-dev
_andrewtoth_ has joined #bitcoin-core-dev
Guyver2 has joined #bitcoin-core-dev
vasild has joined #bitcoin-core-dev
Guyver2 has left #bitcoin-core-dev [#bitcoin-core-dev]
_andrewtoth_ has quit [Remote host closed the connection]
bitdex has quit [Remote host closed the connection]
evanlinjin has quit [Remote host closed the connection]
yanmaani has quit [Remote host closed the connection]
ghost43 has quit [Remote host closed the connection]
SpellChecker has quit [Write error: Broken pipe]
brunoerg has joined #bitcoin-core-dev
ghost43 has joined #bitcoin-core-dev
yanmaani has joined #bitcoin-core-dev
bitdex has joined #bitcoin-core-dev
evanlinjin has joined #bitcoin-core-dev
SpellChecker has joined #bitcoin-core-dev
brunoerg has quit [Client Quit]
brunoerg has joined #bitcoin-core-dev
vasild has quit [Remote host closed the connection]
vasild has joined #bitcoin-core-dev
bytes1440000 has joined #bitcoin-core-dev
<bytes1440000> I see a need for "janitorial maintainer" who understands the importance of privacy in core and privacy of contributors. Since #25560 is still open and intent of reviewers doesn't look they want to improve anything in the process, there is no "call for maintainers" thing.
<gribble> https://github.com/bitcoin/bitcoin/issues/25560 | [WIP] Add a MAINTAINERS.md file and desribe the role of maintainer more directly by JeremyRubin · Pull Request #25560 · bitcoin/bitcoin · GitHub
<bytes1440000> I would prefer someone who has already contributed to bitcoin privacy and privacy in general, not living in first world countries and not funded by brink.
bytes1440000 has left #bitcoin-core-dev [#bitcoin-core-dev]
<laanwj> ariard: thank you!
AaronvanW has quit [Remote host closed the connection]
Kaizen_Kintsugi_ has quit [Ping timeout: 268 seconds]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 244 seconds]
cmirror has quit [Remote host closed the connection]
cmirror has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
AaronvanW has joined #bitcoin-core-dev
<jnewbery> laanwj: Thank you for all of your hard work. Enjoy your retirement :)
AaronvanW has quit [Remote host closed the connection]
pablomartin has joined #bitcoin-core-dev
pablomartin has quit [Client Quit]
SpellChecker has quit [Remote host closed the connection]
SpellChecker has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 268 seconds]
<lightlike> +1, thank you laanwj!
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
vasild has quit [Remote host closed the connection]
vasild has joined #bitcoin-core-dev
vysn has quit [Ping timeout: 255 seconds]
Kaizen_Kintsugi_ has quit [Ping timeout: 252 seconds]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Kaizen_K_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 252 seconds]
Kaizen_K_ has quit [Ping timeout: 268 seconds]
dviola has left #bitcoin-core-dev [#bitcoin-core-dev]
_apex2_ has joined #bitcoin-core-dev
jarthur has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
___nick___ has joined #bitcoin-core-dev
dviola has joined #bitcoin-core-dev
_apex2_ has quit [Ping timeout: 268 seconds]
djsbtc[m] has quit [Quit: You have been kicked for being idle]
<vasild> #proposedmeetingtopic vasild for a new maintainer with a focus on P2P/networking
djsbtc[m] has joined #bitcoin-core-dev
<vasild> I volunteer, if there is a perceived need for that.
djsbtc[m] has left #bitcoin-core-dev [#bitcoin-core-dev]
Kaizen_Kintsugi_ has quit [Remote host closed the connection]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
cornfeedhobo has quit [Quit: ZNC - https://znc.in]
cornfeedhobo has joined #bitcoin-core-dev
___nick___ has quit [Quit: https://quassel-irc.org - Chat comfortably. Anywhere.]
sudoforge has joined #bitcoin-core-dev
___nick___ has joined #bitcoin-core-dev
___nick___ has quit [Client Quit]
Intdtti has quit [K-Lined]
___nick___ has joined #bitcoin-core-dev
Talkless has joined #bitcoin-core-dev
jamesob has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] 0xB10C opened pull request #25832: tracing: network connection tracepoints (master...2022-05-connection-tracepoints) https://github.com/bitcoin/bitcoin/pull/25832
_andrewtoth_ has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] satsie opened pull request #25833: doc: minor updates to guix README (master...update-guix-readme) https://github.com/bitcoin/bitcoin/pull/25833
Kaizen_Kintsugi_ has quit [Remote host closed the connection]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Read error: Connection reset by peer]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 268 seconds]
Kaizen_K_ has joined #bitcoin-core-dev
SpellChecker has quit [Remote host closed the connection]
brunoerg has quit [Remote host closed the connection]
brunoerg has joined #bitcoin-core-dev
SpellChecker has joined #bitcoin-core-dev
Kaizen_K_ has quit [Ping timeout: 252 seconds]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 255 seconds]
charon has joined #bitcoin-core-dev
_apex2_ has joined #bitcoin-core-dev
charon has quit [Client Quit]
brunoerg has joined #bitcoin-core-dev
_apex2_ has quit [Ping timeout: 252 seconds]
<laanwj> jnewbery: lightlike: thank you!!! hehe, retirement
brunoerg has quit [Ping timeout: 268 seconds]
brunoerg has joined #bitcoin-core-dev
sipsorcery has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 264 seconds]
bytes1440000 has joined #bitcoin-core-dev
<bytes1440000> vasild: thanks for volunteering and i think you are the best candidate for p2p/privacy. I wish we had one for mempool/privacy and wallet/privacy as well.
andrewtoth_ has joined #bitcoin-core-dev
brunoerg has joined #bitcoin-core-dev
<bytes1440000> I had one question after I looked at #25355 and some conversation with u/alreadybrunt on i2p subreddit. Java router and I am assuming C++ router as well has some limits.
<gribble> https://github.com/bitcoin/bitcoin/issues/25355 | I2P: add support for transient addresses for outbound connections by vasild · Pull Request #25355 · bitcoin/bitcoin · GitHub
<bytes1440000> What if an attacker tries to establish outbound connections with your node from different i2p addresses out of max sessions. Didn't get time to test it.
_andrewtoth_ has quit [Remote host closed the connection]
bytes1440000 has left #bitcoin-core-dev [#bitcoin-core-dev]
Kaizen_Kintsugi_ has quit [Remote host closed the connection]
brunoerg has quit [Ping timeout: 255 seconds]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
andrewtoth_ has quit [Remote host closed the connection]
andrewtoth_ has joined #bitcoin-core-dev
ynnitan has joined #bitcoin-core-dev
<vasild> bytes1440000: an attacker can easily do that even without #25355
<gribble> https://github.com/bitcoin/bitcoin/issues/25355 | I2P: add support for transient addresses for outbound connections by vasild · Pull Request #25355 · bitcoin/bitcoin · GitHub
<vasild> with tor or i2p
brunoerg has joined #bitcoin-core-dev
<vasild> I mean - open too many connections to a "victim" which cannot figure out that the many connections originate from one place
* vasild zZzZ
brunoerg has quit [Ping timeout: 268 seconds]
sipsorcery has quit [Ping timeout: 268 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 268 seconds]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Ping timeout: 244 seconds]
<achow101> #startmeeting
<core-meetingbot> Meeting started Fri Aug 12 19:01:21 2022 UTC. The chair is achow101. Information about MeetBot at https://bitcoin.jonasschnelli.ch/ircmeetings.
<core-meetingbot> Available commands: action commands idea info link nick
<achow101> #bitcoin-core-dev Wallet Meeting: achow101 _aj_ amiti ariard BlueMatt cfields Chris_Stewart_5 darosior digi_james dongcarl elichai2 emilengler fanquake fjahr furszy gleb glozow gmaxwell gwillen hebasto instagibbs jamesob jarolrod jb55 jeremyrubin jl2012 jnewbery jonasschnelli jonatack josibake jtimon kallewoof kanzure kvaciral laanwj larryruane lightlike luke-jr maaku marcofalke meshcollider michagogo moneyball morcos Murch nehan NicolasDorier
<achow101> paveljanik petertodd phantomcircuit promag provoostenator ryanofsky sdaftuar S3RK sipa vasild
<furszy> hi
<achow101> There are no pre-proposed wallet meeting topics. Does anyone have any last minute topics to discuss?
<S3RK_> hi
<achow101> From yesterday's general meeting, it seems like we've decided to try to get #19602 for 24.0. The feature freeze is on Monday
<gribble> https://github.com/bitcoin/bitcoin/issues/19602 | wallet: Migrate legacy wallets to descriptor wallets by achow101 · Pull Request #19602 · bitcoin/bitcoin · GitHub
<S3RK_> I started reviewing this a long time ago, but lost all the context now. Need to start from scratch
<S3RK_> achow101 do you think we can get it reviewed by Mon?
<achow101> we may allow the feature freeze to slip a bit to get it in
<achow101> but probably not more than a week
<S3RK_> I'd imagine testing a big part of shipping this feature. What's the status with testing?
<achow101> I think reviewers have been testing manually
<achow101> there is also a functional test
<achow101> that tests several different scenarios
brunoerg has joined #bitcoin-core-dev
<achow101> the rpc is also marked as experimental
<S3RK_> I agree it'd be great to have it 24.0 can only blame myself for not paying more attention to it
<S3RK_> I can spend some time on it from Mon. Probably can do either review or some manual testing. What do you think would be more helpful?
<furszy> should be more a "what might be missing" work than actual testing
<achow101> I think review would be more helpful
<achow101> the main thing is that legacy wallets can have some weird script scenarios that I might have missed
SpellChecker_ has joined #bitcoin-core-dev
<achow101> and manual testing probably won't get to those
<S3RK_> got it
SpellChecker has quit [Ping timeout: 268 seconds]
brunoerg has quit [Ping timeout: 255 seconds]
<achow101> is there anything else to discuss?
<S3RK_> one thing
<S3RK_> I thought maybe it could be interesting to do coin selection simulations for each release
<S3RK_> we do them for some PR which we think could have an effect, but sometimes the consequences are hard to predict
<S3RK_> do you think it'll be beneficial to just do checkpoints at release time? or maybe at some other interval?
<achow101> that could be interesting
<achow101> it would probably make sense to do one on the first rc of each release?
<S3RK_> yep. that sounds reasonable
<S3RK_> is there a checklist or something where we can add a bullet point?
<furszy> goal is bench speed/performance or the selection results?
<instagibbs> or feature freeze, in case you want to get a head start on a regression fix :P
<instagibbs> unless it's just informational
<S3RK_> catching a regression is definitely a part of it
<achow101> there's a release checklist in doc/release-process.md
<achow101> but this seems more like something that a wallet contributor remembers to do around release time
<achow101> it would be nice if we had an automated way to do simulations
<S3RK_> furszy: not speed, but rather effectiveness in terms of fees/utxo set/changeless tx
<S3RK_> for starters: a step in release process could be just to poke wallet maintainer :D
<S3RK_> ofc it'll be great to automate things. What are the challenges today? I saw achow101's optimizations got merged
<achow101> I think it's just a matter of doing it
<achow101> I know that josibake has a project that kind of automates simulations
<S3RK_> yep, I used it on a VM. Do you imagine just running simulation as a CI for each PR?
<achow101> I think that would be useful, but would probably have to be separate from our actual CI
<achow101> since they can take a really long time
<S3RK_> wouldn't we hit some limits in terms of CPU/jobs or run time?
brunoerg has joined #bitcoin-core-dev
<S3RK_> sound heavy to run sims for all PRs
<achow101> yeah
<furszy> and there shouldn't be many PRs changing behavior there
<achow101> maybe something like the "guix build requested" label
<S3RK_> that's interesting
<instagibbs> yeah
<achow101> so we could just run it on the prs that would effect coin selection
<Murch> hi
<Murch> Yeah, that sounds interesting
<Murch> So far I've been running simulations manually, it would definitely be nice to get it automatically. How could we control how often it gets run though?
<Murch> Let's say if you spot a typo and forcepush to fix briefly after pushing, it shouldn't do the whole simulation twice.
brunoerg has quit [Ping timeout: 268 seconds]
<Murch> Perhaps if we leave a few hours until kicking off the simulation and only run it for the latest version?
brunoerg has joined #bitcoin-core-dev
<S3RK_> I like the idea with using a label to trigger the CI
<achow101> if we do it the same way that drahtbot guix builds work, then it's basically just triggered manually whenever the label is added
<achow101> the bot removes the label when it's done
<Murch> Who can add labels?
<achow101> anyone who has the permission
<S3RK_> hm.. is it a separate permission? I think I can't modify lables even for my PRs
brunoerg has quit [Ping timeout: 264 seconds]
Kaizen_Kintsugi_ has quit [Remote host closed the connection]
<achow101> it's a separate permission, but I think we give it out pretty freely
<S3RK_> it's good that it's separate
SpellChecker has joined #bitcoin-core-dev
vysn has joined #bitcoin-core-dev
<S3RK_> If nobody have more topics, I have one more thing about #25647
<gribble> https://github.com/bitcoin/bitcoin/issues/25647 | wallet: return change from SelectionResult by S3RK · Pull Request #25647 · bitcoin/bitcoin · GitHub
<achow101> go ahead
<S3RK_> Murch pointed out a small issue with that PR
<S3RK_> at high fee rates LOWER_CHANGE won't cover change_fee
<S3RK_> I think easy way to fix that is to make LOWER_CHANGE dynamic = max(50000sat, change_fee)
SpellChecker_ has quit [Remote host closed the connection]
<S3RK_> any reasons why it's bad or won't work?
<S3RK_> sorry it's CHANGE_LOWER
<Murch> That sounds fine to me
<Murch> I don't think it'll be hit until well above 1000 s/vB anyway
<Murch> I'm not sure whether a change target that is exactly the fee makes sense, though
<Murch> maybe it should rather be fee+dust
<S3RK_> it's not change traget but rather CHANGE_LOWER
<S3RK_> then change target is calucalated using GenerateChangeTarget
<Murch> Ah right
<achow101> you could just change it to have that parameter include the fee?
<achow101> so m_min_change_target + m_change_fee
<Murch> Well, but it could still mean that if we roll a low value and match very closely, we would fall into the dust territory, right?
<S3RK_> I don't think it's possible. Given that dustRealy is 3sat/vb and we need to bump CHANGE_LOWER at fees about 1000sat/vb
<achow101> or rather GenerateChangeTarget should incorporate the change fee
<S3RK_> I like jsut adding m_min_change_target + m_change_fee
<S3RK_> that's also very clean and simple
<Murch> +1
<S3RK_> I'll add a commit to ensure change_target is always higher than change_fee using achow101 suggestion
<S3RK_> thanks
<achow101> cool
<achow101> anything else to discuss?
<S3RK_> not for my side
<Murch> I've spent a lot of time thinking about #24362 recently, I think that we could probably move it towards merging or closing
<gribble> https://github.com/bitcoin/bitcoin/issues/24362 | wallet: Do not match legacy addresses for change type by achow101 · Pull Request #24362 · bitcoin/bitcoin · GitHub
<Murch> I'm still on the fence
<Murch> right now fees are so low, I'd say close it and just match the legacy outputs, but in the long-term I do think that it would potentially create unexpected costs for users
<Murch> that is, not to merge it
<S3RK_> if you don't want to create legacy change outputs you shouldn't have a descriptor
<S3RK_> it's possible to deactivate a descriptor
<S3RK_> unless we're talking legacy wallets :D
<achow101> well legacy wallets are still a thing
<Murch> S3RK_: I think we have to operate under the assumption that most users will never change a config option or otherwise customize their wallet
<S3RK_> I can see why asking users to understand decsriptors could be too much
<Murch> And it might feel "more surprising" to a user that they suddenly sent themselves a legacy change output than the opposite
<S3RK_> but do you think it's too much for them to be aware what types their wallet supports?
_apex2_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
<Murch> Yeah
<S3RK_> it feels to me adding more special cases like this make wallet behavior opaque and unpredictable in a way
<Murch> I think that most users would probably be aware w
<Murch> of the type of outputs they use for receivling only
<S3RK_> this is already good enough
<Murch> S3RK_: "Special case" as in not matching the recipient with the change just for legacy, or "special case" matching recipient outputs in the first place?
<S3RK_> the first
<achow101> I think right now it's not a problem, and perhaps we could revisit this if users start complaining or we see a noticeable increase in legacy usage?
real_or_random has quit [Quit: Connection closed for inactivity]
<achow101> I would hope this becomes moot and legacy usage dies out
<Murch> achow101: Alright, then let's close it for now?
<achow101> sure
<Murch> Yeah, hopefully before feerates go up again, or at least when feerates start going up again…
<achow101> further proposals like josie[m]'s idea of using a different address type for change when the inputs match the recipients would also mitigate this somewhat
SpellChecker has quit [Remote host closed the connection]
andrewtoth_ has quit [Remote host closed the connection]
vasild has quit [Remote host closed the connection]
_apex2_ has quit [Ping timeout: 268 seconds]
vasild has joined #bitcoin-core-dev
SpellChecker has joined #bitcoin-core-dev
<achow101> anything else to talk about in the last minute of the meeting?
<Murch> I like the idea (but then, it was mine :p), but I think it should only happen when we would use the given input set already for cost reasons until we have a better way to score privacy benefits in a numeric fashion
andrewtoth_ has joined #bitcoin-core-dev
<achow101> #endmeeting
<core-meetingbot> topic: Bitcoin Core development discussion and commit log | Feel free to watch, but please take commentary and usage questions to #bitcoin | Channel logs: http://www.erisian.com.au/bitcoin-core-dev/, http://gnusha.org/bitcoin-core-dev/ | Meeting topics http://gnusha.org/bitcoin-core-dev/proposedmeetingtopics.txt / http://gnusha.org/bitcoin-core-dev/proposedwalletmeetingtopics.txt
<core-meetingbot> Meeting ended Fri Aug 12 20:00:01 2022 UTC.
<S3RK_> thanks achow101
<Murch> Yeah, thanks :)
<S3RK_> Murch, how do folks control receiving types today? Is it possible to setup default in GUI?
brunoerg has joined #bitcoin-core-dev
<Murch> I think it should be possible in the receive tab to pick the output type, yes
<Murch> And otherwise the default is currently bech32, afair
<S3RK_> then you can contorl change type there as well
<furszy> yep, there is a spin box there
<S3RK_> otherwise if you have an option to setup default, it would be a good place to also setup your defaults for change output types
brunoerg_ has joined #bitcoin-core-dev
<S3RK_> I don't really see how recipient types and change types that much different
___nick___ has quit [Ping timeout: 268 seconds]
brunoerg has quit [Ping timeout: 268 seconds]
S3RK_ is now known as S3RK
vasild has quit [Remote host closed the connection]
andrewtoth_ has quit [Remote host closed the connection]
yanmaani has quit [Remote host closed the connection]
bitdex has quit [Read error: Connection reset by peer]
vasild has joined #bitcoin-core-dev
yanmaani has joined #bitcoin-core-dev
bitdex has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Remote host closed the connection]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
andrewtoth has joined #bitcoin-core-dev
SpellChecker has quit [Remote host closed the connection]
bitdex has quit [Remote host closed the connection]
bitdex has joined #bitcoin-core-dev
SpellChecker has joined #bitcoin-core-dev
dougefish has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] achow101 closed pull request #24362: wallet: Do not match legacy addresses for change type (master...rm-legacy-change) https://github.com/bitcoin/bitcoin/pull/24362
vasild has quit [Ping timeout: 268 seconds]
vasild has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Remote host closed the connection]
brunoerg_ has quit [Remote host closed the connection]
brunoerg has joined #bitcoin-core-dev
brunoerg has quit [Client Quit]
<bitcoin-git> [bitcoin] russeree closed pull request #24935: mempool: Add usdt event tracepoints and eBPF logging tool. (master...master) https://github.com/bitcoin/bitcoin/pull/24935
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
vysn has quit [Ping timeout: 268 seconds]
_apex2_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 268 seconds]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Kaizen_K_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 268 seconds]
Kaizen_K_ has quit [Remote host closed the connection]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Kaizen_K_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 268 seconds]
szkl has quit [Quit: Connection closed for inactivity]
Kaizen_K_ has quit [Ping timeout: 268 seconds]
_apex2_ has quit [Ping timeout: 268 seconds]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
ghost43 has quit [Remote host closed the connection]
andrewtoth has quit [Remote host closed the connection]
ghost43 has joined #bitcoin-core-dev
ghost43 has quit [Remote host closed the connection]
ghost43 has joined #bitcoin-core-dev
andrewtoth has joined #bitcoin-core-dev
vasild has quit [Remote host closed the connection]
yanmaani has quit [Remote host closed the connection]
vasild has joined #bitcoin-core-dev
yanmaani has joined #bitcoin-core-dev
jonatack has quit [Quit: Connection closed]
andrewtoth_ has joined #bitcoin-core-dev
andrewtoth has quit [Ping timeout: 268 seconds]
andrewtoth_ has quit [Remote host closed the connection]
andrewtoth_ has joined #bitcoin-core-dev
furszy has quit [Quit: ZNC - https://znc.in]
jarthur_ has joined #bitcoin-core-dev
furszy has joined #bitcoin-core-dev
jarthur has quit [Ping timeout: 255 seconds]
ynnitan has left #bitcoin-core-dev [#bitcoin-core-dev]
Kaizen_Kintsugi_ has quit [Read error: Connection reset by peer]
jarthur has joined #bitcoin-core-dev
jarthur_ has quit [Ping timeout: 255 seconds]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev
Kaizen_Kintsugi_ has quit [Ping timeout: 252 seconds]
Kaizen_Kintsugi_ has joined #bitcoin-core-dev