01:14:14 @tevador I hate to rush, but is there a timeline when I can expect the mx25519 PRs to be reviewed and merged by? 01:26:32 they aren't connected on IRC side yet, so maybe they'd look at logs 09:03:04 @kayabanerve:monero.social: gentle ping for https://github.com/monero-oxide/monero-oxide/pull/187 and https://github.com/monero-oxide/monero-oxide/pull/188 09:10:24 Hey all, I'm going to try to continuously maintain a work breakdown structure for FCMP++/Carrot here: https://github.com/jeffro256/fcmp-carrot-plan 09:11:05 Right now, it's still a pretty rough sketch, but hopefully it gives some insight for some people 09:19:53 can you export that into gh pages or something so it can be loaded up to the latest version on a browser without having to clone? 09:20:23 can't remember how you do that nowadays on github but it should be able to at least expose that html 11:29:09 @sneedlewoods_xmr:matrix.org: I had a look at the new code built around api_error_2_rpc_error. IMHO this is much better, and came out easier as expected. 11:34:23 Currently I only have a small "uneasiness" about the new method: the assumption that the error codes of the Wallet API and that of the RPC wallet server are identical. Maybe it's ok to use that and simply assign, maybe using a switch statement to assign the 15 or so Wallet API error codes to their RPC equivalents would be a thing. 11:35:36 In any case, this would produce references of the identifiers in the code, and if you wanted to know for one of them where in the code they are used in any way, you could locate this. 11:36:12 And would be less brittle should the codes diverge nevertheless for some reason in the future. 11:36:25 But, you PR, your call, ok for me either way :) 11:41:39 Thanks for the feedback, will look into it. Also there are 2 more comments you made I haven't addressed,yet. Will try to make 2 pushes, one for rebasing and fixing conflicts the other one with review changes. 11:43:01 Splendid! 13:28:22 Sorry, there were so many conflicts (some still are there, but I know how to fix them and others maybe revealed later), so I need more time to first properly update "updated Wallet API" #9464 then "remove cached password from Wallet API" #10232 and finally the cli #10233 and rpc #10819 21:55:03 @jeffro256 I have submitted a batch of reviews for mx25519