Config
Log for #openttd on 6th February 2023:
Times are UTC Toggle Colours
13:28:52  *** KiriDore has quit IRC
14:12:40  <Samu> i just realized queue eats memory
14:12:47  <Samu> i changed into vector
14:14:19  <Samu> oh snap, this isn't working correctly
14:32:01  <andythenorth[d]> why isn't memory free?
14:32:10  <andythenorth[d]> we need a lot more
14:40:50  <petern> Free memory is wasted memory.
14:49:26  <dP> I'm so tempted to just throw away all the command template crap and write my own handling from scratch
14:54:22  <petern> Go on then.
14:55:31  *** WormnestAndroid has quit IRC
14:56:38  <LordAro> it feels like it should be pretty trivial to insert an "intercept" handler in the middle of it
14:56:57  <LordAro> well, as trivial as it can be for a C++ template monstrosity
14:57:31  <Samu> question
14:57:33  <Samu> 		auto p = queue.front();
14:57:33  <Samu> 		std::vector<std::pair<TileIndex, TileIndex>>::iterator pop = std::remove(queue.begin(), queue.end(), p);
14:57:33  <Samu> 		queue.erase(pop);
14:57:41  <Samu> is this gonna do what i want?
14:57:50  <dP> LordAro: I didn't even get to the intercept part yet xD
14:58:09  <Samu> i want to get the item at the front of a vector, then delete it
14:59:14  <dP> I just wanted to have a callback... and if callback in a client requiring some ridiculous hacks is kinda understandable, callback in a server can just be stored as it isn't going anywhere
14:59:17  <Samu> from the queue, which is no longer a queue, it's a vector
14:59:21  <dP> except it can't because of template type hell
14:59:23  <Samu> maybe i should rename
15:04:14  <Samu> nooo, it works, but it's super slow
15:04:17  <Samu> :(
15:04:44  <petern> Why do you need to remove?
15:04:58  <LordAro> and how do you define "super slow"
15:05:11  <Samu> i got about 16 million pairs of tiles
15:05:19  <Samu> or 1.6m
15:05:23  <Samu> 4096x4096
15:05:39  <petern> Why do you have a list that big in the first palce?
15:05:44  <LordAro> ^
15:05:54  <Samu> because... the map is big :)
15:05:57  <Samu> hehe
15:06:01  <LordAro> first rule of optimisation, do less on less data
15:06:22  <Samu> going back to queue being queue instead of vector again
15:07:23  <petern> If you need an entry for every tile on the map, probably your  algorithm is wrong.
15:08:03  <Samu> i want an alternative to queue, because queue is memory hungry
15:08:13  <Samu> vector is light, but doesn't let me pop items in front
15:08:40  <LordAro> i don't see why queue would be (noticably) any more memory intensive than vector
15:10:16  <Samu> one uses about 200 mb, the other 900
15:10:27  <Samu> at least in debug mode
15:10:55  <LordAro> debug mode can bloat a lot, but i wouldn't have expected that much...
15:11:01  <LordAro> not by comparison anyway
15:12:48  <Samu> i tried list, eats even more memory
15:12:50  <Samu> std::list
15:17:50  *** WormnestAndroid has joined #openttd
15:21:21  *** nielsm has joined #openttd
15:27:34  <petern> Still never really answered the question.
15:43:57  <glx[d]> Yeah why store all these pairs ?
15:44:41  <glx[d]> What is the exact goal ?
16:13:32  <Samu> river generation on heightmaps is incredibly slow
16:14:33  <Samu> okay, not that incredible, but it stalled for quite 1 minute
16:19:42  *** gelignite has joined #openttd
16:20:38  *** keikoz has joined #openttd
16:32:22  <DorpsGek> [OpenTTD/OpenTTD] SamuXarick opened issue #10452: [Bug]: Long stalls during river generation on heightmap https://github.com/OpenTTD/OpenTTD/issues/10452
16:36:42  <DorpsGek> [OpenTTD/OpenTTD] owen commented on issue #10251: [Bug]: blue tint with hardware acceleration on https://github.com/OpenTTD/OpenTTD/issues/10251
16:54:52  <LordAro> Samu: i think that might be the wrong screenshot :D
16:54:55  <petern> RGB/BGR?
16:55:16  <glx[d]> slow generation is not surprising on big maps
16:55:26  <LordAro> wait, or are you actually using a screenshot to generate a heightmap?
16:57:14  <glx[d]> IIRC generating a 4k² map takes about 10 minutes for me with debug build
16:57:42  <glx[d]> (or more than 10 minutes, can't remember, I rarely do it)
16:57:50  <LordAro> might be worth a profile to see if there's anything easy
16:58:13  <LordAro> (of heightmaps, not mapgen in general)
17:00:50  <JGR> There's not much point doing any performance measurements of debug builds
17:01:37  <JGR> It's not going to give results which are usefully applicable to proper builds
17:03:39  <TrueBrain> https://news.ycombinator.com/item?id=34674066
17:03:39  <TrueBrain> Frontpage again 😄
17:06:23  <TrueBrain> If you ever feel down, read that thread .. lovely kind words
17:07:26  *** HerzogDeXtEr has joined #openttd
17:10:08  <Samu> that was on 13.0
17:12:00  <petern> Hmm, dare I use my mitre saw?
17:24:37  <TallTyler> It's funny how much people like the horn feature, was not expecting that
17:26:55  * petern spots more UI fixes to do
17:30:43  <frosch> yes, every thread on the internet is about the honking hotkey 🙂
17:30:47  <frosch> same on reddit
17:30:47  <petern> (OpenMSX doesn't sound great)
17:32:30  <frosch> TrueBrain: you can also read the thread as: if zephyris didn't do half of opengfx, openttd would be irrelevant 🙂
17:35:29  <TrueBrain> And they are not wrong 🙂
17:36:41  <LordAro> i do like the "if all these other projects /just/ made their own assets" comments
17:41:38  <TrueBrain> How long can that actually take, I mean, I can make a drawing in 10 minutes, so a talented person only needs a day for all assets right? /s
17:41:58  <LordAro> yeah, we all just have our priorities wrong
17:50:45  <TallTyler> Speaking of OpenGFX, I'm looking forward to seeing it updated with Zephyris' new sprites. Not exactly sure who's taking the lead on adding them though, if it's Zephyris or if there's been a miscommunication somewhere: https://www.tt-forums.net/viewtopic.php?p=1257992#p1257992
17:51:18  <glx[d]> well OpenGFX repo accepts PR
17:52:16  <LordAro> not exactly clear from his last message
17:54:08  <glx[d]> seems to be not ready to be published WIP
17:55:40  <glx[d]> and if opengfx repo can be cleaner as a result it's a bonus
18:01:06  <petern> I don't know how OpenGFX was originally done, but I read that Zeph thinks he just needs to post sprites as an attachment in the forums and someone else will merge it all.
18:01:17  <petern> *i read it as
18:03:48  <andythenorth[d]> has anyone DMed Zeph to say join discord?
18:03:53  <andythenorth[d]> otherwise I shall do that now
18:04:28  <petern> There's 2 zephyris via @...
18:06:14  <andythenorth[d]> ok I DMed anyway to see
18:12:07  *** tokai|noir has joined #openttd
18:12:08  *** ChanServ sets mode: +v tokai|noir
18:18:51  *** tokai has quit IRC
18:37:41  *** supermop_toil has joined #openttd
18:38:58  <DorpsGek> [OpenTTD/eints] Boltyansky opened issue #131: Gender variation for strings returns error https://github.com/OpenTTD/eints/issues/131
18:42:24  <petern> Hmm, okay, my server is working now.
18:44:33  <DorpsGek> [OpenTTD/eints] frosch123 commented on issue #131: Gender variation for strings returns error https://github.com/OpenTTD/eints/issues/131
18:44:53  <DorpsGek> [OpenTTD/OpenTTD] DorpsGek p

Powered by YARRSTE version: svn-trunk