Page MenuHomeWildfire Games
Feed All Stories

Today

wraitii committed rP25435: Fix rP25434.
Fix rP25434
Fri, May 14, 1:12 PM
wraitii requested review of D3962: Don't include ComponentManager.h in ScriptComponents.
Fri, May 14, 1:04 PM
Angen added a comment to D3959: Add some translation.

Translation of label and description will not have effect on other mods. I dont think it is worth the call to translate.

Fri, May 14, 12:58 PM
Harbormaster failed to build B17221: rP25434: Split off JSON-related function from ScriptInterface, clean up headers. for rP25434: Split off JSON-related function from ScriptInterface, clean up headers.!
Fri, May 14, 12:34 PM
wraitii committed rP25434: Split off JSON-related function from ScriptInterface, clean up headers..
Split off JSON-related function from ScriptInterface, clean up headers.
Fri, May 14, 12:19 PM
wraitii closed D3961: Split off JSON-related function from ScriptInterface, clean up headers..
Fri, May 14, 12:18 PM
wraitii added inline comments to D3961: Split off JSON-related function from ScriptInterface, clean up headers..
Fri, May 14, 12:15 PM
wraitii requested review of D3961: Split off JSON-related function from ScriptInterface, clean up headers..
Fri, May 14, 11:43 AM
wraitii committed rP25433: Remove unused m_rng member from ScriptInterface..
Remove unused m_rng member from ScriptInterface.
Fri, May 14, 11:27 AM
wraitii closed D3960: Remove unused m_rng member from ScriptInterface..
Fri, May 14, 11:27 AM
wraitii requested review of D3960: Remove unused m_rng member from ScriptInterface..
Fri, May 14, 11:02 AM
wraitii added inline comments to rP14877: Second (main) commit for the SpiderMonkey upgrade..
Fri, May 14, 10:34 AM
wraitii updated the summary of D3216: Speed up GetAvailableMods for archives..
Fri, May 14, 10:27 AM
wraitii added a comment to D3216: Speed up GetAvailableMods for archives..
In D3216#170616, @Stan wrote:

Just creating a folder and dumping the zip in it. That's in most instructions other than the pyromod way.

I'm still confused :P . This worked before and it will still work. It's just sub-optimal at the moment.
Incidentally I think I probably should write the JSON when it's encountered in the archive but not the mod, to fix the inconsistency.

Fri, May 14, 10:15 AM
autobuild committed rP25432: [i18n] Updated POT and PO files..
[i18n] Updated POT and PO files.
Fri, May 14, 9:23 AM
wowgetoffyourcellphone added a comment to D3930: [Gameplay] Make Athenian Marines and Merc Archers accessible from the Town Phase and in the Barrack.

If you move the Athenian Marine to the barracks or gymnasium, then I suggest you rename the unit back to its original name from years ago: Athenian Ekdromos. Full circle.

Fri, May 14, 8:54 AM
autobuild committed rP25431: [Windows] Automated build..
[Windows] Automated build.
Fri, May 14, 5:53 AM
nwtour requested review of D3959: Add some translation.
Fri, May 14, 2:53 AM
nwtour retitled D3936: Human readable strings in colorizeHotkey/Hotkey picker from Hide long strings in colorizeHotkey for extra hotkeys to Human readable strings in colorizeHotkey/Hotkey picker.
Fri, May 14, 2:50 AM
Vulcan added a comment to D3936: Human readable strings in colorizeHotkey/Hotkey picker.

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

Fri, May 14, 2:45 AM
Vulcan added a comment to D3936: Human readable strings in colorizeHotkey/Hotkey picker.

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

Fri, May 14, 2:42 AM
nwtour added a comment to D3936: Human readable strings in colorizeHotkey/Hotkey picker.

Fri, May 14, 2:28 AM
nwtour updated the diff for D3936: Human readable strings in colorizeHotkey/Hotkey picker.

Use getReadableHotkeyName() in HotKey Picker.
Fix lint warnings

Fri, May 14, 2:27 AM

Yesterday

