Page MenuHomeWildfire Games

Silier
vixen

User Details

User Since
Jul 28 2017, 4:19 PM (227 w, 1 d)

Recent Activity

Today

Silier added a comment to D4097: Gamesetup: make tooltips compatible with enabled & disabled settings..

Anything else except the dots and pointed change?

Sat, Dec 4, 5:53 PM
Silier added a comment to D4334: [PetraAI] - Use enum-like variables instead of magic values. [1/n].

Thanks for the patch

Sat, Dec 4, 10:02 AM

Yesterday

Silier added a comment to D4361: Removes AO multipliers and normalizes AO textures.

Can't the values be totally nuked if they will not be used at all?

Fri, Dec 3, 3:22 PM

Wed, Dec 1

Silier added a comment to D4334: [PetraAI] - Use enum-like variables instead of magic values. [1/n].

We don't need enums, thats correct.
But they help when changing strings/values to make change only on one place.

Wed, Dec 1, 5:56 PM
Silier added a comment to D4334: [PetraAI] - Use enum-like variables instead of magic values. [1/n].

Why do we need strict checking? == is totally sufficient here.

Wed, Dec 1, 5:47 PM

Mon, Nov 29

Silier awarded rP26023: Drops custom utf16 string implementation (from rP684), uses C++11 one. a Love token.
Mon, Nov 29, 1:16 PM
Silier added a comment to rP25123: Technically seperate Turrets from GarrisonHolder..

Also performance is not the question since number of messages for turreted is just subset of original number of messages from garrisoned.
Only additional cost are extra variables but amount of data is not affected.

Mon, Nov 29, 12:49 PM
Silier added a comment to rP25123: Technically seperate Turrets from GarrisonHolder..

Its not complete now.
If someone would want to use it, they would need to update proxy and other stuff in common api.

Mon, Nov 29, 12:31 PM

Sun, Nov 28

Silier added a comment to rP25123: Technically seperate Turrets from GarrisonHolder..

Why to give option to occupy turret then to api?

Sun, Nov 28, 9:12 AM
Silier added a comment to rP25123: Technically seperate Turrets from GarrisonHolder..

How no?
Ai gets information through proxy and messages.
If she doesn't get memo that unit is in turret state, she will act like it would not be.

Sun, Nov 28, 8:19 AM
Silier raised a concern with rP25123: Technically seperate Turrets from GarrisonHolder..
Sun, Nov 28, 8:02 AM
Silier raised a concern with rP26000: Split tasks from ProductionQueue..

See comments on diff

Sun, Nov 28, 7:54 AM

Sat, Nov 27

Silier added inline comments to D4356: Check for squaredRange being negative.
Sat, Nov 27, 7:43 PM
Silier published D4356: Check for squaredRange being negative for review.
Sat, Nov 27, 7:20 PM
Silier added inline comments to D4334: [PetraAI] - Use enum-like variables instead of magic values. [1/n].
Sat, Nov 27, 3:54 PM
Silier added a comment to rP25123: Technically seperate Turrets from GarrisonHolder..

@Freagarach GarrisonManager and AIProxy?

Sat, Nov 27, 3:00 PM
Silier updated the diff for D4318: Fix timeout confirmation box.

messageObject fix

Sat, Nov 27, 12:54 PM
Silier updated the diff for D4318: Fix timeout confirmation box.

fix translate

Sat, Nov 27, 12:51 PM
Silier updated the diff for D4318: Fix timeout confirmation box.

rewrite to class, edit displayed message

Sat, Nov 27, 12:50 PM
Silier added inline comments to D4224: Total idle time.
Sat, Nov 27, 10:50 AM
Silier added a comment to D4224: Total idle time.

thank you for working on this.
Good job so far.

Sat, Nov 27, 10:46 AM
Silier added a comment to rP26017: Draws fancy water and its shore waves only for GLSL shaders..

Can you please elaborate why to draw it only with glsl?

Sat, Nov 27, 8:04 AM
Silier awarded D4354: Adds more flexible dependencies to options a Like token.
Sat, Nov 27, 7:48 AM

Fri, Nov 26

