Page MenuHomeWildfire Games

Freagarach
User

Projects

User Details

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

Recent Activity

Today

Freagarach raised a concern with rP25133: Exclude Towers from Town->City phase requirements..

Concern that it may not be forgotten after the release.

Wed, Jul 28, 7:04 AM

Yesterday

Freagarach accepted D4201: Prevent formation members to run when they should not.

Fixes the running in formation without need.

Tue, Jul 27, 8:17 PM
Freagarach added inline comments to D4200: [PetraAI] inline in DefenseManager.
Tue, Jul 27, 8:08 PM
Freagarach added inline comments to D4200: [PetraAI] inline in DefenseManager.
Tue, Jul 27, 7:42 PM
Freagarach added a comment to D4154: Fix attack sound not playing in some cases when chasing a fleeing unit.

It made chasing hard again, so not really.

Tue, Jul 27, 5:03 PM

Sun, Jul 25

Freagarach added a comment to rP25837: Check range extra while gathering to prevent animation changes when gathering….

Fixes #6245.

Sun, Jul 25, 10:11 PM
Freagarach committed rP25837: Check range extra while gathering to prevent animation changes when gathering….
Check range extra while gathering to prevent animation changes when gathering…
Sun, Jul 25, 8:20 PM
Freagarach closed D4198: Short circuit dropping resources which are very near..
Sun, Jul 25, 8:20 PM

Sat, Jul 24

Freagarach added inline comments to D3998: [PetraAI] - [WIP] Implement an emergency mode in case of huge losses..
Sat, Jul 24, 9:25 AM
Freagarach requested review of D4198: Short circuit dropping resources which are very near..
Sat, Jul 24, 7:01 AM

Fri, Jul 23

Freagarach committed rP25835: Don't lose trade gain when ordering a trader..
Don't lose trade gain when ordering a trader.
Fri, Jul 23, 7:25 AM
Freagarach closed D4197: Don't lose trading gain when ordering a trader..
Fri, Jul 23, 7:25 AM

Thu, Jul 22

Freagarach added inline comments to D4192: [WIP] - [PetraAI] - Manage bases in separate BaseManager..
Thu, Jul 22, 8:57 PM
Freagarach removed 1 auditor(s) for rP25655: Standardize prepare time for ranged units.: ValihrAnt.

As per above.

Thu, Jul 22, 8:40 PM
Freagarach requested review of D4197: Don't lose trading gain when ordering a trader..
Thu, Jul 22, 4:51 PM

Wed, Jul 21

Freagarach added a comment to rP25655: Standardize prepare time for ranged units..

@ValihrAnt Is this concern fixed?

Wed, Jul 21, 7:01 PM

Tue, Jul 20

Freagarach updated the diff for D4192: [WIP] - [PetraAI] - Manage bases in separate BaseManager..
  • More const.
  • Some more rearrangements.