Stan added a comment to D3216: Speed up GetAvailableMods for archives..
In D3216#170610, @Stan wrote:

We might want to add that to the A24 to A25 page for I remember seeing instructions that would not work

I'm not sure what you mean? That what would not work?

Thu, May 13, 9:14 PM
wraitii added a comment to D3216: Speed up GetAvailableMods for archives..
In D3216#170610, @Stan wrote:

We might want to add that to the A24 to A25 page for I remember seeing instructions that would not work

Thu, May 13, 8:52 PM
Stan added a comment to D3216: Speed up GetAvailableMods for archives..

We might want to add that to the A24 to A25 page for I remember seeing instructions that would not work

Thu, May 13, 7:55 PM
Vulcan added a comment to D3216: Speed up GetAvailableMods for archives..

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

Thu, May 13, 7:32 PM
Vulcan added a comment to D3216: Speed up GetAvailableMods for archives..

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

Thu, May 13, 7:28 PM
wraitii committed rP25430: Split off Object-related functions from ScriptInterface.
Split off Object-related functions from ScriptInterface
Thu, May 13, 7:25 PM
wraitii closed D3956: Split off Object-related functions from ScriptInterface.
Thu, May 13, 7:24 PM
wraitii updated the diff for D3216: Speed up GetAvailableMods for archives..

Update. This is retro-compatible by opening the archive if the mod.json cannot be found, which makes more sense.

Thu, May 13, 7:12 PM
Stan added a comment to rP25429: Add a decal to the berry bushes to make them more visible and make the fruits….

It's fine, it's a bug in blender.

Thu, May 13, 7:05 PM
Vulcan added a comment to D3956: Split off Object-related functions from ScriptInterface.

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

Thu, May 13, 6:57 PM
Vulcan added a comment to D3956: Split off Object-related functions from ScriptInterface.

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

Thu, May 13, 6:56 PM
asterix added inline comments to rP25429: Add a decal to the berry bushes to make them more visible and make the fruits….
Thu, May 13, 6:48 PM
wraitii updated the diff for D3956: Split off Object-related functions from ScriptInterface.

Also rebase the tests :p

Thu, May 13, 6:39 PM
Vulcan added a comment to D3956: Split off Object-related functions from ScriptInterface.

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

Thu, May 13, 6:31 PM
Vulcan added a comment to D3956: Split off Object-related functions from ScriptInterface.

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

Thu, May 13, 6:24 PM
Vulcan added a comment to D3956: Split off Object-related functions from ScriptInterface.

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

Thu, May 13, 6:23 PM
wraitii updated the diff for D3956: Split off Object-related functions from ScriptInterface.

Rebased properly.

Thu, May 13, 6:17 PM
wraitii planned changes to D3952: IPV6-readiness diff: update StunClient to be mostly IPV6-ready.

K I'm actually changing my mind : I think it'll be easier to maintain an IPV6-compatible fork of Enet and reuse it everywhere than maintain our own socket compatibility layer (the job is mostly the same anyways).

Thu, May 13, 6:14 PM
Langbart added a comment to D3957: Make fruit (berries) obstruction smaller.

TEST PLAN
Test it

Thu, May 13, 5:56 PM
Langbart abandoned D3768: Quit confirmation dialog hotkey and button.

Another user will try to implement it, I will not work on this one anymore.
see D3958 - Add an extra button for seeing the summary page when quitting a session (13/May/21).

Thu, May 13, 5:42 PM
Schweini requested review of D3958: Add an extra button for seeing the summary page when quitting a session.
Thu, May 13, 5:28 PM
nani added a comment to D3957: Make fruit (berries) obstruction smaller.

Is intended for those who don't overite the parent setting, which means is intended.

Thu, May 13, 5:16 PM
Stan added a comment to D3957: Make fruit (berries) obstruction smaller.

This also affects all trees in simulation/templates/gaia/fruit. Is intended?
Refs: rP25259

