- User Since
- Jan 6 2017, 3:41 PM (286 w, 2 d)
Did you forget the test file?
Sat, Jul 2
Thu, Jun 30
More auras less performance by the way.
Have you seen https://trac.wildfiregames.com/ticket/3362 ? It also talks about memory pools :)
Also worth mentioning it could maybe used for the Pathfinder https://trac.wildfiregames.com/ticket/3588#comment:4
Wed, Jun 29
@Freagarach I think this is superseeded by your patch.
Tue, Jun 28
Fair enough. Did you test multiplayer? Rejoining a match ? This shouldn't affect the simulation but it would be bad if it broke serialization or something.
Mon, Jun 27
Yes. I'd like people to accept it. It's a “big” change.
Fix @Langbart's notes, mark autoresearched techs as root, and reference paired techs. add deps for identity techs.
Thanks for testing. Wonder why it does not find the others I will investigate.
Seems there is no more leak anymore. :)
Did you test on combat demo huge ?
Feel free to make some improvements as well. :)
Profiler2 also allows you to load jsonp files from other runs to do the comparison :)
Sun, Jun 26
#!/usr/bin/env python3 # -*- mode: python-mode; python-indent-offset: 4; -*- # SPDX-License-Identifier: GPL-2.0-or-later # SPDX-FileCopyrightText: © 2022 Wildfire Games
Thanks for the patch.
Sat, Jun 25
Does it make that code slower?
Can you fix all the while( to while ( and for( to for (?
Fri, Jun 24
Can we mitigate this? Now that we have a new RB we gained more time to fix things better.
Thu, Jun 23
Thanks for the patch.
LGTM, I was afraid it would mean a too small shadowmap (and thus low quality) on high end settings, but if @vladislavbelov says it's okay then it's fine by me.
Wed, Jun 22
$ python ./find-missing-config2-includes.py -m "CONFIG2_" -v INFO - Parsing C:\Dev\Perso\0ad-svn\source INFO - Found 709 source files to parse. INFO - Found 795 header files to parse. INFO - Found 14 files with missing headers for CONFIG2_. ERROR - C:\Dev\Perso\0ad-svn\source\graphics\TextureConverter.cpp ERROR - C:\Dev\Perso\0ad-svn\source\lobby\scripting\JSInterface_Lobby.cpp ERROR - C:\Dev\Perso\0ad-svn\source\renderer\backend\gl\ShaderProgram.cpp ERROR - C:\Dev\Perso\0ad-svn\source\renderer\backend\gl\DeviceCommandContext.cpp ERROR - C:\Dev\Perso\0ad-svn\source\ps\Profiler2GPU.cpp ERROR - C:\Dev\Perso\0ad-svn\source\graphics\TextureConverter.h ERROR - C:\Dev\Perso\0ad-svn\source\renderer\ShadowMap.cpp ERROR - C:\Dev\Perso\0ad-svn\source\ps\Profiler2GPU.h ERROR - C:\Dev\Perso\0ad-svn\source\ps\GameSetup\HWDetect.cpp ERROR - C:\Dev\Perso\0ad-svn\source\renderer\backend\gl\Device.cpp ERROR - C:\Dev\Perso\0ad-svn\source\renderer\SceneRenderer.cpp ERROR - C:\Dev\Perso\0ad-svn\source\lib\posix\posix_aio.h ERROR - C:\Dev\Perso\0ad-svn\source\ps\Util.cpp ERROR - C:\Dev\Perso\0ad-svn\source\ps\scripting\JSInterface_Game.cpp
I'm a bit torn between ugly buttons and overlapping text ^^ At least people are used to overlapping text?
AIEND seems not to support the removal of units, though.
Tue, Jun 21
Might wanna test units demo
We were talking about https://m.youtube.com/watch?v=qH6sSOr-yk8&feature=youtu.be might be of some interest to you too
I think you introduced a memory leak on Windows. This shows up in the test output failing them.
Can you show some screenshots and fix the lint warnings? :)
Mon, Jun 20
Might ask @AIEND specifically.
Fine by me.
@borg- feel free to accept if you like it. The sooner the balancing patches get merged the sooner RC2 comes and maybe becomes A26.
@real_tabasco_sauce No, it's not wrong to accept, in fact, we are more inclined to committing patches that have been formally accepted, since it's hard to get a consensus, it can give us a sense of majority.
Thanks for the update. No need to add the po files an automatic script takes cares of them :)
Well we agree. We were talking about it with Vladislav. The issue is that as of right now we don't have a way to restrict options per platform (e.g. the option menu doesn't know about the hardware and there is currently no coded way to make it aware of it) this is something we hope to have for A27
Sun, Jun 19
Sad, but okay. Might be nice to clean the template and remove the bonuses while at it.
Sat, Jun 18
Fri, Jun 17
Patch is complete, see P277.
You can commit it or I will.
Thu, Jun 16
Sure more options are always nice. I don't mind this being committed if @Freagarach agrees :)
Does it make sense to have two randoms? E.g. Random culture and Random ? I agree this should be the default.
Wed, Jun 15
Tue, Jun 14
Mon, Jun 13
Any change needed in BuildingAI?
Is it threads or threats?
Sun, Jun 12
I don't see why we would ever need two profiler instances? Running multiple matches?
Looks like a bugfix that could go in for me but your call :)