05:17:38 I think it's mostly standard cache times 05:17:55 in other words, we aren't using cache hints to have it not be cached (not that cache hints are always adhered to either) 05:53:51 downloads stuff is gong up via LFS now, all seems to be working well 05:54:14 should keep the actual repo size pretty small (if you're not pulling down LFS files) 05:54:32 we should consider switching to LFS for the big files in the site repo (eg. movies) 12:04:23 https://github.com/monero-project/monero-downloads 12:04:24 all done 12:20:14 nice 13:20:14 Whitepaper reviews added to `monero-site`: https://github.com/monero-project/monero-site/pull/1111 13:20:18 CI still in progress 13:22:19 sarang: i already opened #1109 13:23:03 So you did :) 13:23:33 yeah did it yesterday. I think i posted it here too :P 13:23:40 I'd recommend adding to the `pubs` directory for consistency with other PDFs 13:23:46 (1111 does this) 13:23:57 ErCiccione: can't expect me to read _everything_ now! 13:24:09 Lol, you are right 13:24:34 sure, i didn't put them there beecause they are not "official" pubblication, but if it's fine for you it's fine for me 13:24:34 Hooray, sarang wins! 13:24:46 What would be official? 13:25:21 The "unpublished" `ge_fromfe_frombytes_vartime` document lives there already anyway 13:25:34 (not sure what "unpublished" is supposed to mean there...) 13:26:00 I meant "published" pubblications. Those PDFs are just comments of pubblications. Don't know if i explained myself properly here 13:26:22 nevermind. I understood your point :P 13:26:30 oh ok 13:26:44 I'm probably rumbling nonsense right now 13:26:49 I see your point 13:27:11 There are 38 degrees (celsius) here, i'll blame that 13:28:02 jinkies 13:28:19 I don't personally care where the PDFs live, directory-wise 13:28:56 Merge whichever makes the most sense! 13:29:05 And maybe I'll check for existing PRs before writing new ones =p 13:30:31 I'm super fast 13:31:07 I moved them to pubs: https://github.com/monero-project/monero-site/pull/1109 13:31:24 a review would be appreciated 13:32:46 wilco 13:33:07 Will it generate a new preview from the force-push? 13:33:37 I'll check the code, but being able to directly test links in preview would be nice 13:34:08 Silly question: any reason the `baseurl` is used for the `href` tags for the other papers, but not these? 13:34:12 yep 13:34:26 every force push triggers a new netlify preview 13:34:29 k 13:35:44 But yeah, the link format is different on the whitepapers apparently 13:35:47 That's the absolute url i used a relative url. No difference 13:36:18 OK, no performance diffs? 13:36:35 We could keep everything the same for consistency 13:36:50 Yeah, might as well use one format 13:37:01 Oh, the netlify bot just updates its original comment; neat 13:37:03 didn't see that :D 13:37:10 I'll wait until you update and then test 13:37:14 Or just use 1111 instead :D 13:38:40 pushed 13:44:24 approved 13:45:18 what's 1111 i missed that reference? 13:45:51 PR 1111 13:46:00 did the same thing as PR 1109 13:46:09 I just closed 1111 13:47:06 O yeah. Sorry man, today i am a bit off 13:47:13 No worries! 13:47:18 Looks like 1109 does the trick 13:47:30 and preview links work fine 13:47:50 Perfect 13:48:10 .merge + #1109 13:49:21 .merges 13:49:21 -xmr-pr- #1027 #1042 #1051 #1072 #1078 #1084 #1087 #1091 #1093 #1094 #1104 #1106 #1110 #937 #982 #993 13:49:53 -.- 13:50:02 alright, i'm gonna go get a drink or something 13:50:19 .merge+ #1109 13:50:19 Added 13:50:23 .merges 13:50:23 -xmr-pr- #1027 #1042 #1051 #1072 #1078 #1084 #1087 #1091 #1093 #1094 #1104 #1106 #1109 #1110 #937 #982 #993 13:50:27 winner 13:50:39 yaaay 13:50:44 Too bad the bot doesn't list in numerical order 13:50:59 Looks like it's lexicographic order 13:51:14 Who runs the bot? 13:52:43 dsc 13:53:34 dsc_: possible to update the bot to list merges numerically? 13:53:54 ErCiccione: are you waiting on review for other PRs right now that need to get in the queue? 13:54:00 ^ thanks moneromooo 13:54:02 :D 13:54:06 And add PRs without using a # sign, it requires it for some reason. Or would it otherwise cause false positives ? 13:54:26 Failing that, I'll stop forgetting with time. 13:54:34 Usually it did list them in numerical order 13:54:40 sarang: checking 13:54:48 ErCiccione: was that when < 1000 ? 13:55:14 lol, looks like you're right, it's alpha. 13:55:21 The transition from 9xx to 1yyyy would ruin things 13:55:27 s/1yyyy/1yyy 13:56:09 I think so. the bot already freaked out in past but now seems to be that. 13:58:48 I wonder if its code is public 14:00:02 sarang: #1046 #1076 #1083 #1087 #1105 #1108 need a review or a final review 14:00:10 I wonder if its code is public -> it's not AFAIK 14:00:24 k 14:13:21 Reviewed 1046 14:14:46 ErCiccione: is there a way to force a new preview build? 14:14:53 1076's preview has expired 14:16:06 Those are the old previews made with the free accounts that now got deleted. I can force push to trigger the new barolo-time previews 14:17:04 sarang: numerically? hmm.. Going to have to devise an algorithm... it will be hard, but I think I can do it 14:17:39 Done, it's building 14:18:49 dsc_: really? 14:18:55 How is it done now? 14:19:07 Thanks ErCiccione 14:19:18 Too bad the bot doesn't update its comments after previews expire 14:19:25 oof sarang, it's complicated. But I will persevere 14:19:35 * sarang can't tell if serious :/ 14:19:54 That's because they are linked to different accounts. Previews don't expire afaik 14:20:06 hehe sorry it's /s next update will have: 14:20:08 - numerical listing 14:20:09 - add without # 14:20:21 Thanks :) 14:21:36 .merges 14:21:39 -xmr-pr- 937 982 993 1027 1042 1051 1072 1078 1084 1087 1091 1093 1094 1104 1106 1109 1110 14:21:46 .merge- 937 14:21:46 Removed 14:21:49 .merge+ 937 14:21:49 Added 14:22:59 Oh, because the octothorpes look like channel links? 14:23:15 dsc_: that was fast =p 14:24:08 Another satisfied customer! 14:25:01 dsc_: will it work anyway if i use #? i will 100% forget about the change 14:25:24 .merge+ #938 14:25:24 Added 14:25:26 Sorry. I meant for it to be optional only :) 14:25:30 .merge- #938 14:25:30 Removed 14:25:34 Nice. 14:25:34 ErCiccione: seems # still works :) 14:25:40 tested 1076 14:25:50 nice. Thanks 14:26:00 in fact; 14:26:06 .merge+ #1046 14:26:06 Added 14:26:17 .merge+ blablabla111blablabla 14:26:18 Added 14:26:23 .merge- blablabla111blablabla 14:26:23 Removed 14:26:30 `re.findall(r'[\d]+', trigger.group(2))` 14:26:31 :P 14:27:12 .merge+ not123 14:27:12 Added 14:27:17 .merge 14:27:19 .merge? 14:27:24 .merges 14:27:24 -xmr-pr- 123 937 982 993 1027 1042 1046 1051 1072 1078 1084 1087 1091 1093 1094 1104 1106 1109 1110 14:27:30 .merge- not123 14:27:30 Removed 14:27:50 .merge+ 1e3 14:27:50 ... 14:28:22 heh 14:28:24 .merge+ 1e12 14:28:27 .merges 14:28:28 -xmr-pr- 1 3 12 937 982 993 1027 1042 1046 1051 1072 1078 1084 1087 1091 1093 1094 1104 1106 1109 1110 14:28:31 ha :) 14:28:40 .merge- 1 3 12 14:28:41 Removed 14:28:51 bad mooo stop breaking my bot 14:32:49 One last time, sir ? 14:33:11 okiii 14:33:13 .merge+ 1 3 14:33:13 Added 14:33:17 .merge? 14:33:19 .merges 14:33:19 -xmr-pr- 1 3 937 982 993 1027 1042 1046 1051 1072 1078 1084 1087 1091 1093 1094 1104 1106 1109 1110 14:33:25 .merge- 1 3 14:33:25 Removed 14:38:56 .merge+ 1076 14:38:56 Added 16:14:44 CLSAG audit PR: https://github.com/monero-project/monero-site/pull/1112 16:14:48 Please review ASAP 16:15:47 on it 16:17:13 :D 16:17:16 Waiting for CI preview now 16:28:40 ErCiccione: making those changes now 16:29:42 Ok. i'm reviewing the rest 16:31:35 sgp wants me to use the svg instead 16:31:55 OK to just use that in the `img` tag? I don't know about browser support 16:32:34 Are you referring to the svg sgp_ is talking about or the liquid tag? 16:32:58 Let me test it with that svg, but shouldn't be an issue to use that instead 16:33:01 the one he's talking about 16:33:04 I'm adding it presently 16:33:11 and the change you requested 16:33:13 will push momentarily 16:34:42 browser support shouln't be a problem. 16:34:42 +1 for svg, the old image was blurry on my phone 16:35:03 SVG is a nice to have 16:35:23 almost done with new push 16:35:25 one sec 16:35:43 the svg looks good 16:37:05 also only 9 kB 16:37:11 compared to something like 100 16:39:11 force-pushed just now 16:39:28 Let me know about the logo sizing 16:40:04 waiting on CI preview 16:41:21 The logo is good 16:42:19 Did the preview update with the force push? 16:43:10 there it goes 16:43:53 Looks good on my browser too 16:43:55 Lemme check mobile 16:44:20 looks good on mobile too 16:44:24 fast loading too :D 16:44:28 so fast 16:44:50 Can someone give a thumbs-up to the updated commit? 16:45:26 Forcing the width wasn't necessary, but doesn't hurt either 16:45:56 I didn't know how it would be handled otherwise with SVG scaling 16:46:16 Expanding to fill that page element seems like it'd be awkward 16:46:43 The width in the svg is 412px 16:46:50 it's good tho, no worries 16:47:10 ah ok 16:47:57 updated anyway =p 16:48:15 :) 16:48:17 I know it'd bother me otherwise... 16:48:31 What'd be the ETA on build/deploy? 16:48:34 OSTIF's post is up 16:48:47 I happen to like my post better, but I'm obviously biased :D 16:49:58 I like yours better too :p 16:50:30 Just one minor thing sarang: could make bold "tests indicate that users will see a 20% improvement in signature verification, and at least a 10% overall improvement for typical transactions!"? 16:50:50 can do 16:50:50 would be good to have some @ links to the moneropedia for words like "transactions", but that's not really important 16:50:58 I'll do the bold right now 16:51:02 Other links could be done later? 16:51:45 Sure, no big deal. 16:52:01 done 16:56:43 just left my approval 16:58:03 sarang: About the ETA: If luigi1111 is around we can merge and fluffypony, binaryfate or pigeons could deploy 16:58:24 why are there so many merges 16:58:25 .merges 16:58:25 -xmr-pr- 937 982 993 1027 1042 1046 1051 1072 1076 1078 1084 1087 1091 1093 1094 1104 1106 1109 1110 16:58:29 can you all calm down 16:59:07 .merge+ #1112 16:59:08 Added 16:59:10 get one more! 17:04:23 LOL 17:04:24 .merge+ all the things 17:04:46 .merge+ all the th1ngs 17:04:46 Added 17:04:49 Oops 17:04:51 LOL 17:04:57 .merge- 1 17:04:58 Removed 17:06:11 wut? 17:06:17 .merge+ all the things 17:06:22 ah ok 17:07:05 The blog post is the biggest thing, since OSTIF have already posted theirs 17:09:46 yeah luigi1111 please merge 1112, the rest can wait 17:10:25 done 17:11:09 Thanks! Let's get that deployed! 17:11:30 fluffypony ^ 17:14:14 Who needs sports when you can watch this 17:22:10 Fluffy has pinged others for the build 18:02:34 Looks like no response yet :/ 18:02:41 binaryFate pigeons 18:03:30 spoke to pigeons - we don't want to break binaryFate's stuff, we'll find a uniform way of doing this going forward so this doesn't happen 18:04:51 I see 18:05:20 Like a checklist :) 18:18:38 Anyone know if binaryFate will be online today? 19:42:24 Probably better to PM him. Haven't seen hin online much lately.