Thu, May 13, 5:04 PM
nani requested review of D3957: Make fruit (berries) obstruction smaller.
Thu, May 13, 4:59 PM
Stan added inline comments to D3952: IPV6-readiness diff: update StunClient to be mostly IPV6-ready.
Thu, May 13, 4:48 PM
Angen added inline comments to D2150: [PetraAI] - Improve support for classes..
Thu, May 13, 3:17 PM
wraitii requested review of D3956: Split off Object-related functions from ScriptInterface.
Thu, May 13, 2:11 PM
Vulcan added a comment to D3952: IPV6-readiness diff: update StunClient to be mostly IPV6-ready.

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

Thu, May 13, 2:08 PM
Vulcan added a comment to D3952: IPV6-readiness diff: update StunClient to be mostly IPV6-ready.

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

Thu, May 13, 1:57 PM
Vulcan added a comment to D3952: IPV6-readiness diff: update StunClient to be mostly IPV6-ready.

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

Thu, May 13, 1:57 PM
wraitii updated the diff for D3952: IPV6-readiness diff: update StunClient to be mostly IPV6-ready.

Aight, so as it turns out winsock2 is a bit of a pain.

Thu, May 13, 1:45 PM
Stan committed rP25429: Add a decal to the berry bushes to make them more visible and make the fruits….
Add a decal to the berry bushes to make them more visible and make the fruits…
Thu, May 13, 1:19 PM
Stan edited P243 Script to replace B&W background in transparent pixels on PNG textures.
Thu, May 13, 1:12 PM
wraitii committed rP25428: Split JS conversions from ScriptInterface..
Split JS conversions from ScriptInterface.
Thu, May 13, 11:43 AM
wraitii closed D3953: Split JS conversions from ScriptInterface..
Thu, May 13, 11:43 AM
wraitii requested review of D3955: Simplify & fix precompiled header logic..
Thu, May 13, 11:25 AM
autobuild committed rP25427: [Windows] Automated build..
[Windows] Automated build.
Thu, May 13, 10:44 AM
Vulcan added a comment to D3952: IPV6-readiness diff: update StunClient to be mostly IPV6-ready.

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

Thu, May 13, 10:28 AM
Vulcan added a comment to D3952: IPV6-readiness diff: update StunClient to be mostly IPV6-ready.

Build is unstable, some tests have failed - The Moirai have given mortals hearts that can endure.

Thu, May 13, 10:03 AM
Vulcan added a comment to D3952: IPV6-readiness diff: update StunClient to be mostly IPV6-ready.

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

Thu, May 13, 9:55 AM
Vulcan added a comment to D3953: Split JS conversions from ScriptInterface..

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

Thu, May 13, 9:51 AM
Vulcan added a comment to D3953: Split JS conversions from ScriptInterface..

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

Thu, May 13, 9:47 AM
wraitii added a comment to D3950: Prefer using const on immutable values..

I'm a firm "don't care" here, so do whatever you want from my end :)

Thu, May 13, 9:33 AM
wraitii updated the diff for D3952: IPV6-readiness diff: update StunClient to be mostly IPV6-ready.

Rebased.

Thu, May 13, 9:30 AM
wraitii updated the diff for D3953: Split JS conversions from ScriptInterface..

Rebased.

Thu, May 13, 9:29 AM
autobuild committed rP25426: [Windows] Automated build..
[Windows] Automated build.
Thu, May 13, 5:50 AM

Wed, May 12

Vulcan added a comment to D3954: Fix crash in Atlas..

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

Wed, May 12, 9:34 PM
wraitii committed rP25425: Fix crash in Atlas..
Fix crash in Atlas.
Wed, May 12, 9:25 PM
wraitii closed D3954: Fix crash in Atlas..
Wed, May 12, 9:25 PM
wraitii added a comment to D3886: Resistance paradigm change: Unspecified value = invulnerable, not 0 resistance..
In D3886#170491, @bb wrote:

Please redo your math. If you do 0.9^x you are making everything more easy, more performant and you get 0=undefined= immume, so no damage being dealt. Makes total sense to me. Just have to rename stuff and work out the gui.

