Page MenuHomeWildfire Games

Freagarach
User

Projects

User Details

User Since
Apr 5 2019, 7:29 PM (133 w, 2 d)

Recent Activity

Today

Freagarach added a comment to D4297: Fixing a problem where the AI attacked bridges.

Yes, as soon as I find the time. (But feel free, @bb or @Angen to do so.)

Mon, Oct 25, 9:13 AM
Freagarach accepted D4314: Double click tutorial bug.

Still good to go.

Mon, Oct 25, 9:10 AM
Freagarach added a comment to D4304: [Gameplay] - [WIP] Spartan spikes.

A very short range, I guess.

Mon, Oct 25, 9:08 AM

Fri, Oct 22

Freagarach accepted D3958: Add an extra button for seeing the summary page when quitting a session.

This code looks good, works as advertised and is a nice addition (I will use the option to skip the summary extensively whilst testing).

Fri, Oct 22, 12:59 PM
Freagarach added a comment to D4316: [Reference Suite] Show mercenaries' correct rank.

I'm not saying this is bad, but I guess changing the templates to train advanced rank mercenaries sounds like a good idea as well?

Fri, Oct 22, 7:46 AM
Freagarach added a comment to D4296: Layout changes to the replay menu.

(Even better.)

Fri, Oct 22, 7:39 AM
Freagarach accepted D4314: Double click tutorial bug.

Reads correct.

Fri, Oct 22, 7:36 AM
Freagarach requested review of D4295: Ignore formation selection when clicking the icon..

Due to popular request, I abandon the search for a situation which is more to my liking. ;)

Fri, Oct 22, 7:30 AM

Thu, Oct 21

Freagarach added a comment to D4295: Ignore formation selection when clicking the icon..

(With 'current behaviour' I meant 'click the icon selects the formation'.)

Thu, Oct 21, 8:11 PM
Freagarach requested changes to D3958: Add an extra button for seeing the summary page when quitting a session.

One may expect to go back to the replay list when quiting a replay.

Thu, Oct 21, 7:58 PM
Freagarach accepted D4296: Layout changes to the replay menu.

Code looks good.
Works as advertised.
This looks like an improvement to me. (Although I'm not quite sure about the orange.)

Thu, Oct 21, 7:52 PM
Freagarach added a comment to D4295: Ignore formation selection when clicking the icon..

So you say: ,,Screw the current behaviour.''?

Thu, Oct 21, 7:01 PM
Freagarach added inline comments to D4314: Double click tutorial bug.
Thu, Oct 21, 6:58 PM
Freagarach added inline comments to D4313: Add a linter for project name misspellings.
Thu, Oct 21, 6:51 PM

Tue, Oct 19

Freagarach added a comment to D3958: Add an extra button for seeing the summary page when quitting a session.

I'll take another look when I'm less tired, but this seems to be good to go.

Tue, Oct 19, 8:57 PM
Freagarach added a comment to rP25966: There have been quite a bit of number of questions how to change scale of the….
[20:04:32] <elexis> to whom it may concern, ./common/timer.js                                
(,,,)
[20:25:11] <elexis> for Angens concern
[20:25:21] <elexis> in the options timeout
[20:26:10] <elexis> also if one wanted to one could use the size property, which is a JS object, modify the width/height/post of that object and push that object back, this way one doesnt hardcode the 50%
Tue, Oct 19, 8:29 PM
Freagarach updated the diff for D4308: Fix pop indications of paused items..

Fix all hands.

Tue, Oct 19, 7:08 PM
Freagarach added inline comments to rP25779: Improvement to autoqueue usability.
Tue, Oct 19, 6:51 PM

Mon, Oct 18

Freagarach retitled D2016: Rename "ElevationBonus" and "Delay" to "Origin" and "EffectDelay", respectively. from Rename "ElevationBonus" and "Delay" to "AttackHeight" and "EffectDelay", respectively. to Rename "ElevationBonus" and "Delay" to "Origin" and "EffectDelay", respectively..
Mon, Oct 18, 8:01 PM
Freagarach updated the diff for D2016: Rename "ElevationBonus" and "Delay" to "Origin" and "EffectDelay", respectively..

Rebased, use origin.

Mon, Oct 18, 8:01 PM
Freagarach added inline comments to D4313: Add a linter for project name misspellings.
Mon, Oct 18, 6:31 PM
Freagarach added a comment to D368: Gameplay Scripting: Entity and Actor coding for Secondary Attacks.

During our game I noticed that chaps were leaving their melee target to go and attack some random dude who was firing at range at them (causing them to get picked off) have you noticed that/managed to fix that? (Else, consider this as a bug report. ;) )