Silier awarded rP26016: Fix Researchers' GUI without a tech cost multiplier. a Like token.
Fri, Nov 26, 10:11 PM
Silier added inline comments to D4349: Fix infinite loop when queuing a gather order after garrison..
Fri, Nov 26, 6:52 PM
Silier added inline comments to D4349: Fix infinite loop when queuing a gather order after garrison..
Fri, Nov 26, 6:48 PM
Silier added inline comments to D4349: Fix infinite loop when queuing a gather order after garrison..
Fri, Nov 26, 6:47 PM
Silier updated Silier.
Fri, Nov 26, 5:52 PM

Wed, Nov 24

Silier added inline comments to D4334: [PetraAI] - Use enum-like variables instead of magic values. [1/n].
Wed, Nov 24, 10:08 AM
Silier added inline comments to D2016: Rename "ElevationBonus" and "Delay" to "Origin" and "EffectDelay", respectively..
Wed, Nov 24, 10:04 AM
Silier added a comment to D4319: New game mode: capture the wonder.

Can't comment on patch itself but...

Wed, Nov 24, 9:58 AM

Sun, Nov 21

Silier updated the summary of D4349: Fix infinite loop when queuing a gather order after garrison..
Sun, Nov 21, 11:29 AM

Thu, Nov 18

Silier added inline comments to D4352: Some fixes after r26000..
Thu, Nov 18, 8:41 PM

Wed, Nov 17

Silier added inline comments to rP26000: Split tasks from ProductionQueue..
Wed, Nov 17, 4:01 PM
Silier added inline comments to rP26000: Split tasks from ProductionQueue..
Wed, Nov 17, 3:58 PM

Sun, Nov 14

Silier updated the Trac tickets for D4350: Fixes border pixels in rP25993.
Sun, Nov 14, 11:42 AM
Silier added a comment to D4333: Split tasks from ProductionQueue..

That is sus (playcode)

const ret = [];
Sun, Nov 14, 9:32 AM

Sat, Nov 13

Silier removed 1 auditor(s) for rP21121: Add a repair time tooltip: mimo.
Sat, Nov 13, 3:24 PM
Silier accepted D4305: Fix bug where scenario map was not loading as revealed when set to revealed..

@Freagarach can you also have a look if something is not overlooked just in case

Sat, Nov 13, 1:31 PM
Silier added a comment to D4330: Show the right message when exiting the game as a client.

Thank you for the patch

Sat, Nov 13, 1:27 PM
Silier committed rP25995: Show the correct message when exiting the multiplayer match as a client.
Show the correct message when exiting the multiplayer match as a client
Sat, Nov 13, 1:27 PM
Silier closed D4330: Show the right message when exiting the game as a client.
Sat, Nov 13, 1:27 PM
Silier removed reviewers for D2620: use non-breaking space in 0 A.D.: Silier, Restricted Owners Package.
Sat, Nov 13, 1:20 PM
Silier requested changes to D4334: [PetraAI] - Use enum-like variables instead of magic values. [1/n].
Sat, Nov 13, 12:10 PM
Silier added a comment to D4333: Split tasks from ProductionQueue..

some comments

Sat, Nov 13, 11:36 AM
Silier added inline comments to D4334: [PetraAI] - Use enum-like variables instead of magic values. [1/n].
Sat, Nov 13, 10:48 AM

Fri, Nov 12

Silier requested changes to D4343: [PetraAI] - Use enum-like variables instead of magic values. [2/n].

please do not replace undefined

Fri, Nov 12, 9:31 PM
Silier requested changes to D4334: [PetraAI] - Use enum-like variables instead of magic values. [1/n].
Fri, Nov 12, 9:24 PM

Mon, Nov 8

Silier added a reviewer for D4338: Fix passibilty overlay in atlas: Restricted Owners Package.
Mon, Nov 8, 6:28 AM

Sun, Nov 7

Silier added a comment to D4330: Show the right message when exiting the game as a client.

Forgot explaination:
You had it inside enabled function, which was returning true or false depending if dialog is enabled, so you assigned it to variable each time, this function would be called.