Right, assuming '1' means '100% resistance'. But then you get the awkwardness that there is no '0' value, and adding 1 at lower armour levels will have a greater absolute effect than at lower effects, which is the nice property that led us to exponential armour in the first place.

Wed, May 12, 8:08 PM
bb added a comment to D3886: Resistance paradigm change: Unspecified value = invulnerable, not 0 resistance..

which is that undefined values can be assumed to mean invulnerable.

Then all falsy values should mean that, and build a paradigm around that. The 1/x proposal might be not ideal, but consider e^(-x) (or if you are looking at the code one might consider 0.9^x)

Look, a system in which a resistance of '0' means 'infinite' and '9999' means 'none' is dumb, there's no way around that.

Please redo your math. If you do 0.9^x you are making everything more easy, more performant and you get 0=undefined= immume, so no damage being dealt. Makes total sense to me. Just have to rename stuff and work out the gui.

Wed, May 12, 7:56 PM
chrstgtr added a comment to D3898: [Gameplay] - Make outposts useful again..

I noticed in A23 outposts actually had 80 vision by default, increasing to 120 with the pidgeon tech. It's D2933 that made it 100.

This gives them 100 base vision, and the pidgeon tech gives them 150. That's a _lot_ of range, but I agree that it should outrange units, and units have around 80 vision, so...

I didn't add a garrison-related tech because nothing works well with turrets. The only point of garrisoning is that you can attack.
I'm wondering if outposts shouldn't be made un-garrisonable and have an actor inside anyways now.

I'm wary that the new LOS values might be a bit too high. Perhaps unit should lose 10 vision, and outposts be 90-135 ?

Wed, May 12, 7:43 PM
Vulcan added a comment to D3953: Split JS conversions from ScriptInterface..

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

Wed, May 12, 7:25 PM
Vulcan added a comment to D3898: [Gameplay] - Make outposts useful again..

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

Wed, May 12, 7:23 PM
Vulcan added a comment to D3936: Human readable strings in colorizeHotkey/Hotkey picker.

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

Wed, May 12, 7:11 PM
wraitii added a comment to D3886: Resistance paradigm change: Unspecified value = invulnerable, not 0 resistance..

"undefined means unable to do x", seems like something we use throughout the sim. E.g. if you don't have a value for gathering wood you can't. If you don't have the wood resource supply, it cannot be gathered from you. In that respect undefined == immune sounds good.

Well, except that a gather rate of 0 also means "can't gather wood", so that can also be used to say that "0 resistance" should be infinite resistance.

Wed, May 12, 6:47 PM
Freagarach added a comment to D3886: Resistance paradigm change: Unspecified value = invulnerable, not 0 resistance..

"undefined means unable to do x", seems like something we use throughout the sim. E.g. if you don't have a value for gathering wood you can't. If you don't have the wood resource supply, it cannot be gathered from you. In that respect undefined == immune sounds good.

Wed, May 12, 6:41 PM
wraitii published D3953: Split JS conversions from ScriptInterface. for review.

Gonna have to rebase this tomorrow

Wed, May 12, 6:28 PM
Vulcan added a comment to D3898: [Gameplay] - Make outposts useful again..

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

Wed, May 12, 6:23 PM
Vulcan added a comment to D3936: Human readable strings in colorizeHotkey/Hotkey picker.

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

Wed, May 12, 6:19 PM
wraitii added a comment to D3886: Resistance paradigm change: Unspecified value = invulnerable, not 0 resistance..
In D3886#170469, @bb wrote:

Not all usecases will iterate this particular object. One might iterate an attackers attack object instead, and turn into trouble. In fact this is exactly what we do in current SVN.

Mh, you're right, I should change that actually.

Wed, May 12, 6:14 PM
bb added a comment to D3886: Resistance paradigm change: Unspecified value = invulnerable, not 0 resistance..
In D3886#170423, @bb wrote:

Having two falsy values (undefined and 0) meaning opposite things is highly unwanted and must be avoided: Checking for !resistanceType is likely to happen one day and is totally ill-defined.