Mon, Oct 18, 8:33 AM
Freagarach added inline comments to D4291: [gameplay] camouflaged ambush camp.
Mon, Oct 18, 8:29 AM

Sun, Oct 17

Freagarach added a comment to D4291: [gameplay] camouflaged ambush camp.

I'm really uncertain whether this is something to add to the main game. Certainly the idea is nice, but what technical debt do we add by including it? (It doesn't seem to be much so maybe just worth a shot.)

Sun, Oct 17, 8:30 AM
Freagarach added a comment to D3930: [Gameplay] Make Athenian Marines and Merc Archers accessible from the Town Phase and in the Barrack.

I agree with @wowgetoffyourcellphone about the naming. Also, why is the tech moved to the CC? Why don't we tie it to the hero?

Sun, Oct 17, 8:18 AM
Freagarach added a comment to D3958: Add an extra button for seeing the summary page when quitting a session.

You'll have to use translate with a comment above that explains the word used. See my inline for an example.

Sun, Oct 17, 8:13 AM
Freagarach added a comment to D4308: Fix pop indications of paused items..

The auto-produce icons have all a hand on it.

Aye, they sometimes also have a partly progressed progress bar. Probably something to do with not (re)setting those for ghosts. I will look at that later.

Sun, Oct 17, 8:09 AM
Freagarach added a comment to D4292: Moving the mininmal resolution to 1280x800.

We need horizontal scrolling. ;)

Sun, Oct 17, 8:05 AM
Freagarach added a comment to D4305: Fix bug where scenario map was not loading as revealed when set to revealed..

In my opinion, the bug that this patch was intended to address was fixed, and therefore should be merged, and a separate ticket should be created for adding more features such as those mentioned above. If a lot of extra code were added to this patch, it would take a while longer for the bugfix to get merged while the new code is written, tested, and debugged. Also I think more courteous to the developer, who agreed to fix this bug, and did so in a timely manner.

Okay, point taken. I was thinking about addressing @vladislavbelov's comment more correctly than with a comment.

Sun, Oct 17, 8:01 AM
Freagarach added a comment to D4310: cheaper Kush architecture tech..

I was toying with the idea that it could increase the garrison limit of the building by 20%(or 40%), which would make it a good tech for defense.

Please keep in mind that the garrison limit is not decimal safe, so one could use a percentage but someone (likely me) needs to fix the code first.

Sun, Oct 17, 7:58 AM
Freagarach added a comment to D4309: [gameplay] Roman bonus - structures +2 population space.

I'd rather have you change the templates than apply a tech.

Sun, Oct 17, 7:44 AM
Freagarach added a comment to D4307: Renaming of map size options.

Might be an option. Do have Medium as the median, please. ;)

Sun, Oct 17, 7:42 AM
Freagarach added a member for Contributors: LetswaveaBook.
Sun, Oct 17, 6:56 AM

Fri, Oct 15

Freagarach added a comment to D4308: Fix pop indications of paused items..

(Notice this brings a whole new idea for a can of worms in my head: that item should just be a "thing" (object) of its own. That would simplify the queue code a lot. ^^ )

Fri, Oct 15, 8:14 AM
Freagarach added a comment to D4307: Renaming of map size options.

Please consider D2927 (and its commit).

Fri, Oct 15, 8:08 AM

Thu, Oct 14