Sun, Nov 7, 10:03 PM
Silier updated Silier.
Sun, Nov 7, 2:01 PM
phabadmin renamed Silier from Angen to Silier.
Sun, Nov 7, 1:58 PM
Silier removed reviewers for D4330: Show the right message when exiting the game as a client: bb, Freagarach.
Sun, Nov 7, 8:51 AM

Sat, Nov 6

Silier accepted D4330: Show the right message when exiting the game as a client.
Sat, Nov 6, 3:36 PM
Silier removed 1 auditor(s) for rP21984: Corinthian Helmet Update:: elexis.

irrelevant now

Sat, Nov 6, 1:59 PM
Silier added inline comments to D4330: Show the right message when exiting the game as a client.
Sat, Nov 6, 1:52 PM
Silier requested changes to D4330: Show the right message when exiting the game as a client.
Sat, Nov 6, 1:51 PM
Silier added a comment to D4334: [PetraAI] - Use enum-like variables instead of magic values. [1/n].

Hi,
thank you for picking this up.

Sat, Nov 6, 1:07 PM
Silier committed rP25989: Fix rP25374 and rP25413.
Fix rP25374 and rP25413
Sat, Nov 6, 12:10 PM
Silier closed D4335: Fix rP25374 and rP25413.
Sat, Nov 6, 12:09 PM

Fri, Nov 5

Silier updated the diff for D4335: Fix rP25374 and rP25413.

trying to fix my stupidity

Fri, Nov 5, 9:06 PM
Silier updated the diff for D4335: Fix rP25374 and rP25413.
Fri, Nov 5, 8:52 PM
Silier published D4335: Fix rP25374 and rP25413 for review.
Fri, Nov 5, 8:02 PM
Silier updated Silier.
Fri, Nov 5, 6:26 PM

Oct 31 2021

Silier added a comment to D4328: Delete Savanna biome loading tip.

There are no tips for any other biomes

Not argument to remove existing ones, if something is missing, finish/make it.

Oct 31 2021, 3:16 PM

Oct 30 2021

Silier requested verification of rP25410: Do not allow to start pyrogenesis with incompatible mods.
Oct 30 2021, 12:45 PM
Silier added inline comments to D2016: Rename "ElevationBonus" and "Delay" to "Origin" and "EffectDelay", respectively..
Oct 30 2021, 9:59 AM
Silier accepted D4322: Required experience points for promotion.
Oct 30 2021, 9:52 AM
Silier resigned from D4323: Display a tooltip for the health and capture points and fix the civ tooltip style..
Oct 30 2021, 9:50 AM

Oct 29 2021

Silier added inline comments to D4305: Fix bug where scenario map was not loading as revealed when set to revealed..
Oct 29 2021, 8:43 PM

Oct 28 2021

Silier removed 1 auditor(s) for rP25546: Mod data parsing rework: Silier.
Oct 28 2021, 10:35 PM
Silier resigned from rP25546: Mod data parsing rework.
Oct 28 2021, 10:35 PM
Silier committed rP25979: Fix pathname not matching name incompatible mod detection.
Fix pathname not matching name incompatible mod detection
Oct 28 2021, 10:34 PM
Silier closed D4324: Fix pathname not matching name incompatible mod detection.
Oct 28 2021, 10:34 PM
Silier added inline comments to D4323: Display a tooltip for the health and capture points and fix the civ tooltip style..
Oct 28 2021, 10:22 PM
Silier added a comment to D4323: Display a tooltip for the health and capture points and fix the civ tooltip style..
Oct 28 2021, 10:11 PM
Silier published D4324: Fix pathname not matching name incompatible mod detection for review.
Oct 28 2021, 9:32 PM
Silier raised a concern with rP25546: Mod data parsing rework.
Oct 28 2021, 9:32 PM
Silier added a comment to D4323: Display a tooltip for the health and capture points and fix the civ tooltip style..

Nice catch.
There are also more tooltips with "wrong" style if you are interested in fixing them, but thats for another patch.

Oct 28 2021, 7:45 PM

Oct 27 2021

Silier added a comment to D4317: [atlas] Limit the mercenary experience bar for the basic rank.

Thank you for the patch 🐺

