Page MenuHomeWildfire Games
Feed All Stories

Apr 25 2019

Stan retitled D1848: Consider Google Closure Compiler type expressions in JSdoc in Health.js (rP22196) from Fix rP22196 Coding Convention breach to Fix rP22196 Health.js Coding Convention breach.
Apr 25 2019, 3:11 PM
wraitii added inline comments to D1840: RunSpeedMultiplier cleanup: improve variable names and reuse a duplicated function (rP22197).
Apr 25 2019, 3:11 PM
Stan added a comment to D1848: Consider Google Closure Compiler type expressions in JSdoc in Health.js (rP22196).
In D1848#76171, @elexis wrote:

If we have more than one revision that is called "Fix <number>" then noone will know which one is which.
So always name them according to their content.

Apr 25 2019, 3:10 PM
wraitii added a comment to D1828: Health.js cleanup: add tests, add an "IsInjured" function, use GetHitpoints everywhere.
In D1828#76156, @elexis wrote:

Seems unlikely that GetHitpoints returns something other than this.hitpoints, especially inside the same component.
I know the JIT can optimize, but I don't know if it can achieve this specific thing.
It's easy to change this.hitpoints if we ever need some logic change.
If we change it to use the function, then we should have a reason to (i.e. construct a realistic example where the logic would be different from return this.hitpoints).

Apr 25 2019, 3:07 PM
Stan retitled D1848: Consider Google Closure Compiler type expressions in JSdoc in Health.js (rP22196) from Fix rP22196 to Fix rP22196 Coding Convention breach.
Apr 25 2019, 2:56 PM
elexis added a comment to D1848: Consider Google Closure Compiler type expressions in JSdoc in Health.js (rP22196).

If we have more than one revision that is called "Fix <number>" then noone will know which one is which.
So always name them according to their content.

Apr 25 2019, 2:54 PM
Freagarach added inline comments to D1846: Adds a "properties"-property to resources and let mods be able to prevent resources from being bartered, traded and/or tributed..
Apr 25 2019, 2:54 PM
Stan added inline comments to D1846: Adds a "properties"-property to resources and let mods be able to prevent resources from being bartered, traded and/or tributed..
Apr 25 2019, 2:53 PM
Vulcan added a comment to D1846: Adds a "properties"-property to resources and let mods be able to prevent resources from being bartered, traded and/or tributed..

Successful build - Chance fights ever on the side of the prudent.

Apr 25 2019, 2:48 PM
Freagarach updated the diff for D1846: Adds a "properties"-property to resources and let mods be able to prevent resources from being bartered, traded and/or tributed..

Fixed tests.

Apr 25 2019, 2:47 PM
Stan added inline comments to D1846: Adds a "properties"-property to resources and let mods be able to prevent resources from being bartered, traded and/or tributed..
Apr 25 2019, 2:27 PM
Imarok added a comment to D665: Make ship pickup nicer.
In D665#76122, @elexis wrote:

The expected player experience is that the ship moves to the closest point to the shoreline where the units to be picked up are.
The current behavior is that the ship moves to the correct island, but upon every garrisoning moves to a presumed better location again (which turns out to be a waste of time in many cases).
If I understood the patch correctly, the behavior of the patch is that the ship moves to the closest shoreline, which may be on an entirely different island, at least.

So if I understand the patch correctly, it changes one defect for another.
But the patch might be adapted to match the expected user behavior.

One easy fix would be that the ship won't move if the units can be garrissoned with the ship not moving.
Or just never move the ship.

Apr 25 2019, 2:26 PM
elexis added a comment to D1840: RunSpeedMultiplier cleanup: improve variable names and reuse a duplicated function (rP22197).

Will update with your comments, except for the "runspeedmultiplier" thingy.

Thanks for taking the time.
It's important that we are precise about the words that we use instead of hoping that reader already knows what is meant (because not every reader does, sometimes noone knows. And having precise definitions means we can test our claims against the definition of the words).

Apr 25 2019, 2:26 PM
elexis added a comment to D1828: Health.js cleanup: add tests, add an "IsInjured" function, use GetHitpoints everywhere.

Seems unlikely that GetHitpoints returns something other than this.hitpoints, especially inside the same component.
I know the JIT can optimize, but I don't know if it can achieve this specific thing.
It's easy to change this.hitpoints if we ever need some logic change.
If we change it to use the function, then we should have a reason to (i.e. construct a realistic example where the logic would be different from return this.hitpoints).