Tue, Jul 20, 2:50 PM
Freagarach added inline comments to D4149: Call to the Arms-Button (#1364).
Tue, Jul 20, 10:22 AM
Freagarach committed rP25828: Fix PetraAI constructing ships as ordinary units..
Fix PetraAI constructing ships as ordinary units.
Tue, Jul 20, 7:20 AM
Freagarach closed D4193: Fix PetraAI training ships as ordinary units..
Tue, Jul 20, 7:20 AM
Freagarach updated the summary of D4193: Fix PetraAI training ships as ordinary units..
Tue, Jul 20, 7:16 AM
Freagarach accepted D4195: [AI] Simplify getMetadata in sharedscript.

One could use the ?. notation to make it a one-liner.

Tue, Jul 20, 6:53 AM
Freagarach added a comment to D4196: Limited abilities for Introductory Tutorial.

I think it is cleaner to disable stuff using triggers.

Tue, Jul 20, 6:48 AM · Contributors
Freagarach added a member for Contributors: azayrahmad.
Tue, Jul 20, 6:46 AM

Mon, Jul 19

Freagarach updated the diff for D4193: Fix PetraAI training ships as ordinary units..

Don't exclude ships explicitly ships.

Mon, Jul 19, 4:04 PM
Freagarach added inline comments to D4149: Call to the Arms-Button (#1364).
Mon, Jul 19, 3:57 PM
Freagarach requested changes to D4149: Call to the Arms-Button (#1364).
Mon, Jul 19, 8:24 AM
Freagarach updated the diff for D4193: Fix PetraAI training ships as ordinary units..

Need both classes (fixes PetraAI training loads of cavalry).

Mon, Jul 19, 7:00 AM
Freagarach accepted D4191: Enable Cartography at Start of Game Option.

Code looks good, I haven't tested this.

Mon, Jul 19, 6:49 AM

Sun, Jul 18

Freagarach requested review of D4193: Fix PetraAI training ships as ordinary units..
Sun, Jul 18, 11:18 AM
Freagarach updated the diff for D4192: [WIP] - [PetraAI] - Manage bases in separate BaseManager..

Separate baseless base.

Sun, Jul 18, 10:28 AM
Freagarach committed rP25826: (Partly) Fix PetraAI building a lot of siege and not using it..
(Partly) Fix PetraAI building a lot of siege and not using it.
Sun, Jul 18, 7:07 AM
Freagarach closed D4184: Fix PetraAI building a lot of siege and not using it..
Sun, Jul 18, 7:07 AM
Freagarach updated the Trac tickets for D4184: Fix PetraAI building a lot of siege and not using it..
Sun, Jul 18, 6:56 AM

Sat, Jul 17

Freagarach added a comment to D4192: [WIP] - [PetraAI] - Manage bases in separate BaseManager..

I agree, that base is special, so it can also be treated special.
Can I conclude that you agree to the concept?

Sat, Jul 17, 10:08 AM
Freagarach updated the diff for D4184: Fix PetraAI building a lot of siege and not using it..

Fix PetraAI building Siege Towers now (which she doesn't understand how to use).

Sat, Jul 17, 10:04 AM
Freagarach added a comment to rP25824: Fix treasure in statistics.

Thanks for this. It shows that I should use better naming.

Sat, Jul 17, 8:45 AM

Sun, Jul 11

Freagarach updated the diff for D4192: [WIP] - [PetraAI] - Manage bases in separate BaseManager..
  • Address @Angen's comment.
  • const
  • Moved some functionality to basesManager.
Sun, Jul 11, 8:53 AM
Freagarach planned changes to D4192: [WIP] - [PetraAI] - Manage bases in separate BaseManager..
Sun, Jul 11, 6:45 AM

Sat, Jul 10

Freagarach requested review of D4192: [WIP] - [PetraAI] - Manage bases in separate BaseManager..
Sat, Jul 10, 6:30 PM
Freagarach added inline comments to D4191: Enable Cartography at Start of Game Option.
Sat, Jul 10, 6:11 PM
Freagarach added inline comments to D4191: Enable Cartography at Start of Game Option.
Sat, Jul 10, 5:39 PM
Freagarach added inline comments to D4191: Enable Cartography at Start of Game Option.
Sat, Jul 10, 5:23 PM
Freagarach updated the diff for D4184: Fix PetraAI building a lot of siege and not using it..

Use build order name.

Sat, Jul 10, 1:05 PM
Freagarach requested changes to D4149: Call to the Arms-Button (#1364).

One may want to push this order to the front (to go back to gathering when finished) so I think it is worth to check whether that route can be implemented.

Sat, Jul 10, 8:39 AM
Freagarach requested changes to D4191: Enable Cartography at Start of Game Option.

Some code comments. I haven't tested this yet, but thanks for the patch already!

Sat, Jul 10, 8:25 AM

Tue, Jul 6

wowgetoffyourcellphone awarded D2175: [WIP] - Basic battalionish support. a Yellow Medal token.
Tue, Jul 6, 4:36 AM

Mon, Jul 5

Freagarach retitled D4184: Fix PetraAI building a lot of siege and not using it. from [WIP] - Fix PetraAI building a lot of siege and not using it. to [WIP] - Fix PetraAI building a lot of siege and ships and not using it..
Mon, Jul 5, 8:50 AM
Freagarach committed rP25819: Disable autoqueue when changing ownership..
Disable autoqueue when changing ownership.
Mon, Jul 5, 8:19 AM
Freagarach closed D4185: Disable the autoqueue when changing ownership..
Mon, Jul 5, 8:19 AM

Sat, Jul 3

Freagarach added inline comments to D4189: Additional loading screen tips.
Sat, Jul 3, 7:50 AM
Freagarach updated subscribers of D4189: Additional loading screen tips.

@Stan, can you add @Grapjas to the contributers list, please?

Sat, Jul 3, 7:49 AM
Freagarach added a comment to D4182: Fix gamesetup crash when Daytime resets..

Should there be a comment/ref. to ticket?

Sat, Jul 3, 7:19 AM

Wed, Jun 30

Freagarach added a comment to D4187: Citizen soldiers attack gaia animals without provocation.

Keep in mind that people will be annoyed because they will lose their units sooner, since the dangerous animals will attack first.

Wed, Jun 30, 6:21 PM
Freagarach requested review of D4185: Disable the autoqueue when changing ownership..
Wed, Jun 30, 5:10 PM
Freagarach added a comment to D4184: Fix PetraAI building a lot of siege and not using it..

True, true, but I'm merely saying that it was broken already, but less so than it is now. "It will work" as in hot-fix. Feel free to point me in another direction for a more proper fix.

Wed, Jun 30, 1:24 PM
Freagarach added a comment to D4184: Fix PetraAI building a lot of siege and not using it..
In D4184#177941, @Angen wrote:

But why was Matchclasslist not working?

Because we try to match a more general condition (Siege) on a more specific class (Siege+Melee).
So it was working. This condition checks string presence, so that will work.

Wed, Jun 30, 11:18 AM
Freagarach updated the diff for D4184: Fix PetraAI building a lot of siege and not using it..

Siege was built in the wrong queue.

Wed, Jun 30, 10:47 AM
Freagarach requested review of D4184: Fix PetraAI building a lot of siege and not using it..
Wed, Jun 30, 9:14 AM
Freagarach added a comment to rP25584: Improve support for classes in PetraAI..

D4184, but there is more, since she keeps constructing siege without using it.

Wed, Jun 30, 9:04 AM

Tue, Jun 29

Freagarach added inline comments to rP25584: Improve support for classes in PetraAI..
Tue, Jun 29, 5:23 PM
Freagarach added a comment to rP25584: Improve support for classes in PetraAI..

Either causes or exposes https://wildfiregames.com/forum/topic/45178-a25-svn-version-25810-not-playable/.
I guess a concern would be in place.

Tue, Jun 29, 12:54 PM
Freagarach committed rP25814: Give picking up treasures back its sound..
Give picking up treasures back its sound.
Tue, Jun 29, 11:04 AM
Freagarach closed D4180: Give picking up treasures a sound..
Tue, Jun 29, 11:04 AM
Freagarach committed rP25813: Don't show turreted entities as idle..
Don't show turreted entities as idle.
Tue, Jun 29, 10:53 AM
Freagarach closed D4179: Don't show turreted entities as idle..
Tue, Jun 29, 10:53 AM
Freagarach added a comment to D4180: Give picking up treasures a sound..

That's how it should be, right?

Well, ideally not, I guess, but for now, yes.

Tue, Jun 29, 10:34 AM
Freagarach accepted D4181: Adding small remark to background pause in options.

Good change.

Tue, Jun 29, 10:21 AM
Freagarach added a comment to rP25216: Move function to perform trade to substate..

(May or may not be worth a concern.)

Tue, Jun 29, 10:09 AM
Freagarach accepted rP25772: RM generation: improve metal mines generation.

My concern is fixed in rP25811.

Tue, Jun 29, 8:59 AM

Jun 24 2021

Freagarach requested changes to D2772: Prevent Upgrading units from moving.

Taking it out of the queue, but I might take this over some day (TM).

Jun 24 2021, 8:16 AM
Freagarach added a comment to D4123: Fix some gaia units having incorrect players.

For fauna: I guess, yes. Though you can also train e.g. zebu?
Flora: Certainly, they should be owned by GAIA.
Ruins: Not sure, for those can be player owned?

Jun 24 2021, 7:44 AM
Freagarach published D4180: Give picking up treasures a sound. for review.
Jun 24 2021, 6:58 AM
Freagarach published D4179: Don't show turreted entities as idle. for review.
Jun 24 2021, 6:57 AM

Jun 22 2021

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

or could I just open up another account ?

Be aware that it is against the Terms Of Service, that you agreed to, to create a second account, unless allowed by WFG (in person of @user1 in this case). So if you think you really need it you should ask @user1.

Jun 22 2021, 7:12 AM

Jun 21 2021

Freagarach updated subscribers of rP25151: Refuse to serialize NaN values..

I just had a discussion with @elexis, it would be good if we can't crash from JS (might have been a thing in the past). So it would be nice if we could reliably serialise NaN and error, or just don't serialise but do error.

Jun 21 2021, 10:30 AM
Freagarach added a comment to rP25381: Enable production entities to autoqueue..

Feature should not be changed just to please someones liking but to adress flaws it has. In case it does not have them, it should not be changed.

I agree, but the concern seemed to be a matter of preference, so hence the way I asked the question. But I guess it is not enough, since @nani did not resign from this commit yet.

Jun 21 2021, 9:06 AM

Jun 20 2021

Freagarach accepted rP24360: Make units wait a few seconds at waypoints during Patrol..

My concern was fixed in rP25804.

Jun 20 2021, 8:20 AM
Freagarach accepted D4176: Fixup rP25772.
Jun 20 2021, 8:19 AM
Freagarach added a comment to rP25381: Enable production entities to autoqueue..

@nani has the autoqueue been adapted enough to your liking?

Jun 20 2021, 8:18 AM
Freagarach updated the summary of D4178: Fix Atlas build on OpenSuse.
Jun 20 2021, 7:47 AM

Jun 18 2021

Freagarach added a comment to rP24366: Add support for HAIKU detection by the SDL..
libsdl.cpp
../../../source/lib/external_libraries/libsdl.cpp: In function ‘const char* GetSDLSubsystem(SDL_Window*)’:
../../../source/lib/external_libraries/libsdl.cpp:89:13: warning: this statement may fall through [-Wimplicit-fallthrough=]
   89 |   subsystem = "Haiku";
      |   ~~~~~~~~~~^~~~~~~~~
../../../source/lib/external_libraries/libsdl.cpp:92:2: note: here
   92 |  default:
      |  ^~~~~~~
Jun 18 2021, 4:48 PM

Jun 17 2021

Freagarach committed rP25804: Fix serialisation crash when patrolling..
Fix serialisation crash when patrolling.
Jun 17 2021, 9:40 AM
Freagarach closed D4177: Fix crash when patrolling in MP..
Jun 17 2021, 9:40 AM
Freagarach requested review of D4177: Fix crash when patrolling in MP..
Jun 17 2021, 7:06 AM
Freagarach raised a concern with rP24360: Make units wait a few seconds at waypoints during Patrol..
Jun 17 2021, 6:40 AM

Jun 16 2021

Freagarach added a comment to rP25789: Update some MR map previews..

Well, it is a nice feature when it can be automated.

Jun 16 2021, 5:50 PM
Freagarach updated subscribers of rP25789: Update some MR map previews..

As noted by @elexis, this reverts rP21679.

Jun 16 2021, 5:47 PM
Freagarach raised a concern with rP25772: RM generation: improve metal mines generation.

As stated inline.

Jun 16 2021, 5:44 PM
Freagarach added inline comments to rP25772: RM generation: improve metal mines generation.
Jun 16 2021, 5:15 PM
Freagarach added inline comments to rP25772: RM generation: improve metal mines generation.
Jun 16 2021, 5:11 PM
Freagarach added a comment to D4174: Siege workshop tooltip correction.

Have you searched for the reason this string is as is? (Hint: search.)

Jun 16 2021, 4:16 PM
Freagarach added a comment to D4173: Fix bug with map-dependent random settings.

You've based the patch wrongly. ;)

Jun 16 2021, 7:59 AM

Jun 15 2021

Freagarach updated the diff for D4171: Mirage unrepairable in cmpRepairable..

Don't store the variable when not present.

Jun 15 2021, 9:32 AM
Freagarach planned changes to D3853: [Gameplay] - Don't chase when there are still targets around..

I can only think of expensive calculations like checking LOS for enemies and filtering those to closer units.

Jun 15 2021, 9:23 AM
Freagarach added a comment to D3740: delete several useless fauna templates.

I think you might be able to use the mixins here.

Jun 15 2021, 8:44 AM
Freagarach requested review of D4171: Mirage unrepairable in cmpRepairable..
Jun 15 2021, 7:20 AM

Jun 14 2021

Freagarach accepted D4170: Fix options occasionally thinking they've been modified when they haven't..

Works for me, no clue about any other implications.

Jun 14 2021, 10:48 AM
Freagarach committed rP25796: Provide Magadha description..
Provide Magadha description.
Jun 14 2021, 8:15 AM
Freagarach closed D4169: Updating Magadha description.
Jun 14 2021, 8:15 AM