03:33:21 Experimental (bcrawl) branch on underhound.eu updated to: 0.23-a0-5208-geafff8c3b6 03:58:14 <09hjklyubn> @qwqwqwqwqwqwqw @dolorous_84348 recently, hedge wizard's starting conj skill was buffed from 1 to 2 in this commit https://github.com/crawl/crawl/commit/f62c45d80e09186633edcc221ca77e5aeaee0cfd but somehow, this was rolled back in current trunk version. maybe this commit was the cause? https://github.com/crawl/crawl/commit/17d8ae0da54e5c3c50e6444fe3f3751fe2dfe328 I'm completely ignorant about coding, so I could be very wrong. thanks 03:58:15 for your work as always, devs. 04:31:05 hjklyubn: good spot, obviously got missed when my datify-jobs PR was updated to current trunk. That'll be a simple fix 05:03:33 <13qwqwqwqwqwqwqw> I'll fix it, thanks! 05:09:25 03elliptic02 07* 0.32-a0-1709-g9e1b529eb0: Fix HW's Conj starting skill being accidentally changed to 1 (acrobat) 10(5 minutes ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/9e1b529eb061 05:31:31 <09hjklyubn> thanks! 05:37:38 <06dolorous_84348> That was an oversight on my part, yes. 06:16:44 03dolorous02 07* 0.32-a0-1710-g677be5308e: Remove unused variable (geekosaur). 10(86 seconds ago, 1 file, 0+ 1-) 13https://github.com/crawl/crawl/commit/677be5308e4e 06:24:12 03dolorous02 07* 0.32-a0-1711-g888de62073: Fix more unused variable warnings. 10(2 minutes ago, 1 file, 2+ 0-) 13https://github.com/crawl/crawl/commit/888de62073cd 06:35:57 03dolorous02 07* 0.32-a0-1712-g1c079de84e: Fix fire crab quote. 10(6 minutes ago, 1 file, 6+ 5-) 13https://github.com/crawl/crawl/commit/1c079de84e33 07:07:58 03dolorous02 07* 0.32-a0-1713-gbbbd308e8c: Add a quote for the Malign Gateway portal. 10(4 minutes ago, 1 file, 12+ 0-) 13https://github.com/crawl/crawl/commit/bbbd308e8c60 07:11:11 03dolorous02 07* 0.32-a0-1714-gda96e870eb: Fix spacing. 10(46 seconds ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/da96e870eb59 08:38:44 03dolorous02 07* 0.32-a0-1715-g582db3b95e: Consistently use python3 in scripts (Aliscans). 10(7 minutes ago, 2 files, 2+ 2-) 13https://github.com/crawl/crawl/commit/582db3b95ecf 08:55:19 <04melvinkitnick> FYI, revived CXC server is online: https://crawl.xtahua.com/ Link to the announcement in Xtahua Discord server: https://discord.com/channels/398132596325875723/398132596325875725/1259534984792768575 @gammafunk I see GET queries from CAO in apache logs, though not with the correct path: /crawl/meta/0.31/logfile instead of /crawl/meta/crawl-0.31/logfile (example). I did fix these, though server is now responding with HTTP/416. I'll 08:55:19 look into it. 09:09:51 <04melvinkitnick> 416/HTTP were from what I presume to be another client. Requests from CAO (I have this IP address: 204.140.17.55) comes from wget user-agent (http) and are redirected to https. I only see redirection (301), but no 200/HTTP. Do you have a precise way to ensure data is updated on CAO ? 09:10:54 <09gammafunk> they are probably just sequell 09:11:00 <09gammafunk> well, they may be sequell 09:11:13 <09gammafunk> remember that CAO is for scoring pages, and Sequell is the bot that polls the logs as well 09:11:43 <09gammafunk> might be one of these two: > might need to more carefully account for resist properties when considering domination 09:11:45 <09gammafunk> oops 09:12:00 <04melvinkitnick> I'm thinking trafic should be http, not https. wget might not be able to check cert. 09:12:37 <09gammafunk> traffic should be https but I'm not sure if CAO can check the cert properly 09:13:26 <04melvinkitnick> Ok, this is not one of those; I have: 116.203.245.250. And no trafic so far from loom.shallot.org. 09:13:40 <09gammafunk> I'm not sure which IP sequell is using 09:13:47 <09gammafunk> it might not be either of those 09:13:55 <09gammafunk> but I do see that sequell is configured to use https 09:14:23 <09gammafunk> it looks like sequell has the urls without crawl- 09:15:11 <04melvinkitnick> Ok, I corrected those paths 09:15:13 <09gammafunk> so I think CXC didn't use the prefix in the past 09:15:26 <09gammafunk> yeah, they look ok from my browser 09:15:30 <09gammafunk> !lg * cxc 09:15:35 <04Cerebot> 1366450. Xoff the Destroyer (L10 GnCj of Vehumet), slain by a wyvern on D:8 on 2024-07-01 04:32:40, with 4679 points after 8018 turns and 0:18:09. 09:15:48 <09gammafunk> but sequell still doesn't see any games since shutdown 09:17:28 <09gammafunk> !lg * cxc x=file 09:17:31 <04Cerebot> 1366450. [file=cxc/crawl/meta/git/logfile] Xoff the Destroyer (L10 GnCj of Vehumet), slain by a wyvern on D:8 on 2024-07-01 04:32:40, with 4679 points after 8018 turns and 0:18:09. 09:18:32 <09gammafunk> Have to step away for now, but hopefully we can figure out why it's not pulling. I wonder if the hostname change is lagging simply due to DNS propogation 09:18:51 <09gammafunk> I'm not sure what CXCs TTLs were for its A records in dns etc 09:19:14 <09gammafunk> since I think you only just pointed the host over this morning 09:19:20 <04melvinkitnick> So I have these requests that doesn't seem to hit files (no 200/HTTP afterwards): /var/log/apache2/access.log:204.140.17.55 - - [07/Jul/2024:18:14:12 +0200] "GET /crawl/meta/0.31/logfile HTTP/1.1" 301 623 "-" "Wget/1.16.3 (linux-gnu)" And those ones, that doesn't seem to end up ok (206 or 416): /var/log/apache2/other_vhosts_access.log:crawl.touron.eu:443 116.203.245.250 - - [07/Jul/2024:18:04:25 +0200] "GET 09:19:20 /crawl/meta/0.31/logfile HTTP/1.1" 206 2542 "-" "Wget/1.21.4" /var/log/apache2/other_vhosts_access.log:crawl.touron.eu:443 116.203.245.250 - - [07/Jul/2024:18:14:47 +0200] "GET /crawl/meta/0.31/logfile HTTP/1.1" 416 2300 "-" "Wget/1.21.4" 09:19:48 <04melvinkitnick> 3600 for subdomains 09:20:08 <09gammafunk> has it been an hour since the change? 09:20:16 <09gammafunk> not sure when you did that 09:20:18 <04melvinkitnick> No, it was only like ~2h ago 09:20:32 <04melvinkitnick> a little less even 09:20:47 <09gammafunk> yeah, but 3600 is only one hour 09:21:58 <09gammafunk> ok, I'll take a look at things a bit later and let you know if I come up with anything. Thanks for getting the server working though, I see webtiles loads, so that's good! 09:23:05 <04melvinkitnick> I'm wondering if I should present the whole cert chain instead of just server cert. I'll try a few things and let you know if there's a change. 09:26:55 <04melvinkitnick> Oh, I now see sequell requests, though with a 416/HTTP also: crawl.touron.eu:443 95.217.144.49 - - [07/Jul/2024:18:25:10 +0200] "GET /crawl/meta/0.30/logfile-sprint HTTP/1.1" 416 4141 "-" "Sequell httpfetch/1.0" 09:27:36 <04melvinkitnick> !lg * cxc x=file 09:27:40 <04Cerebot> 1366452. [file=cxc/crawl/meta/git/logfile] SoakedWettersCrisps the Caller (L2 FeSu), blasted by Sigmund (magic dart) on D:2 on 2024-07-07 16:05:57, with 19 points after 1088 turns and 0:03:06. 09:28:57 <04melvinkitnick> Seems ok 🙂 bash 18:28|root@rex:~# tail -1 /var/www/crawl/meta/git/logfile 09:28:58 v=0.32-a0:vlong=0.32-a0-1708-g51ab8bf2ec:lv=0.1:tiles=1:name=SoakedWettersCrisps:race=Felid:cls=Summoner:char=FeSu:xl=2:sk=Summonings:sklev=4:title=Caller:place=D::2:br=D:lvl=2:absdepth=2:hp=-5:mhp=13:mmhp=13:mp=4:mmp=6:bmmp=6:str=4:int=18:dex=16:ac=1:ev=16:sh=0:start=20240607160250S:dur=186:turn=1088:aut=10877:kills=16:gold=13:goldfound=13:goldspent=0:scrollsused=0:potionsused=0:sc=19:ktyp=beam:killer=Sigmund:killer_flags=unique:dam=9:sdam=9:tdam=9:kau 09:28:58 x=magic dart:end=20240607160557S:killermap=uniq_sigmund:seed=14258242502543083399:tmsg=blasted by Sigmund:vmsg=blasted by Sigmund (magic dart) 09:30:09 <04melvinkitnick> Now we just need to ensure scoring on CAO is updated as well. I'll see what I can find; TTYL ! 09:36:26 03dolorous02 07* 0.32-a0-1716-g64a9791703: Avoid crashing at Yes/No/All prompts. 10(5 minutes ago, 1 file, 10+ 1-) 13https://github.com/crawl/crawl/commit/64a9791703af 09:36:26 03dolorous02 07* 0.32-a0-1717-g484b3ae991: Remove duplicate word in comment. 10(2 minutes ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/484b3ae99189 09:47:08 03dolorous02 07* 0.32-a0-1718-g281234c535: Properly check for unsigned char range (oops). 10(3 minutes ago, 1 file, 2+ 4-) 13https://github.com/crawl/crawl/commit/281234c53597 11:55:17 03elliptic02 07[trog-gifts] * 0.32-a0-1691-g866b5a703d: Increase the effect of Trog's Hand further while mesmerised with TrogMesm 10(39 minutes ago, 2 files, 7+ 2-) 13https://github.com/crawl/crawl/commit/866b5a703d29 11:55:17 03elliptic02 07[trog-gifts] * 0.32-a0-1692-gdab9969f9c: Tweak the enchantment distribution on Trog gifts 10(2 minutes ago, 1 file, 2+ 1-) 13https://github.com/crawl/crawl/commit/dab9969f9cef 15:10:57 <09gammafunk> !lg * cxc won 15:10:59 <04Cerebot> 20258. Nebesus the Champion of Chaos (L27 MiFi of Makhleb), escaped with the Orb and 5 runes on 2024-07-01 00:37:31, with 3102241 points after 76297 turns and 1:50:18. 15:39:00 Unstable branch on underhound.eu updated to: 0.32-a0-1718-g281234c535 (34) 19:39:08 -!- The topic of #crawl-dev is: Crawl Development | https://github.com/crawl/crawl | Logs: http://s-z.org/crawl-dev/, temporarily http://crawl.akrasiac.org/logs/cheibriados/ | People with +v have commit access, devs on bridged discord as well | General Crawl-related chat to #crawl | Long stuff to a pastebin service, please 19:39:08 -!- The topic of #crawl is: Play Dungeon Crawl Stone Soup online now! Type ??online for instructions, ??lg / !lg for play stats | PM Sequell for long queries | http://crawl.develz.org | FooTV game replays: ??footv for instructions | #crawl-dev for dev discussion, #crawl-offtopic for offtopic 21:30:55 <12asciiphilia> @gammafunk - If the DEXPERIMENTAL_BRANCH is included in the build process, the explbr column is added to the logfiles and milestones. - This column is intended for recording experimental branches and is generally not used. (Is it okay if I remove this using commands like sed?) - This explbr column is displayed in all build versions (trunk + stable) on the CNC server. Is this correct for the current situation? 21:31:55 <12asciiphilia> Ah, there's a conditional branch in the build command. I see it now. 21:32:04 <12asciiphilia> bash ifneq ($(SRC_BRANCH),$(filter master release stone_soup-%, $(SRC_BRANCH))) DEFINES_L += -DEXPERIMENTAL_BRANCH=""$(SRC_BRANCH)"" endif 21:32:08 <09gammafunk> That is correct, although I'm not sure what you mean by removing using that using sed 21:32:23 <09gammafunk> Yes, that Makefile block will run a git command to check the branch 21:32:56 <09gammafunk> and add the - DEXPERIMENTAL_BRANCH if the git command does not output either master or a string beginning with stone_soup- 21:32:57 <12asciiphilia> I meant to ask if I need to edit the log files if those are incorrectly recorded. 21:33:14 <09gammafunk> I would strongly advise against that 21:33:29 <09gammafunk> those files are written to constantly by crawl processes 21:33:43 <09gammafunk> you really want to fix the build itself so they don't get added 21:34:04 <09gammafunk> for now, we have sequell strip the values out anyhow for cnc 21:34:16 <09gammafunk> so you can fix this at your leisure, but I think it's better to fix it the right way 21:34:31 <12asciiphilia> Of course, I will modify the build process to fix the issue, rather than just changing it once. But I am asking whether I should change the already recorded entries. 21:34:42 <09gammafunk> manipulating the logfile/milestones files while active crawl processes are running is dangerous 21:34:48 <09gammafunk> no, not necessary I think 21:34:52 <09gammafunk> sequell has the correction already 21:35:02 <09gammafunk> and for CAO scoring, I don't think it reads this field at all 21:36:21 <09gammafunk> yeah, in this case, you really can't edit those files I think, because Sequell has already read them and will be confused if their file sizes change after an edit 21:36:24 <12asciiphilia> Later, I wanted to support experimental branches on my server if possible. Will this cause any issues in the future? If there are no major issues, I will not make any modifications. 21:37:08 <09gammafunk> no, presumably if you fix your git branch, it will just mean that master/stable branches don't get the explbr field added 21:37:18 <09gammafunk> if you had an experimental, it would get added (due to the Makefile) 21:37:30 <09gammafunk> I think the key place to fix is your git repository 21:37:39 <09gammafunk> I'm not sure why it represents the branches the way it is 21:37:59 <09gammafunk> it's possible we could improve the Makefile to check the branch name more robustly 21:38:16 <09gammafunk> but I'm not sure what would be a better command 21:41:14 <12asciiphilia> I'm currently in the process of applying to graduate school, so I can't fix this issue right away. I have an idea of what might be causing the problem, but I won't be able to test it until the weekend. 21:41:30 <09gammafunk> It's not problem, please take your time 21:41:44 <09gammafunk> elliptic put a fix in sequell to deal with the issue 21:42:14 <12asciiphilia> Yes, I noticed that something was added that seems to modify it with a regular expression. 21:43:25 <09gammafunk> @asciiphilia And just to explain regarding logfile/milestones file edits. You want to avoid editing those logfile and milestones files after they have been consumed by Sequell and CAO. Those services rely on hashes of lines and remember how many bytes they have read of those files. Both of those things would become invalid if you edited the logfile or milestones files in any way other than adding new lines (adding new lines happens 21:43:26 all the time from crawl processes and is expected) 21:45:52 <09gammafunk> So it's best to just use the Sequell fix for the old lines that have the explbr field and then fix the build to eventually stop new lines from having that field 21:46:13 <09gammafunk> Rather than editing the files, since that will cause problems with Sequell and CAO scoring 21:46:41 <09gammafunk> (I'm not sure of the exact details of all the problems it will cause, but it won't be good) 21:48:05 <09gammafunk> And good luck with gradaute school applications! 21:48:56 <12asciiphilia> Since I'm not very familiar with sequell, I couldn't understand everything. However, is it correct to say that because it has a mechanism to recognize line-by-line hashes, it would be dangerous for me to arbitrarily modify the file, and that since sequell already has a feature to correct erroneous parts of the log file, I can use that feature to fix the previous incorrect parts? 21:50:25 <09gammafunk> Right. It would cause problems for CAO scoring as well. Not because of line-by-line hashes, but because byte ranges would be invalid. Both services remember how many bytes they have read of those files and only request bytes since their last request 21:50:41 <09gammafunk> This is because these files can become extremely large and we don't want to request the whole file each time 21:51:03 <09gammafunk> So editing the files breaks multiple aspects 21:51:55 <12asciiphilia> Anyway, I will try to resolve the issue of incorrect labeling when I have time. Thank you for the detailed explanation. Once this issue is resolved and there are no other potential problems, can it be integrated with the score server? 21:53:11 <09gammafunk> Hopefully it can, I need to ask advil to add it, or try to add it myself. Sometimes CAO has issues with certificates as well, but we can find out when trying to add CNC 21:53:53 <09gammafunk> But yes, I can look into adding it to the score page maybe tomorrow or soon after 21:54:57 <12asciiphilia> Alright, I’ll get back to you (and anvil) once the issue is resolved. 21:55:29 <09gammafunk> fr: advil has an alt called anvil 21:56:38 <12asciiphilia> Haha, sorry, I made a spelling mistake. (Are you serious, not joking?) 21:57:04 <09gammafunk> it was a joke, yes. I actually think it would make a great alternative nick for him though! 21:58:08 <09gammafunk> (Advil medicine can cure headaches and headaches feel like pounding on your head like it's an anvil...) 22:28:43 03Isaac Clancy02 07https://github.com/crawl/crawl/pull/3873 * 0.32-a0-1364-g46f4bfe78b: Don't let the player take actions when their turn is over 10(3 weeks ago, 45 files, 1301+ 657-) 13https://github.com/crawl/crawl/commit/46f4bfe78be5 22:35:34 Unstable branch on crawl.develz.org updated to: 0.32-a0-1718-g281234c535 (34) 22:58:40 Windows builds of master branch on crawl.develz.org updated to: 0.32-a0-1718-g281234c535 23:13:39 Unstable branch on cbro.berotato.org updated to: 0.32-a0-1718-g281234c535 (34) 23:55:23 Monster database of master branch on crawl.develz.org updated to: 0.32-a0-1718-g281234c535