Apr 25 2019, 2:18 PM
elexis awarded D1847: Optimise out of frustum rendering of textures overlays a Like token.
Apr 25 2019, 2:15 PM
Harbormaster failed remote builds in B7290: Diff 7846 for D1846: Adds a "properties"-property to resources and let mods be able to prevent resources from being bartered, traded and/or tributed.!
Apr 25 2019, 2:09 PM
Vulcan added a comment to D1846: Adds a "properties"-property to resources and let mods be able to prevent resources from being bartered, traded and/or tributed..

Build failure - The Moirai have given mortals hearts that can endure.

Apr 25 2019, 2:09 PM
Freagarach updated the diff for D1846: Adds a "properties"-property to resources and let mods be able to prevent resources from being bartered, traded and/or tributed..

Some fixes and Petra-AI should now follow the constraints, which is difficult to test fully.

Apr 25 2019, 2:07 PM
Vulcan added a comment to D1848: Consider Google Closure Compiler type expressions in JSdoc in Health.js (rP22196).

Successful build - Chance fights ever on the side of the prudent.

Apr 25 2019, 2:03 PM
Stan updated the diff for D1848: Consider Google Closure Compiler type expressions in JSdoc in Health.js (rP22196).

Add missing semicolon to fix Vulkan's warning

Apr 25 2019, 2:01 PM
elexis retitled D1816: Use all three color channels when loading heightmaps following rP21113 from Remove wrong assignment introduced in rP21113 to Use all three color channels when loading heightmaps following rP21113.
Apr 25 2019, 2:01 PM
elexis added a comment to D1190: Allow restoring lost wall turrets.

The new behavior is that lostwall turrets are stored in a local entity and can be restored by the wallplacement algorithm?

Apr 25 2019, 1:59 PM
Vulcan added a comment to D1848: Consider Google Closure Compiler type expressions in JSdoc in Health.js (rP22196).

Successful build - Chance fights ever on the side of the prudent.

Apr 25 2019, 1:59 PM
Stan updated the summary of D1848: Consider Google Closure Compiler type expressions in JSdoc in Health.js (rP22196).
Apr 25 2019, 1:54 PM
wraitii added inline comments to D1839: Fix hotkey event sync with hotkey state..
Apr 25 2019, 1:53 PM
elexis added a comment to rP22182: Add ConquestCritical to structures that want it, instead of removing it from….

structures that want it

(Tiny grammar improvement: Neither structures nor code wants to do something, not even the authors want to do something, but actions are performed to achieve purposes. )
(-> Add ConquestCritical to structures that should have it, instead of removing it from those that should not).

Apr 25 2019, 1:53 PM
Stan updated the summary of D1848: Consider Google Closure Compiler type expressions in JSdoc in Health.js (rP22196).
Apr 25 2019, 1:52 PM
Stan created D1848: Consider Google Closure Compiler type expressions in JSdoc in Health.js (rP22196).
Apr 25 2019, 1:52 PM
elexis added inline comments to rP22188: Move tooltips on gamesetup dropdowns from xml to js, since by current gamestup….
Apr 25 2019, 1:46 PM
Stan added inline comments to rP22196: Correctly handle receiving 0 damage as not receiving any damage..
Apr 25 2019, 1:45 PM
elexis updated subscribers of D1739: Introduce a replacement container for EntityMap, std::map, std::unordered_map and boost::unordered_map.

@fabio would be happy to add boost references if it really improves performance?
(So the question then would be does it, and is it sufficent performance benefit or just an experiment)

Apr 25 2019, 1:43 PM
Stan added inline comments to rP22196: Correctly handle receiving 0 damage as not receiving any damage..
Apr 25 2019, 1:38 PM
elexis added a comment to D665: Make ship pickup nicer.

The expected player experience is that the ship moves to the closest point to the shoreline where the units to be picked up are.
The current behavior is that the ship moves to the correct island, but upon every garrisoning moves to a presumed better location again (which turns out to be a waste of time in many cases).
If I understood the patch correctly, the behavior of the patch is that the ship moves to the closest shoreline, which may be on an entirely different island, at least.

Apr 25 2019, 1:36 PM
wraitii added a comment to D1828: Health.js cleanup: add tests, add an "IsInjured" function, use GetHitpoints everywhere.
In D1828#76107, @elexis wrote:

But why replace this.hitpoints with this.GetHitpoints()? Only adds performance cost without adding benefits, doesn't it?

