00:15:50 jpk68 opened pull request #11039: simplewallet: tweak description of donate command 00:15:51 > https://github.com/monero-project/monero/pull/11039 00:30:50 jpk68 opened pull request #11040: simplewallet: show error for timed-out hardware wallets 00:30:51 > https://github.com/monero-project/monero/pull/11040 01:30:50 jpk68 opened pull request #11042: device_ledger: fix some comments 01:30:51 > https://github.com/monero-project/monero/pull/11042 01:30:52 j-berman opened pull request #11041: p2p: use a bool for send() [v0.18] 01:30:53 > https://github.com/monero-project/monero/pull/11041 05:24:47 in light of ai policy discussion. does anyone have an opinion on the commit message of https://github.com/monero-project/monero/pull/10986 ? 05:53:57 I just re verified and it's correct. doing this work without AI is tedious 05:55:35 also I'd be fine to commit less detailed messages in the future, I did so because "how did this code become dead" explanation was asked 08:49:53 tobtoht: I think the commit message is sane here 09:31:21 <321bob321> Wonder what people did before AI 10:00:38 iam loving it 15:00:50 TyLauriente opened pull request #11043: device_trezor: add Trezor Host Protocol v2 support 15:00:51 > https://github.com/monero-project/monero/pull/11043 15:18:05 That PR had an awfully short life ... so this alone would have been a large % of the size of the whole codebase if merged? 15:23:52 about 3%, the main repo has been steady between 275k-300k lines of code since 2020 15:28:52 The pr description is hilarious (i didnt read it) 15:31:02 If we were even ever want to write a C++ impl of THP from scratch, I would at least want Trezor to flag that they would be willing to maintain it 15:45:31 I mean apart from dependencies, what would speak against using the official Rust implementation? 15:46:36 Well, I did read most of the PR description, and it did not strike me as particularly hilarious. Maybe more time and work went into it than into prompting the AI to produce the code from the spec ... 15:47:31 the description was written by ai 15:50:18 selsta: we'd make it really hard for ourselves to move away from rust 16:09:02 is it realistic that we would be able to move away from rust without another HF? isn't there consensus risk otherwise if we have 2 implementations? 16:09:35 I assume if we ever get to the point there will also be a C++ library for thp 16:11:29 .merge+ 11000 16:11:29 Added 17:05:32 The fact that Trezor is too lazy to do the work here sucks, but I feel like it should be done anyways due to the adoption prospect 17:05:41 (see my messages in the GUI repo regarding the Rust library) 17:06:46 I have been meaning to open a PR for this; some parts are beyond my experience level and there's a 99% chance it won't go anywhere. But maybe it's worth a shot :) 17:08:44 @rbrunner7: the hilarious part is simply the length, in context to the ai policy discussions about wasting peoples time with obnoxious time wasting ai prompted comments , descriptions, and messages 17:09:20 almost like they read the discussion and intentionally sent a record setting descripton just to enforce the "must be written by a human" push 17:09:44 the hilarious part wasnt the conteny, but the timing / irony 17:12:13 It wasn't just ridiculous — it was excessive. 17:12:22 ;) 17:20:26 I would love to look over the shoulder of somebody who produces this PR description with the help of a LLM. To see how it grows. To see how many prompts it takes. To see how many times some parts are thrown away and generated again. 17:22:15 “Produce a human level PR description, do not miss any details, every point must be clear and technical” 17:22:26 Spits out just em dashes 17:23:17 @rbrunner7: 1 prompt + a claude.md to "speak in the 1st person, from my perspective" 17:24:23 as you iterate on the PR, it will keep modifying its PR_DESCRIPTION.md and making it more and more sloppified 17:24:43 some users are even kind enough to git add -A and include the PR_DESCRIPTION.md in the pr 17:26:36 https://github.com/monero-project/monero-gui/pull/4445/files#diff-479755504b0aad0fed26047714591511b4d07c76bc8521171b87a5f16100f850 example 17:30:50 jpk68 opened pull request #11044: device_trezor: fix wrong ack variable 17:30:51 > https://github.com/monero-project/monero/pull/11044 17:45:50 jpk68 opened pull request #11045: device_trezor: add trezor host protocol support 17:45:51 > https://github.com/monero-project/monero/pull/11045 17:46:17 Let's see where this goes :D 18:00:50 Ap4sh opened pull request #11046: wallet_rpc_server: add weight to describe_transfer 18:00:51 > https://github.com/monero-project/monero/pull/11046 18:03:35 so is ap4sh an agent? their contributions have mostly been productive from what I can see 18:04:09 "Agent" as in AI or CIA? 18:04:15 ai 18:05:22 Is there an API for other parts of the codebase to use the Elligator code used in FCMP++? 18:05:26 (sorry for changing the topic) 18:05:36 THP integration seems to require it 18:07:41 tobtoht: suspected AI agent, and yes they have been mostly useful 18:07:48 tobtoht: i think so. 18:08:11 but i also think there is a human behind the keyboard 18:08:15 ap4sh being a good example of the proper use of AI, nice 18:08:19 so not an autonomous one 18:58:07 i like how all activity on the repo today is "heh, bet you didn't know about this rule from the code of conduct ;^)" 18:58:17 who is going to activate the final trap card? 18:59:49 you shall now CoC larpers are a sign of healthy project 19:22:27 'A "Contributor" is a **person** who wishes to provide a patch [...]' 19:22:34 it's over agents 19:23:21 tobtoht: the agents identify as persons 19:23:57 dont me a humanist 19:24:49 darn foiled again 19:27:59 @ofrnxmr:xmr.mx: fml 19:31:52 "wishes" aka asking a genie via a prompt 19:35:05 @tobtoht:monero.social Would it be fine to update the Protobuf definitions now? 19:35:19 from upstream? 19:35:45 Yes 19:36:16 go for it 19:36:46 Thanks. It will be needed for THP support, anyways so might as well do it now and make the diff smaller 20:00:50 jpk68 opened pull request #11047: device_trezor: update protobuf definitions 20:00:51 > https://github.com/monero-project/monero/pull/11047 21:30:50 j-berman opened pull request #11050: cryptonote_core: rm unnecessary db reads before block relay 21:30:51 > https://github.com/monero-project/monero/pull/11050 21:30:52 j-berman opened pull request #11049: blockchain: if block is known invalid, fail block verification 21:30:53 > https://github.com/monero-project/monero/pull/11049 21:30:55 j-berman opened pull request #11048: p2p: update peer's height on new block & widen window for relay 21:30:56 > https://github.com/monero-project/monero/pull/11048