Oct 27 2021, 9:05 PM
Silier committed rP25974: Prevent division by 0 for experience bar.
Prevent division by 0 for experience bar
Oct 27 2021, 9:00 PM
Silier closed D4317: [atlas] Limit the mercenary experience bar for the basic rank.
Oct 27 2021, 9:00 PM
Silier added a comment to rP25410: Do not allow to start pyrogenesis with incompatible mods.

ping @wraitii

Oct 27 2021, 8:51 PM
Silier added a comment to D4322: Required experience points for promotion.

Check with code for promotion, if no rounding is done there, then ceil instead floor else you end up with 150/150 and not promoting what looks worse

Oct 27 2021, 11:50 AM

Oct 25 2021

Silier planned changes to D4318: Fix timeout confirmation box.

Update message

Oct 25 2021, 3:18 PM
Silier added inline comments to D4318: Fix timeout confirmation box.
Oct 25 2021, 2:08 PM
Silier added inline comments to D4318: Fix timeout confirmation box.
Oct 25 2021, 2:06 PM
Silier added a reviewer for D1729: Missing tileclass in archipelago / fix fish on land: Restricted Owners Package.
Oct 25 2021, 12:17 PM
Silier added inline comments to D4305: Fix bug where scenario map was not loading as revealed when set to revealed..
Oct 25 2021, 11:22 AM
Silier added a comment to D4305: Fix bug where scenario map was not loading as revealed when set to revealed..

All user chosen settings are persisted when changing the selected map or maptype, except where the selected map overwrites the setting value and except for Scenario maps which still use the default value where the map doesn't specify the setting value.

Oct 25 2021, 11:14 AM
Silier added a comment to D4305: Fix bug where scenario map was not loading as revealed when set to revealed..
In D4305#183890, @Angen wrote:

There is still issue.
If map does not define attribute and you switch maps, the attribute from previous map will carry over so you will end up with 2 and possibly more settings for the same scenario map depending from which one you switch.

As far as I can tell, this bug exists in A25, but I've fixed it in this patch :).

Oct 25 2021, 10:41 AM
Silier added a comment to D4305: Fix bug where scenario map was not loading as revealed when set to revealed..

The original logic done by elexis was as follows as far as i remember:
If map defines some property, that property should be set.
If map does not define property, preciously set property by player or one from persistent match settings should be used.
Current issue with scenario maps is, even if property is missing from the map, value cannot be changed by player.
Notice we have this issue only with scenarios because they are locked by default.
Other maps should not get values reset.
This probably happens with other settings, will need to check that.

Oct 25 2021, 10:39 AM

Oct 24 2021

Silier requested changes to D4305: Fix bug where scenario map was not loading as revealed when set to revealed..

There is still issue.
If map does not define attribute and you switch maps, the attribute from previous map will carry over so you will end up with 2 and possibly more settings for the same scenario map depending from which one you switch.

Oct 24 2021, 4:42 PM

Oct 23 2021

Silier added a comment to D4256: Adjusting the placeholder text for inputs.

Or generated placeholderText could get valign parameter and adjust position internally.
If someone would want to adjust such a position on another places.

Oct 23 2021, 7:04 PM
Silier added a comment to D4297: Fixing a problem where the AI attacked bridges.

@Freagarach do you plan to commit this as it is right now?

Oct 23 2021, 6:01 PM
Silier added inline comments to D4305: Fix bug where scenario map was not loading as revealed when set to revealed..
Oct 23 2021, 6:00 PM
Silier published D4318: Fix timeout confirmation box for review.
Oct 23 2021, 12:42 PM
Silier added a comment to rP25966: There have been quite a bit of number of questions how to change scale of the….

hmm, yes, but thats again /mod dependent on /public
probably would need to move it to /mod
also, need a way to get remaining time

Oct 23 2021, 12:02 PM
Silier accepted D4317: [atlas] Limit the mercenary experience bar for the basic rank.

Ok, i get it now

Oct 23 2021, 9:19 AM
Silier added a comment to D4317: [atlas] Limit the mercenary experience bar for the basic rank.

Wait, didn't you say they are promoted to next rank?

Oct 23 2021, 9:14 AM