Apr 25 2019, 1:32 PM
wraitii added a comment to D1840: RunSpeedMultiplier cleanup: improve variable names and reuse a duplicated function (rP22197).

Will update with your comments, except for the "runspeedmultiplier" thingy.

Apr 25 2019, 1:29 PM
elexis raised a concern with rP22196: Correctly handle receiving 0 damage as not receiving any damage..

Taking the red ink because either the coding conventions should be changed or this new Google syntax style and the practices of not abiding by conventions should change.

Apr 25 2019, 1:27 PM
Stan added inline comments to D1828: Health.js cleanup: add tests, add an "IsInjured" function, use GetHitpoints everywhere.
Apr 25 2019, 1:19 PM
elexis added inline comments to D1839: Fix hotkey event sync with hotkey state..
Apr 25 2019, 1:09 PM
elexis added a comment to rP22200: Remove workaround in GetGUIObjectByName.

(Could have said that you commit this unless I commit it.)

Apr 25 2019, 12:49 PM
elexis added a comment to D1828: Health.js cleanup: add tests, add an "IsInjured" function, use GetHitpoints everywhere.

Test looks goodish from a superficial look (assuming that the tested values are the intended values)

Apr 25 2019, 12:47 PM
elexis retitled D1840: RunSpeedMultiplier cleanup: improve variable names and reuse a duplicated function (rP22197) from rP22197 cleanup: improve variable names and reuse a duplicated function. to RunSpeedMultiplier cleanup: improve variable names and reuse a duplicated function (rP22197).
Apr 25 2019, 12:42 PM
elexis added inline comments to D1815: match unit classes with templates.
Apr 25 2019, 12:41 PM
elexis added inline comments to D1840: RunSpeedMultiplier cleanup: improve variable names and reuse a duplicated function (rP22197).
Apr 25 2019, 12:38 PM
vladislavbelov added a comment to D1847: Optimise out of frustum rendering of textures overlays.

