IRC logs of Libera.Chat #BZFlag for Monday, 2023-10-16

*** Cobra_Fast is now away: offline00:03
*** Cobra_Fast is back00:03
BZNotifybzflag: Zehra opened issue #337 "[Bug] flagid in bz_ePlayerDieEvent may be incorrect" (https://github.com/BZFlag-Dev/bzflag/issues/337)02:07
ZehraIn some moments, the proper method of bug reproduction will be demonstrated with associated plug-in.02:39
AgathaZehra: you can pick up a flag from a distance away from it, so this isn't workable even in theory. Why not just keep track of the position of each flag?02:56
*** I_Died_Once <I_Died_Once!~I_Died_On@c-73-184-170-223.hsd1.ga.comcast.net> has quit IRC (Ping timeout: 258 seconds)02:58
ZehraAgatha: Thanks for the insights. This seems like a way more viable approach and makes more sense.02:59
Zehraon second thought, now I realize one thing. I mistook the allowedFlagGrab event with the flagGrabEvent.03:00
ZehraThus I may be able to pull the positional data of the flag before it is grabbed.03:01
BZNotifybzflag: Zehra commented on issue #337 "[Bug] flagid in bz_ePlayerDieEvent may be incorrect" (https://github.com/BZFlag-Dev/bzflag/issues/337#issuecomment-1763657368): The following has been tested and reproduced with `BZFlag 2.4.26`...03:09
ZehraUpdated with steps for reproduction and with plug-in for additional ease of bug reproduction/documentation.03:10
*** Flash <Flash!~Flash@user/flash> has joined #bzflag04:00
*** Flash_ <Flash_!~Flash@user/flash> has quit IRC (Ping timeout: 258 seconds)04:02
*** FastLizard4 is back04:10
*** FastLizard4 is now away: AWAY from keyboard04:12
*** Zehra <Zehra!~Yukari@user/yukari> has quit IRC (Quit: Quit.)04:34
*** FastLizard4 is back05:27
*** FastLizard4 is now away: AWAY from keyboard05:27
*** FastLizard4 is now away: IN-GAME - Starfield05:28
*** Cobra_Fast is now away: offline06:26
*** Cobra_Fast is back06:26
*** Sgeo <Sgeo!~Sgeo@user/sgeo> has quit IRC (Read error: Connection reset by peer)08:02
*** FastLizard4 is back09:06
*** FastLizard4 is now away: AWAY from keyboard09:30
*** I_Died_Once <I_Died_Once!~I_Died_On@c-73-184-170-223.hsd1.ga.comcast.net> has joined #bzflag10:12
*** FastLizard4 is back10:41
*** FastLizard4 is now away: AWAY from keyboard11:22
*** FastLizard4 is now away: GONE - Screen Detached and Disconnected from IRC (I'm probably asleep, at work, or doing something in real life)11:48
*** FastLizard4 is back17:16
*** FastLizard4 is now away: AWAY from keyboard17:25
*** Flash_ <Flash_!~Flash@user/flash> has joined #bzflag18:04
*** Flash <Flash!~Flash@user/flash> has quit IRC (Ping timeout: 258 seconds)18:07
Juestany idea why 2.5/2.6 has increased jitter?20:23
Jueston windows20:23
SpringTankbugs20:49
SpringTankSomeone was coding in light mode and it attracted bugs.20:49
blast007Juest: nope, not yet21:06
blast007does it happen if you host a server locally too?21:07
Juestblast007: dont remember but i think yes probably21:10
blast007how many FPS are you getting when you have the jitter issue?21:14
Juestdidnt check21:14
blast007I am seeing a higher jitter on master than 2.4, but it's still not *high*21:20
blast0072ms on 2.4, 6 to 7 on master21:20
blast007(to a remote server)21:20
Juesti had a jitter high as 70ms21:20
blast007that's what I saw previously, but now I don't know how to recreate it :)21:21
blast007I did just have 4ms even connecting to localhost21:22
blast007oh wow, in 2.4 on localhost, I'm getting 200+ ms of jitter, but only 1ms of lag :P21:24
blast007fixed the jitter on 2.4 (by commenting out the QueryPerformanceCounter recalibration code, which is part of why bzfs hosting on Windows is so bad), and now it shows 0 (as it should be, all things considered)21:29
blast007Juest: have you never run a git bisect before?21:30
Juestblast007: not really something i do but i think i could give it a try21:30
blast007it might be more annoying if you don't have Visual C++ 2017 though, since 2019 is going to probably upgrade the project files21:31
blast007hmm, actually a bisect might not be super useful against a localhost server, since at least some of the revisions will have the buggy QPC recalibration...21:32
blast007I'll see if I can work through a bisect here21:34
Juestso this jitter bug affects 2.4?21:34
Juestblast007: i do have vs 2017, 2019 and 2022 installed21:35
Juestbut not old versions21:35
blast007different bug21:35
Juestoh ok21:35
Juesti had the high jitter on master when joining your server21:36
Juesti can check the jitter on linux to double check21:36
blast007on 2.4 Windows, the QPC recalibration, at least on the server, makes time run at like 2/3rds speed, so as you might imagine, that screws some stuff up :)21:36
blast007I think I removed the recalibration code on master21:37
blast007the basic idea of git bisect is that you find a known good version and a known bad version, and then it will skip to revisions between those and you tell git if that step was good or bad, and it helps you track down the revision that introduced an issue21:38
Juestim seeing 54-66ms of jitter in a local server on master at 180 fps21:39
Jueston a i7 4790k windows 721:39
Juestchecked in release build21:39
blast007let me try on my Windows 7 system21:42
blast007is this on a Wifi connection, or wired?21:42
Juestwired21:42
Juestand localhost21:42
Juesti have more jitter on 2.4 than on master21:42
blast007oh, right.. localhost.. so.. kinda irrelevant :)21:42
blast0072.4 on localhost?21:43
Juestyes on localhost21:43
blast007yeah, the 2.4 server time is skewed, so calculations like that will be very broken21:43
Juest260-270-33021:43
Juestah bummer21:43
Juestwhich bzexcess server i can use as control for testing on 2.4?21:43
blast007any of them.  they're all on the same server.21:44
Juestthe jitter is good on your server21:47
Juest138 +- 2ms21:48
Juesta spike of 17-24 ms at one point21:48
Juestanother master issue is the libpng warnings about gAMA21:49
Juestthe jitter is like 60ms on master21:49
Juesti've grabbed a unknown flag lol21:51
Juestwhat it does?21:51
blast007unknown flag in master?21:51
Juestyes21:52
blast007huh, that's.. weird :P21:52
Jueston your 1.7 good flags map21:53
blast007when you were holding that flag, I also couldn't right click to identify you21:54
Juestoh huh21:54
Juestany idea when those bugs we found broke?22:08
blast007nope22:08
blast007but that's what git bisect can help with :)22:08
Juest:D22:08
blast007we found 4 new bugs:22:09
blast007* Can't respawn after a SteamRoller death22:09
blast007* Can't identify tanks22:09
blast007* Lock on warning does not show up22:09
blast007* "Unknown" flags spawning (shows up as "Unknown" when you grab them and has an abbreviation of --)22:10
Juestwait, the gm trajectory works fine?22:10
blast007yeah, it was still curving towards you on my screen22:10
blast007I was aiming it past you so it didn't hit22:10
Juestah i think i saw that22:10
Juesti see, barely noticed22:10
Juestdoes git bisect take up storage?22:10
blast007no22:11
Juestah good22:11
Juestcould do it22:11
blast007it just jumps back and forth through revisions and updates the working directory to that revision22:11
Juesti could use windbg with the pdb files in place 22:11
Juesti need the pdbs because im not doing debugging on the same computer as the one that built the game22:12
blast007for this you wouldn't need the debugger even22:12
blast007it's more about tracking down which revision introduced a specific issue22:12
Juestfiguring out the piece of code that causes the bug is what i need the debugger for22:12
Juestunless you know the piece of code responsible for this22:12
blast007oh, sure, if you want to go at it from that angle22:13
blast007I was going to bisect it22:13
Juestsure go ahead22:13
blast007solo bots might be enough to test 3 of the issues22:13
Juestoh, and the bug with the rogue skin not applying? what code do you think is it?22:13
blast007I did have a fix for the rogue skin, but to make it easier I also removed that server-side plugin event ;)22:14
blast007so... I haven't pushed that quite yet22:14
JuestAh22:14
Juestunderstood22:14
Juestyour server is not exactly vanilla?22:14
blast007no, it is22:15
Juestwell it has plugins22:15
Juestnvm i did not read that properly22:15
blast007the plugins I have shouldn't really affect anything22:16
blast007nothing gameplay related at least22:16
blast007okay, I lied..  I guess I do have the 'genocide' plugin (which I wonder if that was the Unknown flag) and thiefControl (which does affect the behavior of thief)22:17
Juest:D22:18
Juestdoes the unknown flag go away if you unload the genocide plugin?22:18
blast007or are they ALL unknown if I just do +f G{30} with no other flags :D22:19
Juestcould i ask you to have a server to use as testing ground like this?22:19
Juestor localhost is there for that matter?22:20
Juesti was using the start server functionality22:20
blast007xs.bzexcess.com:5432 is running with +f G{30} +f ST, so it should be 30 geno and 1 stealth22:21
Juesti noticed, there's no way to change ports and it waits until a existing bzfs instance shuts down before starting a instance22:21
Jueston master?22:21
blast007yeah22:21
Juestwhy not use 255432?22:21
Juestoh did you mean 25432?22:22
blast007beause that's higher than the maximum port of 6553522:22
Juestthe master servers are running on 2nnn compared to 2.4 running on the nnnn22:22
blast007let me check if it actually shows up on the list.  I just hand-write a config quick.22:22
Juestah22:22
Juestits not showing up22:23
Juest5432 you sure? not 25432?22:23
blast007it's just a number.22:24
Juestanyways im getting error connecting to server on that port22:24
blast007k, let me check22:24
Juestfor consistency please host it on 25432 to differenciate master and 2.422:24
Juestfirewall is the issue perhaps?22:27
*** Sgeo <Sgeo!~Sgeo@user/sgeo> has joined #bzflag22:28
blast007I had typoed the port number the second time :)22:31
blast007but.. geno wasn't the problem22:31
Juestthe game still accepts input when in the background22:40
blast007ah, we have a Null flag type now22:41
blast007so it's adding that :)22:41
blast007I bet that's part of flag effect decoupling22:42
*** Zehra <Zehra!~Yukari@user/yukari> has joined #bzflag23:20
ZehraI heard of some developments/bug reports for master/2.5/future 2.6 happening.23:22
ZehraI do suggest that the scoring system be changed on quite a few plug-ins, as they lead to sub-optimal scoring.23:23
ZehraAdditionally, there is the concern that such plug-ins do corrupt the scoring of statistic sites. (Virtually all custom game modes within the official distribution are known for said issue.)23:24
Juestwould be nice to have some kind of qa or unit testing to catch those bug gotchas that accumulate over time23:26
ZehraMy suggestion would be that some form of team scoring can be added to KeepAway and KoTH, as both modes do not feature proper team scoring integration, either by flaw or by design.23:26
Juestinteresting23:27
ZehraJuest, I agree. I am very interested in the possibility of aggregating the data of bug reports/known issues and summarizing them in some form of an FAQ.23:27
blast007unit tests for BZ would be like this: https://www.reddit.com/r/ProgrammerHumor/comments/176sggj/allunittestspassing/23:28
Juestwhy not actually fix them besides making bulletins of discovered bugs?23:28
ZehraHence, it would be possible to simply address most bugs or trace them rather easily via search formats.23:28
Juesti see23:28
Zehrawell, some bugs are nearly impossible to reproduce, as they are glitches, generally relating to the cache or similar.23:28
Zehraa.k.a. corrupted configs leading to clients not properly starting..etc23:29
blast007Zehra: no, Github Issues are how we track bugs23:29
blast007why would we create a summary of github issues?23:30
blast007just look at the issues23:30
Juestlol, the joke is that the sink works but its almost broken except for the basic functionality?23:30
Zehrablast007: But many "bugs" are actually user errors.23:30
Zehraa.k.a. a configuration option being incorrect or needing to be added.23:30
ZehraSimilarly, many of the "options" people may want, already exist, but aren't readily known.23:31
blast007Zehra: then if they are user errors, why would they be github issues?23:31
blast007Juest: yeah.  It's passing all the unit tests.  Water runs, goes down the drain.  :)23:32
Zehrablast007: The bugs and known issues would reference the forums bug/known issues sub forum.23:33
Juest...23:33
ZehraActual issues are added to GitHub.23:33
blast007Zehra: I guess I don't understand what your point is then23:34
JuestZehra: configuration errors are bugs as long its not the user misunderstanding23:35
blast007yeah, the configuration file being filled with NUL bytes is a bug23:35
Juestonly if it was not intentional23:35
blast007might be some race condition with writing the config from two clients at once23:35
blast007cuz I did get someone's corrupted config once, and the size was appropriate for a configuration file, but the entire contents was NUL bytes23:36
Juestif you click save configuration on multiple instances at the same time it corrupts?23:36
blast007I was thinking that one scenario would be if you had multiple clients running, all with unsaved changes, and you log off or shut down23:37
Juestthat too23:37
Juesti was having in mind that as well23:37
blast007all the clients would try to save the config are roughly the same time, so that might trigger the issue23:37
Juestsomething like a log off is likely to cause that23:37
Juestparticularly on windows23:37
ZehraOr abrupt power cutoff.23:38
blast007power cutoff wouldn't try to save the config23:38
Juestno Zehra, config gets saved when the application does its shutdown routine23:38
Juestin linux its more common to have killed applications instead of a proper log out like on windows23:39
Juestas in, do a logout via killing apps instead of telling them to shut down23:39
blast007maybe it's something that would be solved by just not overwriting the file to begin with23:40
blast007use a rename operation and a second file with the new contents23:40
Juestmaybe use intermediary file states?23:40
blast007or maybe something where we need to lock the file23:41
Juestyeah something like config.new that then gets moved to config.cfg23:41
Juestgreat idea23:41
Juestconfig.lock23:41
blast007or we go crazy and switch to using an SQLite database ;)23:41
Juestbut it would cause problems with multiple instances23:41
Juestunless you dont care about the config when using multiple instances23:41
Juestabout saving it*23:41
blast007we do23:41
Juestas the user i meant23:41
blast007multiple clients is relatively common23:42
Juestwhy not encourage the practice of specifying the config file for each multiple instance then?23:42
blast007no23:42
Juesthuh23:42
blast007people aren't going to to the command line to run a second client23:42
blast007they're clicking on the menu shortcut23:43
Zehra@blast007: https://forums.bzflag.org/viewtopic.php?t=19628 https://forums.bzflag.org/viewtopic.php?t=20171 Example of two threads relating to bugs which could be resolved by FAQ.23:43
blast007Zehra: that's a game bug23:43
blast007(the first one)23:43
* Juest rolls eyes23:43
blast007the second one would NOT be a *frequently* asked question23:44
blast007that was a very specific scenario23:44
ZehraI probably read the forums too much that everything looks like FAQ :p23:45
Juestsounds like your brain is the problem here :)23:46
ZehraI know, I'm only human after all... don't place the blame on me. :P23:46
ZehraJuest, also I highly doubt most players run their clients via command line or some shortcut.23:47
blast007I'm actually ChatGPT. Gotcha.23:47
Zehralol23:47
blast007Zehra: most *do* run it via a shortcut, cuz that's what Windows uses :)23:48
Juestlinux users who installed from the repositories too23:48
blast007on linux it's through a desktop entry (a .desktop file)23:48
Juestand mac of course, has a very involved way of specifying arguments so most people run it without arguments23:48
blast007mac is special23:48
Juestmhm, binary bundles land23:49
blast007there's no menu there for apps, really23:49
Juestyou invoke the app directly23:49
Juestand its a gui basically23:49
blast007a .app is actually a directory that is treated special23:49
blast007you can cd into an .app 23:49
Juesti know :)23:50
Juestits called bundles23:50
Juestthey're*23:51
Juestis it possible to toggle hideadmin on yourself?23:51
Juesthow are people admins and not admins sometimes?23:51
ZehraI got a few plug-ins for that trick.23:52
blast007it's a group permission23:52
ZehraAlso depends on global groups.23:52
blast007also, some server might use /password instead of groups, so they'd be a normal user until they supply the admin password23:52
blast007servers*23:52
Zehra^23:52
Juesthow can you tweak the global group permissions?23:52
ZehraYou use a global group file...23:53
blast007I don't recommend using /password on public servers though, since it's too easy to typo the command and send the admin password to the public chat :)23:53
Juestglobal groups live on the forum dont they?23:53
Zehraa.k.a. sample.globalgroup: spawn talk...etc23:53
blast007yeah, global groups are created on the forum and you manage members then23:53
Zehragive me a second...23:53
Juesti asked how do you manage the permissions?23:53
blast007but the groupdb file that bzfs reads is what assign permissions to groups23:53
Juestoh 23:53
Juestso the permissions is handled by the groupdb irrespective of global or local group?23:54
blast007yeah23:54
blast007and you don't have to own a group to assign rights to it23:54
Juestlong ago i had a setup of a bzfs23:54
Juestbut i lost that stuff23:54
Zehrathat's how some servers assign admins, like they'll add in one of the more popular group configurations.23:55
Zehraoh, also don't forget, superUser can be used for permissions... no global groups needed.23:56
ZehraBut that is pretty rare, if done at all now days.23:57
ZehraMight be good if you only grant like 3-5 people permissions or so.23:57
* Juest shrugs23:57
Juestthere's some inconsistencies with the server setup and things like that23:58
Juestwhat folder does bzflag read when listing worlds in start server?23:58
Zehrathe /saveworld folder... known as "worlds" IIRC.23:58
Juestah23:59
Zehrabasically where the config file is and a directory down known as "worlds".23:59
ZehraIIRC23:59

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!