Page MenuHomeWildfire Games

Freagarach
User

Projects

User Details

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

Recent Activity

Mon, Sep 18

Freagarach accepted D5133: [Gameplay] Cavalry units created from a academy should be able to enter it.

Seems logical.

Mon, Sep 18, 9:09 AM
Freagarach added a comment to rP27531: Don't error out on complex requirements without tooltip..

I am seeing this warning all the time.

Define "all the time"? Since I don't get it al all. ^^

Mon, Sep 18, 9:00 AM
Freagarach added a comment to D5109: [Gameplay] Add legionaries, add tech for 'Marian Reforms' for Rome.

Could you send me your mainlog.txt file?

Please only ask for interestinglog.html. :) Since that contains the errors and warnings, but the mainlog also has private information.

Mon, Sep 18, 8:42 AM

Wed, Sep 13

Freagarach added a comment to D5127: Fix riscv64 build with GCC 13.

Thanks for the patch, @xctan! I've added you to the "Contributors" group, so the next time you upload a patch, it is built by our CI.
I can't comment on the patch itself, but I can ask you to upload it with context and add yourself to the credits. :)
(See https://trac.wildfiregames.com/wiki/SubmittingPatches#Makingsomechanges.)

Wed, Sep 13, 7:39 AM
Freagarach added a member for Contributors: xctan.
Wed, Sep 13, 7:35 AM
Freagarach requested changes to D5126: standardize game option names, minor changes.
Wed, Sep 13, 7:33 AM

Mon, Sep 11

Freagarach added inline comments to D5117: [Gameplay] Slighly reduce cost of Carth apartment and allow house upgrade to apartment.
Mon, Sep 11, 9:23 AM
Freagarach added a comment to D5120: [Gameplay] Stone tower brit bonus civ.

I'll continu the discussion here. What I meant to say was that for constructing towers in neutral territory you'd probably search for a description like the brochs: "Defensive structures (...) that dotted the rest of the islands." from Wikipedia. (IMHO you can bend the historical justification a bit.) My concern for this patch is that the history in the civ.json talks about large towers and so it doesn't match the effect. Change the history and there's no problem anymore. ;P

Mon, Sep 11, 9:12 AM
Freagarach added inline comments to D5117: [Gameplay] Slighly reduce cost of Carth apartment and allow house upgrade to apartment.
Mon, Sep 11, 9:00 AM

Fri, Sep 8

Freagarach updated the diff for D4584: [WIP] - Technologies to templates..

Just handle ChoiceRoots a bit differently.

Fri, Sep 8, 3:42 PM
Freagarach added a comment to D4972: Fix race condition on defeat..

I'm not yet satisfied, since when the entity list updates and some entry is removed, the PQ won't notice until it is being produced. (After which it'll silently remove it.)

Fri, Sep 8, 12:23 PM
Freagarach updated the summary of D4972: Fix race condition on defeat..
Fri, Sep 8, 12:19 PM
Freagarach updated the diff for D4972: Fix race condition on defeat..

Include the fix.

Fri, Sep 8, 12:16 PM
Freagarach committed rP27832: Add some chat messages to PetraAI..
Add some chat messages to PetraAI.
Fri, Sep 8, 11:23 AM
Freagarach closed D5100: new chat messages where they might be most needed.
Fri, Sep 8, 11:23 AM
Freagarach committed rP27831: Bring back (small) splash for catapults, heavy ship..
Bring back (small) splash for catapults, heavy ship.
Fri, Sep 8, 11:14 AM
Freagarach closed D5053: Bring back (small) splash for catapults, heavy ship.
Fri, Sep 8, 11:14 AM · Balancing
Freagarach added inline comments to D5095: add brief explanations to the tooltips of the alert buttons.
Fri, Sep 8, 11:08 AM
Freagarach added inline comments to D5117: [Gameplay] Slighly reduce cost of Carth apartment and allow house upgrade to apartment.
Fri, Sep 8, 10:39 AM
Freagarach added a comment to D5119: Create a CHARIOT mixin for unit differentiation.

I think to truly bring out the potential of chariots (and elephants), a trample aura and charge bonus need implemented. But I think these values go a long way to differentiate the chariots on a stats basis.

(I could rebase D1838 but who's going to review that,,,)

Fri, Sep 8, 10:35 AM
Freagarach added inline comments to D5120: [Gameplay] Stone tower brit bonus civ.
Fri, Sep 8, 10:30 AM

Thu, Sep 7

Freagarach added inline comments to D5119: Create a CHARIOT mixin for unit differentiation.
Thu, Sep 7, 8:26 AM
Freagarach added inline comments to D5114: [Gameplay] Add Onager for romans.
Thu, Sep 7, 8:17 AM
Freagarach added a comment to D5099: [Gameplay] Agis aura.

@borg- You can close this revision by using the dropdown at the bottom of the page.

Thu, Sep 7, 8:12 AM

Mon, Sep 4

Freagarach requested verification of rP27722: Pull Diplomacy out of cmpPlayer..

Thanks for the reports (and the proposed fix). :)

Mon, Sep 4, 3:07 PM
Freagarach committed rP27824: Fix typo in r27722 / rP27722..
Fix typo in r27722 / rP27722.
Mon, Sep 4, 3:05 PM
Freagarach added a comment to D5116: [Gameplay] Druids Aura..
In D5116#217458, @borg- wrote:

An attack would remove the power to heal.

Because UnitAI is dumb? (For it should work together.)

Mon, Sep 4, 1:00 PM

Thu, Aug 31

Freagarach added a comment to D5114: [Gameplay] Add Onager for romans.
In D5114#217292, @Stan wrote:
The onager (British /ˈɒnədʒə/, /ˈɒnəɡə/, U.S. /ˈɑnədʒər/)[1] was a Roman torsion powered siege engine. It is commonly depicted as a catapult with a bowl, bucket, or sling at the end of its throwing arm. The onager was first mentioned in 353 AD by Ammianus Marcellinus, who described onagers as the same as a scorpion.

Later in the same wiki article it says:
"It may have originated in the third century BC." I think there is enough justification. This is also partially why I initially wanted to lock it behind the reforms upgrade, since this indicates an evolution of the army.

One could ask the historians on the forums.

Thu, Aug 31, 9:18 AM
Freagarach awarded rP27818: Provides entity ID for unit in its constructor. a Love token.
Thu, Aug 31, 9:11 AM

Aug 17 2023

Stan awarded rP27805: Fix missing space in TaC of the userreporter. a Love token.
Aug 17 2023, 6:11 PM
Freagarach committed rP27805: Fix missing space in TaC of the userreporter..
Fix missing space in TaC of the userreporter.
Aug 17 2023, 8:39 AM
Freagarach added a comment to D5096: [Gameplay] Differentiate macedonia.
In D5096#216886, @borg- wrote:

@Freagarach is there any way to limit aura just for Macedonia?

One could leave the usual trickle and give the Mace-specific (then +3 to have a total of +4) a (cheap) requiredTechnology that is Mace-specific.
(I have plans to rewrite Aura's to have the same requirements as everything else, but no time for those plans, sadly.)

Aug 17 2023, 8:22 AM

Aug 16 2023

Freagarach accepted D5100: new chat messages where they might be most needed.

Read fine.

Aug 16 2023, 1:04 PM
Freagarach committed rP27804: Add 10% health on hoplite tradition tech..
Add 10% health on hoplite tradition tech.
Aug 16 2023, 12:23 PM
Freagarach closed D5098: [Gameplay] Add 10% health on hoplite tradition tech.
Aug 16 2023, 12:23 PM
Freagarach added a comment to D5096: [Gameplay] Differentiate macedonia.

Is it intended that:

  • Once another civ seizes the Macedonian barracks, they get the +4 XP trickle as well?
  • Once another civ seizes their Stables they can research the tactical_formation?
Aug 16 2023, 12:16 PM
Freagarach updated subscribers of rP27803: Add a civbonus entry to rome.json explaining centurions..

@borg- (sorry for the missing dash)

Aug 16 2023, 12:08 PM
Freagarach committed rP27803: Add a civbonus entry to rome.json explaining centurions..
Add a civbonus entry to rome.json explaining centurions.
Aug 16 2023, 12:08 PM
Freagarach closed D5083: Add a civbonus entry to rome.json explaining centurions..
Aug 16 2023, 12:07 PM
Freagarach committed rP27802: Add and adjust civbonus entries for spart.json and athen.json..
Add and adjust civbonus entries for spart.json and athen.json.
Aug 16 2023, 12:02 PM
Freagarach closed D5087: add and adjust civbonus entries for spart.json and athen.json.
Aug 16 2023, 12:02 PM
Freagarach added a comment to D5087: add and adjust civbonus entries for spart.json and athen.json.

It does make it a bit annoying to commit, so you may want to look into that. ;) You can even copy-paste tabs.

Aug 16 2023, 11:58 AM
Freagarach committed rP27801: Don't show the amount in the mutli-entity view when merely one type of a unit….
Don't show the amount in the mutli-entity view when merely one type of a unit…
Aug 16 2023, 11:57 AM
Freagarach closed D5075: when only one entity of a type is selected/garrisoned, don't show the number.
Aug 16 2023, 11:57 AM
Freagarach committed rP27800: Fix performance drop and memory leak on networked AI matches..
Fix performance drop and memory leak on networked AI matches.
Aug 16 2023, 11:48 AM
Freagarach closed D5042: Fix performance drop and memory leak on network games with AI.
Aug 16 2023, 11:48 AM
Freagarach resigned from D5102: improve color of repair icon.

Very nice to see you're into everything. :)

Aug 16 2023, 8:06 AM
Freagarach added a comment to D5095: add brief explanations to the tooltips of the alert buttons.

(Notice healers are support class as well, but don't get garrisoned. See in the templates (hint AlertRaiser) what class(es) is(/are) affected and make of that what you want. :) )

Aug 16 2023, 8:03 AM
Freagarach added inline comments to D5097: change isUndeletable to getDeletionPolicy and hide deletion option for animals.
Aug 16 2023, 7:52 AM

Aug 14 2023

Freagarach published D5099: [Gameplay] Agis aura for review.
Aug 14 2023, 10:52 AM
Freagarach added a comment to D5100: new chat messages where they might be most needed.

Nice additions. :)

Aug 14 2023, 10:52 AM
Freagarach published D5100: new chat messages where they might be most needed for review.
Aug 14 2023, 10:47 AM
Freagarach added a comment to D5097: change isUndeletable to getDeletionPolicy and hide deletion option for animals.

I haven't reviewed the logic yet, I'll try that later.

Aug 14 2023, 10:42 AM
Freagarach published D5097: change isUndeletable to getDeletionPolicy and hide deletion option for animals for review.

(If a revision takes too long to get out of 'draft' status, one can manually 'request review' using the dropdown at the bottom of the page 'Add Action'.)

Aug 14 2023, 10:35 AM

Aug 8 2023

Freagarach added a comment to D5094: introduce badge to indicate the phase.

Nice feature!
We may need to take into account that this leaks information to enemies.

Aug 8 2023, 9:30 AM

Aug 7 2023

Freagarach added a comment to D5075: when only one entity of a type is selected/garrisoned, don't show the number.

You can ping me on IRC, or send a PM through the forum. :)
One can start a topic at e.g. https://wildfiregames.com/forum/forum/312-game-development-amp-technical-discussion/. Although I don't think it is necessary for this change.

Aug 7 2023, 8:33 AM

Jul 31 2023

Freagarach accepted D5087: add and adjust civbonus entries for spart.json and athen.json.

Reads fine.

Jul 31 2023, 8:00 AM
Freagarach accepted D5083: Add a civbonus entry to rome.json explaining centurions..

Reads correct.

Jul 31 2023, 7:58 AM
Freagarach added a comment to D5083: Add a civbonus entry to rome.json explaining centurions..

I'll change to \t indentation on commit.

Jul 31 2023, 7:57 AM

Jul 27 2023

Freagarach accepted D4313: Add a linter for project name misspellings.

Reads good. Didn't test.

Jul 27 2023, 2:50 PM
Freagarach added inline comments to D5087: add and adjust civbonus entries for spart.json and athen.json.
Jul 27 2023, 2:43 PM
Freagarach added inline comments to D5084: Recover Wall Archers on Jebel Barkal.
Jul 27 2023, 2:40 PM
Freagarach added inline comments to D5083: Add a civbonus entry to rome.json explaining centurions..
Jul 27 2023, 2:36 PM

Jul 22 2023

Freagarach accepted D5075: when only one entity of a type is selected/garrisoned, don't show the number.

Patch is good, IMHO the feature is an improvement.

Jul 22 2023, 5:18 PM
Freagarach added a comment to D5075: when only one entity of a type is selected/garrisoned, don't show the number.

Oh, you _were_ already in it. ^^' I indeed only checked the nicks. Sorry,,,

Jul 22 2023, 5:16 PM
Freagarach accepted D5042: Fix performance drop and memory leak on network games with AI.

Works, good solution.
This helps a lot!

Jul 22 2023, 5:07 PM
Freagarach committed rP27779: Differentiate Athenians..
Differentiate Athenians.
Jul 22 2023, 4:53 PM
Freagarach closed D3727: [gameplay] differentiating Athens.
Jul 22 2023, 4:52 PM
Freagarach committed rP27778: Add descriptions to formations..
Add descriptions to formations.
Jul 22 2023, 4:15 PM
Freagarach closed D5074: include military formation descriptions .
Jul 22 2023, 4:14 PM
Freagarach published D5075: when only one entity of a type is selected/garrisoned, don't show the number for review.

I failed to ask before, but could you add yourself to the programming credits, please?

Jul 22 2023, 4:11 PM
Freagarach committed rP27777: Fix formation slow-down after stopping..
Fix formation slow-down after stopping.
Jul 22 2023, 4:05 PM
Freagarach closed D5064: Fix formation slow-down after stopping..
Jul 22 2023, 4:05 PM
Freagarach committed rP27776: Fix missing `?` in r27736 / rP27736..
Fix missing `?` in r27736 / rP27736.
Jul 22 2023, 3:57 PM
Freagarach committed rP27775: Differentiate Sparta..
Differentiate Sparta.
Jul 22 2023, 3:55 PM
Freagarach closed D4736: [Gameplay] Differentiating Sparta.
Jul 22 2023, 3:54 PM · Balancing

Jul 21 2023

Freagarach accepted D3727: [gameplay] differentiating Athens.

Looks good on a glance.

Jul 21 2023, 2:26 PM
Freagarach added inline comments to D5082: check if there is enough distance between players to place wall.
Jul 21 2023, 2:18 PM
Freagarach accepted D5077: fix incorect random selection in gamesetup.

Good fix.

Jul 21 2023, 2:12 PM

Jul 17 2023

Freagarach added inline comments to D5077: fix incorect random selection in gamesetup.
Jul 17 2023, 8:36 AM
Freagarach accepted D5080: Fix Liu Bang's cavalry attack aura.

Fine by me.

Jul 17 2023, 8:19 AM

Jul 11 2023

Freagarach accepted D5074: include military formation descriptions .

Looks good, I'll commit it once I find the time to do so. (Anyone else with commit access may beat me to it. :) )

Jul 11 2023, 9:13 AM

Jul 10 2023

Freagarach added a comment to D5074: include military formation descriptions .

I suggest putting it in the tooltip of the Identity component.

Jul 10 2023, 9:03 AM

Jul 3 2023

Freagarach added a comment to D4736: [Gameplay] Differentiating Sparta.
In D4736#215457, @borg- wrote:

@Freagarach can commit it?

When I find the time to do so, yes. ^^

Jul 3 2023, 10:49 AM · Balancing

Jun 26 2023

Freagarach added a comment to D4646: Parse autostart arguments in Javascript.

[10:17:29] <elexis1> for D4646 there is also a big thing all people seem to be missing
[10:18:09] <elexis1> each line in cmd_line_args.js should be moved to gamesettings/attributes/
[10:21:53] <elexis1> GameSettings.prototype.Attributes.Biome = class Biome extends GameSetting
[10:21:53] <elexis1> {
[10:21:55] <elexis1> fromCmdLineArgs(...)
[10:21:57] <elexis1> {
[10:21:59] <elexis1> ....
[10:22:01] <elexis1> }
[10:22:03] <elexis1> }
[10:22:52] <elexis1> if we can have a fromInitAttributes then a fromCmdLineArgs can be implemented too

Jun 26 2023, 10:25 AM
Freagarach added a comment to rP27736: Fix Gaia not being its own mutual ally..
Jun 26 2023, 10:23 AM

Jun 23 2023

Freagarach committed rP27737: Fix typo in `CanTrade`..
Fix typo in `CanTrade`.
Jun 23 2023, 2:58 PM
Freagarach closed D5060: Fix typo in `CanTrade`..
Jun 23 2023, 2:58 PM
Freagarach committed rP27736: Fix Gaia not being its own mutual ally..
Fix Gaia not being its own mutual ally.
Jun 23 2023, 2:54 PM
Freagarach closed D5058: Fix Gaia not being its own mutual ally..
Jun 23 2023, 2:54 PM
Freagarach committed rP27735: Fix missed allied view tech in init..
Fix missed allied view tech in init.
Jun 23 2023, 2:51 PM
Freagarach closed D5057: Fix missed allied view tech in init..
Jun 23 2023, 2:51 PM
Freagarach requested review of D5060: Fix typo in `CanTrade`..
Jun 23 2023, 9:00 AM
Freagarach added a comment to D4736: [Gameplay] Differentiating Sparta.

@borg- can you make your patches with Public view policy, please. :)

Jun 23 2023, 8:58 AM · Balancing
Freagarach added a comment to D4736: [Gameplay] Differentiating Sparta.
In D4736#215040, @borg- wrote:

If you decide to commit this, then can you make these changes?

Will do, sir!

Jun 23 2023, 7:51 AM · Balancing

Jun 22 2023

Freagarach requested review of D5058: Fix Gaia not being its own mutual ally..
Jun 22 2023, 3:02 PM
Freagarach updated the summary of D5057: Fix missed allied view tech in init..
Jun 22 2023, 2:45 PM
Freagarach requested review of D5057: Fix missed allied view tech in init..
Jun 22 2023, 1:04 PM
Freagarach updated subscribers of D4736: [Gameplay] Differentiating Sparta.

@Freagarach I believe this could be committed as others expressed their agreements?

I see merely one acceptance and it is outdated (i.e. the patch has been significantly updated since).

Jun 22 2023, 11:15 AM · Balancing
Freagarach accepted D5030: Defense manager bugfix.

Reads correct. Didn't test.

Jun 22 2023, 11:02 AM