I think perf impact is negligible (it's definitely better, but probably not by much, and the main impact is likely a lower number of draw calls).

Yeah, the same thoughts. It shouldn't increase and might decrease rendering time. So it's worth to not render invisible objects.

Apr 25 2019, 12:36 PM
elexis retitled D1817: Remove unused variable and fail to register component if it has no constructor [ComponentManager] from Remove not needed variable and conversion to object to Remove unused variable and conversion in the ComponentManager.
Apr 25 2019, 12:31 PM
wraitii added a comment to D1847: Optimise out of frustum rendering of textures overlays.

I think perf impact is negligible (it's definitely better, but probably not by much, and the main impact is likely a lower number of draw calls).
Probably a good idea to do it anyways though.

Apr 25 2019, 12:29 PM
elexis added a comment to D1772: Fix TLS Segfault on various mac versions.

It's not that I'm not concerned. It's that I've weighted what I estimate to be the chance of 'it fixes the issue with 0 additional work' vs the chance of 'it actually makes things worse' and I honestly believe the calculation comes out to "overall it should be committed now".

Apr 25 2019, 12:28 PM
vladislavbelov added a comment to D1847: Optimise out of frustum rendering of textures overlays.
In D1847#76051, @Stan wrote:

What's the perf gain like ?

Apr 25 2019, 12:26 PM
vladislavbelov added inline comments to rP22225: Fix D1491 which introduced an ENSURE that should not have been there..
Apr 25 2019, 12:20 PM
Stan added a comment to D1584: Seed random sounds.
In D1584#76088, @elexis wrote:

std::mt19937 looked good to me, as Vladislav said since it's used in the other places too.
Actually the simulation and AI seed use boost::rand48.
But still better than using custom magic numbers?
The only possible performance problem was reinitializing the seed often right?
Wouldn't it be sufficient initialize the seed upon component construction once? Or upon first soundplay?
(Though the magic numbers aren't so bad, only one line, or four including the wrapping.)
(Especially if it's some orders of magnitude faster. But perhaps there are other std or boost RNG that are also orders of magnitude faster.)

Apr 25 2019, 12:17 PM
Freagarach added inline comments to D1846: Adds a "properties"-property to resources and let mods be able to prevent resources from being bartered, traded and/or tributed..
Apr 25 2019, 12:17 PM
elexis added a comment to D1584: Seed random sounds.

std::mt19937 looked good to me, as Vladislav said since it's used in the other places too.
Actually the simulation and AI seed use boost::rand48.
But still better than using custom magic numbers?
The only possible performance problem was reinitializing the seed often right?
Wouldn't it be sufficient initialize the seed upon component construction once? Or upon first soundplay?
(Though the magic numbers aren't so bad, only one line, or four including the wrapping.)
(Especially if it's some orders of magnitude faster. But perhaps there are other std or boost RNG that are also orders of magnitude faster.)

Apr 25 2019, 12:09 PM
wraitii added inline comments to D1754: Pass an argument to GUI events / SendEventToAll, remove loading screen progess workaround.
Apr 25 2019, 11:58 AM
elexis added a comment to D1754: Pass an argument to GUI events / SendEventToAll, remove loading screen progess workaround.

Thanks for the review!

Apr 25 2019, 11:51 AM
Stan added a comment to D1846: Adds a "properties"-property to resources and let mods be able to prevent resources from being bartered, traded and/or tributed..

https://github.com/0ADMods/A-Mythology-mod-f-or-0-A.D.

Apr 25 2019, 11:32 AM
Imarok requested changes to D1842: Set ship status bars in parent.

Seems like that won't work cause of template_unit_ship_fishing.xml

Apr 25 2019, 11:32 AM
lyv added a comment to D1846: Adds a "properties"-property to resources and let mods be able to prevent resources from being bartered, traded and/or tributed..

(A certain other project also has this feature implemented. Mostly irrelevant, but if this is not committed before the first release, it might be of some interest)

Apr 25 2019, 11:31 AM
elexis retitled D1841: efficient -> efficiently in rP22203 from correct rP22203 to efficient -> efficiently in rP22203.
Apr 25 2019, 11:24 AM
elexis added a comment to D1846: Adds a "properties"-property to resources and let mods be able to prevent resources from being bartered, traded and/or tributed..

Might want to link the other code for comparison.

Apr 25 2019, 11:24 AM
Freagarach added a comment to D1846: Adds a "properties"-property to resources and let mods be able to prevent resources from being bartered, traded and/or tributed..

I'm already checking out the code there, it is quite similar but has a few things I didn't think of.

Apr 25 2019, 10:31 AM
wraitii added a comment to rP22225: Fix D1491 which introduced an ENSURE that should not have been there..

It is indeed... We don't use the jpc at the moment so this doesn't crash, but it should be fixed.

Apr 25 2019, 10:15 AM
Stan updated subscribers of D1846: Adds a "properties"-property to resources and let mods be able to prevent resources from being bartered, traded and/or tributed..

@Angen I know you have little time but do you have some suggestions ? :)

Apr 25 2019, 9:47 AM
Itms raised a concern with rP22225: Fix D1491 which introduced an ENSURE that should not have been there..

This is removing state.jpc = m_JumpPointCache[passClass].get(); when it != m_JumpPointCache.end(), no?

Apr 25 2019, 9:15 AM
wraitii added a comment to D1491: Const-Correct the long range pathfinder.

Yup, that ENSURE was broken... Mind you my initial code was _also_ broken since at() would have failed here too. Apparently I didn't playtest that revision and we have no tests calling ComputeJPSPath, which we really ought to (might add some this WE).

Apr 25 2019, 9:09 AM
wraitii committed rP22225: Fix D1491 which introduced an ENSURE that should not have been there..
Fix D1491 which introduced an ENSURE that should not have been there.
Apr 25 2019, 9:08 AM
wraitii added a comment to D1491: Const-Correct the long range pathfinder.
Apr 25 2019, 9:00 AM
wowgetoffyourcellphone added a comment to D1491: Const-Correct the long range pathfinder.

Not sure if related, but I got this in the game uploading loading a map:

Apr 25 2019, 8:58 AM
Stan added a comment to D1847: Optimise out of frustum rendering of textures overlays.

What's the perf gain like ?

Apr 25 2019, 7:57 AM
autobuild committed rP22224: [Windows] Automated build..
[Windows] Automated build.
Apr 25 2019, 7:23 AM
wackyserious committed rP22223: New Texture: Celtic Druid.
New Texture: Celtic Druid
Apr 25 2019, 6:55 AM
wackyserious committed rP22222: Retexture: Illyrian helmet (Bronze and Gold variants).
Retexture: Illyrian helmet (Bronze and Gold variants)
Apr 25 2019, 5:38 AM
wackyserious committed rP22221: New Texture: Elite Greek Hoplite (Bronze cuirass, Scale and Lamellar).
New Texture: Elite Greek Hoplite (Bronze cuirass, Scale and Lamellar)
Apr 25 2019, 4:02 AM
Vulcan added a comment to D1847: Optimise out of frustum rendering of textures overlays.

Successful build - Chance fights ever on the side of the prudent.

Apr 25 2019, 1:57 AM
Vulcan added a comment to D1847: Optimise out of frustum rendering of textures overlays.

Successful build - Chance fights ever on the side of the prudent.

Apr 25 2019, 1:53 AM
vladislavbelov updated the diff for D1847: Optimise out of frustum rendering of textures overlays.
Apr 25 2019, 1:50 AM
vladislavbelov created D1847: Optimise out of frustum rendering of textures overlays.
Apr 25 2019, 1:48 AM
Vulcan added a comment to D1584: Seed random sounds.

Successful build - Chance fights ever on the side of the prudent.

Apr 25 2019, 12:24 AM
Stan updated the diff for D1584: Seed random sounds.

Make FastRand return an u32

Apr 25 2019, 12:22 AM
Vulcan added a comment to D1461: Hint the location for "You have been attacked" notifications.

Successful build - Chance fights ever on the side of the prudent.

Apr 25 2019, 12:14 AM
Imarok updated the diff for D1461: Hint the location for "You have been attacked" notifications.

Do changes according to inlines

Apr 25 2019, 12:11 AM
Imarok added a comment to D1461: Hint the location for "You have been attacked" notifications.
In D1461#73338, @elexis wrote:

Good.

Now the question remains how we make it visible to the players that this click-feature exists.
Usually the mouse cursor changes when one hovers a link. Underlining won't be supported by our small font base, but adding a sprite should be easily possible (resource icons in tooltips.js / setup_resources.xml as an example).

Apr 25 2019, 12:11 AM
vladislavbelov added inline comments to D1584: Seed random sounds.
Apr 25 2019, 12:04 AM
vladislavbelov added inline comments to D1584: Seed random sounds.
Apr 25 2019, 12:03 AM

Apr 24 2019

Itms added inline comments to D1754: Pass an argument to GUI events / SendEventToAll, remove loading screen progess workaround.
Apr 24 2019, 10:49 PM
Stan added inline comments to D1837: Input.js refactoring - separate diplomatic tribute in their own file..
Apr 24 2019, 10:44 PM
Vulcan added a comment to D1584: Seed random sounds.

Successful build - Chance fights ever on the side of the prudent.

Apr 24 2019, 10:43 PM
Imarok added a comment to D1786: Refactor input.js.

Didn't look closely at it, but we definitely shouldn't copy the general fsm definition, but instead put it somewhere where UnitAI and input.js can use it.

Apr 24 2019, 10:43 PM
Stan updated the diff for D1584: Seed random sounds.

Use 0xFFF instead 0x7FF
replace unsigned int by u32
replace uint8_t by u8

Apr 24 2019, 10:37 PM
Imarok added a comment to D1837: Input.js refactoring - separate diplomatic tribute in their own file..

not an expert then again.

Apr 24 2019, 10:32 PM
Itms added a comment to D1754: Pass an argument to GUI events / SendEventToAll, remove loading screen progess workaround.

This looks very good to me, and it works both on SM38 and SM45.

Apr 24 2019, 10:32 PM
wraitii added inline comments to D1584: Seed random sounds.
Apr 24 2019, 9:53 PM
vladislavbelov added inline comments to D1584: Seed random sounds.
Apr 24 2019, 9:46 PM
Stan added inline comments to D1584: Seed random sounds.
Apr 24 2019, 9:36 PM
Stan accepted rP22203: Communicate field diminishing returns to the player in the fields tooltip.
Apr 24 2019, 9:26 PM
lyv added a comment to D1846: Adds a "properties"-property to resources and let mods be able to prevent resources from being bartered, traded and/or tributed..

Angen also have a patch doing this. See the mythology mod.

Apr 24 2019, 9:20 PM
bb requested verification of rP22203: Communicate field diminishing returns to the player in the fields tooltip.
Apr 24 2019, 9:17 PM
bb committed rP22220: Correct the grammer of rP22203.
Correct the grammer of rP22203
Apr 24 2019, 9:16 PM
bb closed D1841: efficient -> efficiently in rP22203.
Apr 24 2019, 9:16 PM
bb accepted D1841: efficient -> efficiently in rP22203.
Apr 24 2019, 9:16 PM
lyv added inline comments to rP22219: Const-Correct the long range pathfinder.
Apr 24 2019, 9:15 PM