Freagarach added a comment to D4291: [gameplay] camouflaged ambush camp.

Asking the question is answering it. ;P

Thu, Oct 14, 5:43 PM
Freagarach added a comment to D4305: Fix bug where scenario map was not loading as revealed when set to revealed..

[Dropdown] Map state: Concealed, Explored, Revealed.
[Toggle] Allied view

Given the rather minor impact of the bug fixed, and the (seemingly) large (code) readability improvements created, I'd say do that in this patch (which should also fix the bug this patch fixes now).

Thu, Oct 14, 1:11 PM

Wed, Oct 13

Freagarach requested review of D4308: Fix pop indications of paused items..
Wed, Oct 13, 6:33 PM
Freagarach published D4291: [gameplay] camouflaged ambush camp for review.
Wed, Oct 13, 6:25 PM
Freagarach added a comment to D4307: Renaming of map size options.
In D4307#183320, @Stan wrote:

I'm not sure this declutters anything?

That ^ and why limit user choice needlessly? The Normal vs Medium discussion is one that has occured more often.

Wed, Oct 13, 3:50 PM
Freagarach added a comment to D4303: Add new TeamPlacement option "Beside Allies"; Add more TeamPlacement options to Mainland and African Plains: stronghold, line, randomGroup.

Teams together

:)

Wed, Oct 13, 10:37 AM
Freagarach added a comment to D4305: Fix bug where scenario map was not loading as revealed when set to revealed..

Works, would you agree to put those 3 options under one dropdown menu?

Agreed.

Wed, Oct 13, 10:35 AM
Freagarach added a comment to D4303: Add new TeamPlacement option "Beside Allies"; Add more TeamPlacement options to Mainland and African Plains: stronghold, line, randomGroup.

Should the option appear if there are only two players? Better just disable and gray them out.

That seems something for a separate patch, but a good idea indeed.

Wed, Oct 13, 8:09 AM
Freagarach added a member for Contributors: andy5995.
Wed, Oct 13, 8:01 AM

Tue, Oct 12

Freagarach added a comment to rP25953: Add accelerations in unit movement..

I always felt rams were much too mobile. But one may want to balance it with armour/health I guess.

Tue, Oct 12, 9:25 PM
Freagarach added a comment to D3249: Move parabolic range computation to rangemanager.

This works fine and reads correct.

Tue, Oct 12, 8:54 PM
Freagarach added inline comments to D3249: Move parabolic range computation to rangemanager.
Tue, Oct 12, 7:09 PM
Freagarach accepted D4302: Also do a postMove update when the average speed over last turn isn't zero.

Fixes the issue, looks good.

Tue, Oct 12, 6:28 PM
Freagarach added a comment to rP25958: Allow to push items to the front of the ProductionQueue..

I noticed some issues indeed with the "ghosts" of the autotrain.

Tue, Oct 12, 6:12 PM
Freagarach added a comment to D3523: Bring projectile speeds to realistic values.

I guess this is quite safe now?

Tue, Oct 12, 5:48 PM
Freagarach requested verification of rP25368: Let entity react on target death directly..
Tue, Oct 12, 5:27 PM
Freagarach added a comment to D4304: [Gameplay] - [WIP] Spartan spikes.

I'd rather have D1838 for this. ^^'

Tue, Oct 12, 4:53 PM
Freagarach added a comment to D4303: Add new TeamPlacement option "Beside Allies"; Add more TeamPlacement options to Mainland and African Plains: stronghold, line, randomGroup.

Also, have you seen the Frontier random map? That seems to have what you want.

Tue, Oct 12, 7:27 AM
Freagarach added a comment to D4303: Add new TeamPlacement option "Beside Allies"; Add more TeamPlacement options to Mainland and African Plains: stronghold, line, randomGroup.