Point taken, but I don't think what you're describing is quite right, because it's not 'undefined', it's _actually_ undefined, as in the key in the resistance object doesn't exist. Which means, crucially, that iterating the resistance will work as you'd expect out of the box instead of requiring special handling (as in D2044 for example).

Not all usecases will iterate this particular object. One might iterate an attackers attack object instead, and turn into trouble. In fact this is exactly what we do in current SVN,. So this revision is turning an iteration which does exactly what is being expected, into one which has likely breakage.
If you wish all damagetypes in each resistance instance (first consider performance), then fill up the missing entries from the damagtypes globalscript.

Further, and as you can see, this only changes a single file, so I think the risk is limited, and we have tests now, so the risk is further limited.

Not an argument. Anyone working with the code might come to a conclusion that we need to do some checks in another file. Also within one file the risk is huge.

Overall I'm not too concerned

Problematic.

Edit: also, you would lose the whole point of the diff

Well, that could be a conclusion.

Wed, May 12, 6:07 PM
wraitii updated the diff for D3898: [Gameplay] - Make outposts useful again..

I noticed in A23 outposts actually had 80 vision by default, increasing to 120 with the pidgeon tech. It's D2933 that made it 100.

Wed, May 12, 5:56 PM
wraitii awarded rP25424: Major improvements to the lobby bots XpartaMuPP and EcheLOn. a Party Time token.
Wed, May 12, 5:54 PM
user1 committed rP25424: Major improvements to the lobby bots XpartaMuPP and EcheLOn..
Major improvements to the lobby bots XpartaMuPP and EcheLOn.
Wed, May 12, 5:41 PM
user1 closed D2630: Major improvements to the lobby bots XpartaMuPP and EcheLOn..
Wed, May 12, 5:41 PM
Vulcan added a comment to D3952: IPV6-readiness diff: update StunClient to be mostly IPV6-ready.

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

Wed, May 12, 5:37 PM
nwtour updated the diff for D3936: Human readable strings in colorizeHotkey/Hotkey picker.

Unassigned hotkey -> Gray

Wed, May 12, 5:36 PM
Vulcan added a comment to D3952: IPV6-readiness diff: update StunClient to be mostly IPV6-ready.

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

Wed, May 12, 5:34 PM
wraitii updated the diff for D3952: IPV6-readiness diff: update StunClient to be mostly IPV6-ready.

Update. Should go green.

Wed, May 12, 5:34 PM
Stan added a comment to D3936: Human readable strings in colorizeHotkey/Hotkey picker.

How about grey for a color? After all it's disabled.

Wed, May 12, 5:10 PM
wraitii added inline comments to D3952: IPV6-readiness diff: update StunClient to be mostly IPV6-ready.
Wed, May 12, 4:47 PM
Stan added inline comments to D3952: IPV6-readiness diff: update StunClient to be mostly IPV6-ready.
Wed, May 12, 4:45 PM
wraitii added inline comments to D3952: IPV6-readiness diff: update StunClient to be mostly IPV6-ready.
Wed, May 12, 4:42 PM
wraitii added a comment to D3886: Resistance paradigm change: Unspecified value = invulnerable, not 0 resistance..
In D3886#170423, @bb wrote:

Having two falsy values (undefined and 0) meaning opposite things is highly unwanted and must be avoided: Checking for !resistanceType is likely to happen one day and is totally ill-defined.

Point taken, but I don't think what you're describing is quite right, because it's not 'undefined', it's _actually_ undefined, as in the key in the resistance object doesn't exist. Which means, crucially, that iterating the resistance will work as you'd expect out of the box instead of requiring special handling (as in D2044 for example).

Wed, May 12, 4:40 PM
Stan added inline comments to D3952: IPV6-readiness diff: update StunClient to be mostly IPV6-ready.
Wed, May 12, 4:10 PM
wraitii requested review of D3952: IPV6-readiness diff: update StunClient to be mostly IPV6-ready.
Wed, May 12, 3:53 PM