Do you think it could be a general game-setup option? (I'm not sure whether we can generalise it like that.) I envision a checkbox Teams together for random maps that squeezes teammates closer when possible.

Tue, Oct 12, 7:22 AM

Mon, Oct 11

Freagarach added inline comments to D4303: Add new TeamPlacement option "Beside Allies"; Add more TeamPlacement options to Mainland and African Plains: stronghold, line, randomGroup.
Mon, Oct 11, 8:42 PM
Freagarach added a comment to rP25953: Add accelerations in unit movement..

It looks also that formation members outrun the controller and have to move backwards every now and then?

Mon, Oct 11, 8:05 PM
Freagarach added a comment to D4294: smoother formation manouvres.

@alre Have you installed the command-line tools alongside? I have information that TortoiseSVN can't add context.
(See: https://stackoverflow.com/questions/32723100/svn-is-not-recognized-as-an-internal-or-external-command-operable-program-or/53117050#53117050.)

Mon, Oct 11, 8:04 PM
Freagarach edited reviewers for D4303: Add new TeamPlacement option "Beside Allies"; Add more TeamPlacement options to Mainland and African Plains: stronghold, line, randomGroup, added: Restricted Owners Package; removed: Stan, Angen, Freagarach.
Mon, Oct 11, 7:56 PM
Freagarach added a comment to D3544: Regroup members at most once when formation and its members get idle.

Or even move the responsibility of keeping the correct position to the members. (And get back into formation when we become idle.)

Mon, Oct 11, 7:40 PM
Freagarach published D4302: Also do a postMove update when the average speed over last turn isn't zero for review.
Mon, Oct 11, 7:37 PM
Freagarach added a comment to D4303: Add new TeamPlacement option "Beside Allies"; Add more TeamPlacement options to Mainland and African Plains: stronghold, line, randomGroup.

This should definitely be an option, especially when the other option is having almost duplicated maps. ^^

Mon, Oct 11, 7:03 PM

Sun, Oct 10

Freagarach added a comment to rP25953: Add accelerations in unit movement..

When units get pushed, they keep the walking animation although they're idle. See the attached replay (r25958) around ten minutes.

Sun, Oct 10, 9:36 PM
Freagarach committed rP25958: Allow to push items to the front of the ProductionQueue..
Allow to push items to the front of the ProductionQueue.
Sun, Oct 10, 9:07 PM
Freagarach closed D4241: Allow to push items to the front of the ProductionQueue..
Sun, Oct 10, 9:07 PM
Freagarach committed rP25956: Add two missing hotkeys to the intro.txt..
Add two missing hotkeys to the intro.txt.
Sun, Oct 10, 8:40 PM
Freagarach closed D4284: Add two missing hotkeys to the intro.txt..
Sun, Oct 10, 8:40 PM
Freagarach accepted D4301: [Petra] Remove global constant from queueplanBuilding.js.

Correct, having these kind of globals around is bad.
It is not used anywhere else (which would have been really bad).

Sun, Oct 10, 8:36 PM
Freagarach added a comment to D3544: Regroup members at most once when formation and its members get idle.

One could tell the formation controller that we have have left/resumed the IDLE state and that could trigger the reforming?

Sun, Oct 10, 8:15 PM
Freagarach awarded rP25953: Add accelerations in unit movement. a Love token.
Sun, Oct 10, 8:55 AM
Freagarach added inline comments to rP25691: Map flares.
Sun, Oct 10, 8:53 AM
Freagarach accepted D4297: Fixing a problem where the AI attacked bridges.

The reason the bridge had health is because we couldn't have healthless entities earlier (refs. D341, D1268).

Sun, Oct 10, 8:08 AM

Sat, Oct 9

Freagarach updated the diff for D4241: Allow to push items to the front of the ProductionQueue..

Add semicolon. Fix icon being behind the progressslider.

Sat, Oct 9, 7:49 AM
Freagarach updated the summary of D4241: Allow to push items to the front of the ProductionQueue..
Sat, Oct 9, 7:40 AM
Freagarach edited reviewers for D4290: Adjusting the position of the generic/specific name if only one is displayed, added: bb; removed: Freagarach.

(It looks good in-game though.)

Sat, Oct 9, 7:32 AM
Freagarach published D4292: Moving the mininmal resolution to 1280x800 for review.
Sat, Oct 9, 7:02 AM

Fri, Oct 8

Freagarach added a comment to D4296: Layout changes to the replay menu.

It seems that in general, the red buttons are used for pupups, but either way, we should make them not less consistent, but more consistent.

Fri, Oct 8, 8:24 AM

Thu, Oct 7

Freagarach added a comment to D4204: Let PetraAI provide the resource map config instead of hardcoding it in the sharedScript..

I guess the same as the territory and border maps, which are also Petra specific, not player specific? So maybe some Petra-shared map?

Thu, Oct 7, 6:53 PM
Freagarach published D4296: Layout changes to the replay menu for review.
Thu, Oct 7, 4:18 PM

Wed, Oct 6

Freagarach planned changes to D4295: Ignore formation selection when clicking the icon..
Wed, Oct 6, 6:15 PM
Freagarach published D4295: Ignore formation selection when clicking the icon. for review.

There is no way of keeping the current behaviour, so this needs more thought, I guess.

Wed, Oct 6, 6:14 PM

Tue, Oct 5

Freagarach added a comment to D2382: [WIP] - Building Sockets..

I tried rebasing, but it requires more effort. Sorry.

Tue, Oct 5, 9:02 PM
Freagarach added a comment to D2926: [gui] make random maps the default.

What is missing for this patch @Freagarach ?

Agreement, I guess.

Tue, Oct 5, 8:18 PM
Freagarach updated the summary of D4284: Add two missing hotkeys to the intro.txt..
Tue, Oct 5, 8:02 PM
Freagarach updated the diff for D4284: Add two missing hotkeys to the intro.txt..

Some more.

Tue, Oct 5, 8:01 PM
Freagarach added inline comments to D4294: smoother formation manouvres.
Tue, Oct 5, 12:10 PM
Freagarach added a member for Contributors: alre.
Tue, Oct 5, 11:43 AM

Sun, Oct 3

Freagarach updated the diff for D4241: Allow to push items to the front of the ProductionQueue..

Also show paused on the queue.

Sun, Oct 3, 9:00 AM
Freagarach committed rP25949: Use transform for changing a formation template..
Use transform for changing a formation template.
Sun, Oct 3, 8:27 AM
Freagarach closed D4272: Use Transform for changing a formation template..
Sun, Oct 3, 8:27 AM
Freagarach added inline comments to D4272: Use Transform for changing a formation template..
Sun, Oct 3, 8:23 AM
Freagarach committed rP25948: Fix (de)selection functions..
Fix (de)selection functions.
Sun, Oct 3, 8:11 AM
Freagarach closed D4285: Clarify (de)selection functions..
Sun, Oct 3, 8:10 AM
Freagarach committed rP25947: Don't stop gathering after starting by autocontinue when in a formation..
Don't stop gathering after starting by autocontinue when in a formation.
Sun, Oct 3, 8:01 AM
Freagarach closed D4282: Don't stop gathering after starting by autocontinue when in a formation..
Sun, Oct 3, 8:00 AM
Freagarach committed rP25946: Check for visibility on finding treasures in UnitAI..
Check for visibility on finding treasures in UnitAI.
Sun, Oct 3, 7:48 AM
Freagarach closed D4286: Check for visibility on treasures in UnitAI..
Sun, Oct 3, 7:47 AM
Freagarach updated the diff for D4286: Check for visibility on treasures in UnitAI..

Also change the foundation filter.

Sun, Oct 3, 7:35 AM
Freagarach published D4290: Adjusting the position of the generic/specific name if only one is displayed for review.
Sun, Oct 3, 7:22 AM

Sat, Oct 2

Freagarach added inline comments to D4286: Check for visibility on treasures in UnitAI..
Sat, Oct 2, 6:52 AM

Fri, Oct 1

Freagarach planned changes to D4241: Allow to push items to the front of the ProductionQueue..

Okay, I agree, I'll put it in the bottom panel as well.

Fri, Oct 1, 8:49 AM