Page MenuHomeWildfire Games
Feed All Stories

Today

borg- accepted D2668: [gameplay] deprecate rotary mill, introduce special gaul farming technology.

Ok, im fine with 15%.

Thu, Jun 4, 4:11 AM
borg- added a comment to D2782: [Gameplay] make ram less effective.

Up new patch.

Thu, Jun 4, 4:08 AM · Restricted Project, Contributors
borg- updated the summary of D2782: [Gameplay] make ram less effective.
Thu, Jun 4, 4:07 AM · Restricted Project, Contributors
borg- updated the diff for D2782: [Gameplay] make ram less effective.
Thu, Jun 4, 4:02 AM · Restricted Project, Contributors
borg- added a comment to D2782: [Gameplay] make ram less effective.

Restricting Organic already changes the gameplay balance significantly, so I don't see why Unit is suddenly problematic.

Thu, Jun 4, 3:51 AM · Restricted Project, Contributors
borg- added a comment to D2782: [Gameplay] make ram less effective.

A few more things:

  • The current vision range seems rather excessive; perhaps it should be lowered to 40 or less. What do you think?
  • If rams can no longer attack (organic) units, that makes them significantly less effective, so should its cost be changed?
  • Could you change the <PreferredClasses> to "Gates Defensive Structure", in that order (cf. D2684), so rams prioritize structures capable of shooting arrows over others?
Thu, Jun 4, 3:43 AM · Restricted Project, Contributors

Yesterday

nani added inline comments to D2770: Clean up the data structures in RangeManager .
Wed, Jun 3, 10:41 PM
borg- added a comment to D2682: [gameplay] unify hero health.
In D2682#117489, @Stan wrote:

@borg- thoughts?

Wed, Jun 3, 10:33 PM
borg- added a comment to D2533: [gameplay] train war dogs at barracks.

No, actually not: although kennels are buildable in town phase, war dogs themselves don't have a phase restriction. So if a player starts with or captures a kennel, he can train war dogs immediately in the village phase. I think that makes sense, humans had dogs long before the first towns existed.
While I like the idea of moving them to the house (in fact, I suggested it myself), I'm not sure that's indeed the best option. Dogs require 0 population, hence the current training restrictions. If each house allows training two, then there would be no upper limit; e.g. 100 houses allow an additional force of 200 war dogs. Moreover, houses are usually one of the first structures to be built, and earlier and in much larger numbers than the barracks.

Wed, Jun 3, 10:30 PM
bb added a comment to D2782: [Gameplay] make ram less effective.

IIRC they reason why rams can attack units is that otherwise one can surround a ram by a few units, making it useless. Currently in that situation (at least most of the time), it will attack the surrounding units, trying to break free. Though it already prefers attacking structures over units (so it only attacks units if there are no structures around).

Wed, Jun 3, 10:16 PM · Restricted Project, Contributors
wraitii added a comment to D2770: Clean up the data structures in RangeManager .


What I've used for profiling

Wed, Jun 3, 10:02 PM
wraitii updated the summary of D2770: Clean up the data structures in RangeManager .
Wed, Jun 3, 9:57 PM
wraitii added a comment to D2770: Clean up the data structures in RangeManager .

I've ran a 1vAI and the quick-hashes matched, as expected (full hashes don't match as serialisation is different).
Profiling on the following test map, no obvious difference in LOS computation despite LOS taking time.
A debug-hash rejoin test passed.

Wed, Jun 3, 9:56 PM
Nescio added a comment to D2782: [Gameplay] make ram less effective.

A few more things:

  • The current vision range seems rather excessive; perhaps it should be lowered to 40 or less. What do you think?
  • If rams can no longer attack (organic) units, that makes them significantly less effective, so should its cost be changed?
  • Could you change the <PreferredClasses> to "Gates Defensive Structure", in that order (cf. D2684), so rams prioritize structures capable of shooting arrows over others?
Wed, Jun 3, 8:52 PM · Restricted Project, Contributors
Nescio added a comment to D2686: [gameplay] wonders cost stone instead of food.

For civilizations using stone launchers or slingers, it may be unviable.

D2497
As stated in the summary, I doubled the stone cost to keep the resource cost total unchanged, and because wonders dwarf fortresses, which cost 1000 stone. If you think it's better, I could instead just remove the food cost, keeping the other resources unchanged, thus making the wonder cheaper. I don't really care, what matters to me is food is for units, stone for structures.
D2660 is another wonder patch.

Wed, Jun 3, 8:35 PM
Vulcan added a comment to D2668: [gameplay] deprecate rotary mill, introduce special gaul farming technology.

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

Wed, Jun 3, 8:35 PM
Nescio updated the summary of D2668: [gameplay] deprecate rotary mill, introduce special gaul farming technology.
Wed, Jun 3, 8:31 PM
Nescio updated the diff for D2668: [gameplay] deprecate rotary mill, introduce special gaul farming technology.

15%

Wed, Jun 3, 8:30 PM
Vulcan added a comment to D270: Allow Modifiers to affect tokens..

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

Wed, Jun 3, 7:35 PM
wraitii added inline comments to D270: Allow Modifiers to affect tokens..
Wed, Jun 3, 7:30 PM
wraitii updated the diff for D270: Allow Modifiers to affect tokens..

Fix tests, I stopped halfway through.

Wed, Jun 3, 7:30 PM
Vulcan added a comment to D2702: Fix formations not queueing orders properly..

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

Wed, Jun 3, 7:28 PM
Vulcan added a comment to D2702: Fix formations not queueing orders properly..

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

Wed, Jun 3, 7:27 PM
Harbormaster failed remote builds in B11924: Diff 12118 for D2702: Fix formations not queueing orders properly.!
Wed, Jun 3, 7:27 PM
wraitii updated the diff for D2702: Fix formations not queueing orders properly..

Handle entity renaming explicitly (couldn't actually break it in my tests, but it seems prudent).

Wed, Jun 3, 7:26 PM
wraitii added inline comments to D270: Allow Modifiers to affect tokens..
Wed, Jun 3, 7:23 PM
Freagarach resigned from D270: Allow Modifiers to affect tokens..

I don't see myself finding time to review/test this soon ;( Sorry.

Wed, Jun 3, 7:18 PM
Vulcan added a comment to D2702: Fix formations not queueing orders properly..

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

Wed, Jun 3, 7:17 PM
Harbormaster failed remote builds in B11923: Diff 12117 for D2702: Fix formations not queueing orders properly.!
Wed, Jun 3, 7:17 PM
Vulcan added a comment to D2702: Fix formations not queueing orders properly..

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

Wed, Jun 3, 7:17 PM
wraitii updated the diff for D2702: Fix formations not queueing orders properly..

Rebased & don't rebase on top of D2763 but of master.

Wed, Jun 3, 7:15 PM
wraitii added a comment to D2702: Fix formations not queueing orders properly..

How much different is this from resetting finished orders in CallMemberFunction and setting finished order in FinishedOrder? (Apart from the (un)set in position.)
Reason for asking is the shift of responsibility from the entity to the formation(controller).

Wed, Jun 3, 7:08 PM
Vulcan added a comment to D270: Allow Modifiers to affect tokens..

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

Wed, Jun 3, 7:04 PM
Harbormaster failed remote builds in B11922: Diff 12116 for D270: Allow Modifiers to affect tokens.!
Wed, Jun 3, 7:04 PM
Vulcan added a comment to D270: Allow Modifiers to affect tokens..

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

Wed, Jun 3, 7:04 PM
wraitii updated the diff for D270: Allow Modifiers to affect tokens..

Turns out removing the hack wasn't so hard.

Wed, Jun 3, 7:03 PM
Freagarach added a comment to D2702: Fix formations not queueing orders properly..

How much different is this from resetting finished orders in CallMemberFunction and setting finished order in FinishedOrder? (Apart from the (un)set in position.)
Reason for asking is the shift of responsibility from the entity to the formation(controller).

Wed, Jun 3, 6:58 PM
borg- added a comment to D2668: [gameplay] deprecate rotary mill, introduce special gaul farming technology.

If it is historically a problem, then I must agree with @Genava55.
That it is unique tech to Gaul civilization, the gain could be greater, perhaps 15%? The value seems good to 15%.

Wed, Jun 3, 2:59 PM
borg- accepted D2669: [gameplay] new Gaul team bonus.

I agree with 15%.

Wed, Jun 3, 2:53 PM
borg- added a comment to D2686: [gameplay] wonders cost stone instead of food.

I agree with the changes in the pyramid.
About the wonder I agree in parts. Yes I think the cost of food should be removed, but I think a lot of 2000 stone resources. For civilizations using stone launchers or slingers, it may be unviable.
What about 1500w/1000s/1000m? Seems to me to be a fairer value in cost/benefit.

Wed, Jun 3, 2:39 PM
borg- added a comment to D2782: [Gameplay] make ram less effective.

I will create a <Tooltip> as soon as I get home.
It seems that we agree with the three proposed changes.
We cannot change to "UNIT", it would change the balance of the game a lot. Players use ram to destroy other ram or siege weapons. Some civilizations would be very restricted to melee units, to combat siege weapons, this certainly would not work.

Wed, Jun 3, 2:17 PM · Restricted Project, Contributors
Vulcan added a comment to D2783: [gameplay] allow a few more units on medium and long walls.

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

Wed, Jun 3, 2:11 PM
Vulcan added a comment to D2664: Commit resources at a target rather than per type..

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

Wed, Jun 3, 2:07 PM
Vulcan added a comment to D2781: Maintain the highest-priority real directory when loading mods.

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

Wed, Jun 3, 2:03 PM
wraitii added a comment to D1095: Rename modmod to modselection.

I also agree with this change, but it probably needs to be rebased and merged quickly so that it doesn't end up in merge conflicts everywhere. Might be a good candidate for a post-A24 commit.

Wed, Jun 3, 1:56 PM
wraitii added inline comments to D2664: Commit resources at a target rather than per type..
Wed, Jun 3, 1:54 PM
Nescio created D2783: [gameplay] allow a few more units on medium and long walls.
Wed, Jun 3, 1:54 PM
Vulcan added a comment to D576: Fix VisualActor tech changes not being applied in some cases.

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

Wed, Jun 3, 1:54 PM
wraitii updated the diff for D2664: Commit resources at a target rather than per type..

Fix issue noted by Angen.

Wed, Jun 3, 1:53 PM
wraitii updated the diff for D2781: Maintain the highest-priority real directory when loading mods.

Stan's remarks.

Wed, Jun 3, 1:47 PM
wraitii updated the diff for D576: Fix VisualActor tech changes not being applied in some cases.

Fix issues & hack.
Ran a rejointest, & visual/non visual-replays, all appears in order

Wed, Jun 3, 1:40 PM
wraitii added a comment to D2760: [gameplay] add visible garrison points to gates.

I personally somewhat dislike that we rely so much on garrisoning for defence, so I would keep it limited, but I like how the effect looks, so it's good to have the possibility. The first concern is more related to balancing ofc.

Wed, Jun 3, 1:27 PM
Vulcan added a comment to D2769: [gameplay] add visible garrison points to wall towers.

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

Wed, Jun 3, 1:07 PM
Nescio updated the diff for D2769: [gameplay] add visible garrison points to wall towers.
  • aura
  • cost
Wed, Jun 3, 1:03 PM
Nescio updated the summary of D2769: [gameplay] add visible garrison points to wall towers.
Wed, Jun 3, 12:53 PM
Nescio updated the summary of D2760: [gameplay] add visible garrison points to gates.
Wed, Jun 3, 12:44 PM
Vulcan added a comment to D2763: Fix formations reshaping incorrectly & related issues..

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

Wed, Jun 3, 12:43 PM
Nescio added a comment to D2760: [gameplay] add visible garrison points to gates.

Any opinions on the number of visible garrison slots on gates? In principle they could accomodate more units.

Wed, Jun 3, 12:40 PM
Nescio added a comment to D2782: [Gameplay] make ram less effective.

It doesn't seem to have been changed recently though, so something is weird.

Presumably wowgetoffyourcellphone has an outdated version of the UnitAI file in his mod.

Wed, Jun 3, 12:39 PM · Restricted Project, Contributors
wraitii updated the diff for D2763: Fix formations reshaping incorrectly & related issues..

I had indeed removed FinishOrder in Order.Stop by accident.

Wed, Jun 3, 12:36 PM
wraitii added a comment to D2782: [Gameplay] make ram less effective.

The Ram AI still attempts to attack units around them if there aren't any structures in vision range.

Wed, Jun 3, 12:30 PM · Restricted Project, Contributors
wraitii added a comment to rP23731: Enable garrisoning on gates / fix movement of units with visible garrison….

I believe garrison holder is still setting garrisoned flag for variants and art replays animations on variant change if it has active animation.

Wed, Jun 3, 12:20 PM
Angen added a comment to rP23731: Enable garrisoning on gates / fix movement of units with visible garrison….

hmm, that seems like it's trying to close again when already closed, which shouldn't happen. Perhaps it's an art issue as we end up resetting some animation timer. I'll check.

Wed, Jun 3, 12:17 PM
Nescio added a comment to D2782: [Gameplay] make ram less effective.

I can probably do that myself quickly.

Great!

ramming a ship, if not historically accurate, is probably a good way to make a hole in the hull and sink it.

Ships ramming ships is historically accurate, it was the primary mode of naval warfare in classical times. However, ship rams were static extensions of the keel below the water line, whereas battering rams sling above ground. Moreover, there is the question how battering rams would reach the ships on the water. It's probably better if rams won't attempt attacking ships.

Wed, Jun 3, 12:15 PM · Restricted Project, Contributors
wraitii added a comment to rP23731: Enable garrisoning on gates / fix movement of units with visible garrison….

One minor thing is that locked gates still play an opening-and-closing animation when garrisoning or ungarrisoning units; I suppose that's an art problem?

Wed, Jun 3, 12:12 PM
Nescio added a comment to rP23731: Enable garrisoning on gates / fix movement of units with visible garrison….

Thanks. I rebuilt the game and tried it out in combination with D2760. Everything seems to work fine.
One minor thing is that locked gates still play an opening-and-closing animation when garrisoning or ungarrisoning units; I suppose that's an art problem?

Wed, Jun 3, 12:08 PM
wraitii added a comment to D2782: [Gameplay] make ram less effective.

Now I don't know how the UnitAI works, but if adding this restriction means rams still chase and try attacking units, then I think a much better solution would be to simply reduce their damage vs units sharply; see D2684.

I guess the code is "AttackEntitiesByPreference" and that does seem like it's missing checks for restricted classes. But that just sounds like a UnitAI bug, and I would suggest fixing it instead of going for the above hack. I can probably do that myself quickly.

Wed, Jun 3, 12:06 PM · Restricted Project, Contributors
wraitii added inline comments to D2763: Fix formations reshaping incorrectly & related issues..
Wed, Jun 3, 11:56 AM
Nescio added a comment to D2782: [Gameplay] make ram less effective.

As listed in the summary, this patch does three different things, so let's discuss them in that order.

  • I agree with the speed change. Though if the purpose is to allow pikemen to catch rams, it should be lowered even further.
  • I have no objections to the fields restriction. In fact, I also included that in D2508. Also, animals can't attack structures, ships, or siege engines, so it's not unprecendented.
  • I'm unsure about the organic restriction. The fact that it was present at some point and subsequently removed, as wraitii pointed out, suggests it may not have been an entirely good idea. Furthermore, wowgetoffyourcellphone wrote elsewhere:

In DE, Rams are prevented from attacking units. After years of experience with this change, I can see it's not pathfinding that needs adjusting, it's UnitAI. The Ram AI still attempts to attack units around them if there aren't any structures in vision range. What they need to do probably is just go idle, or attempt to flee (haven't given much thought on what they should do, probably just idle; they aren't support units).

Now I don't know how the UnitAI works, but if adding this restriction means rams still chase and try attacking units, then I think a much better solution would be to simply reduce their damage vs units sharply; see D2684.
Besides, I think it's better to restrict the attack vs Unit instead of Organic; battering rams attacking artillery and ships makes historically as little sense as targetting e.g. cavalry.
Furthermore, correct the indentation and add a <Tooltip> under <Identity>.

Wed, Jun 3, 11:42 AM · Restricted Project, Contributors
Angen added inline comments to D2763: Fix formations reshaping incorrectly & related issues..
Wed, Jun 3, 11:36 AM
Angen added a comment to D2753: Look for foundations near the target.

did not test it but looks good by checking the code

Wed, Jun 3, 11:24 AM
Angen added inline comments to D2664: Commit resources at a target rather than per type..
Wed, Jun 3, 11:08 AM
Nescio added a comment to D2533: [gameplay] train war dogs at barracks.

No, actually not: although kennels are buildable in town phase, war dogs themselves don't have a phase restriction. So if a player starts with or captures a kennel, he can train war dogs immediately in the village phase. I think that makes sense, humans had dogs long before the first towns existed.
While I like the idea of moving them to the house (in fact, I suggested it myself), I'm not sure that's indeed the best option. Dogs require 0 population, hence the current training restrictions. If each house allows training two, then there would be no upper limit; e.g. 100 houses allow an additional force of 200 war dogs. Moreover, houses are usually one of the first structures to be built, and earlier and in much larger numbers than the barracks.

Wed, Jun 3, 10:53 AM
ValihrAnt added a comment to D2782: [Gameplay] make ram less effective.

The patch changes ram speed from 8.1 to 7.2, so from the same speed as an unpacked bolt shooter to the same speed as an unpacked catapult, making it easier for infantry and elephants to catch up to them.
Overall, I'm not sure if rams seem OP due to the ranged unit focused meta and some civs simply not getting good counter units to siege weapons or if they're actually OP. Still, I'm for this change as rams can be buffed again when some other meta changing balancing is done.

Wed, Jun 3, 9:16 AM · Restricted Project, Contributors
wraitii added inline comments to D576: Fix VisualActor tech changes not being applied in some cases.
Wed, Jun 3, 9:05 AM
wraitii added inline comments to D2781: Maintain the highest-priority real directory when loading mods.
Wed, Jun 3, 9:04 AM
Stan added inline comments to D576: Fix VisualActor tech changes not being applied in some cases.
Wed, Jun 3, 9:00 AM
Stan added a comment to D2781: Maintain the highest-priority real directory when loading mods.

It's a fairly standard c pattern -> with unsigned types, "-1" is the max positive value.
The correct change is actually to use std::numeric_limits<size_t>::max() though, so I could change it to that.

Wed, Jun 3, 8:58 AM
wraitii added inline comments to D576: Fix VisualActor tech changes not being applied in some cases.
Wed, Jun 3, 8:55 AM
wraitii added a comment to D2704: Create function to distribute a message to several entities..

As noted by stan -> See D576 for a situation that relies on local entities not sending messages to globally subscribed non-local entities.

Wed, Jun 3, 8:54 AM
wraitii added a comment to D2781: Maintain the highest-priority real directory when loading mods.
In D2781#118060, @Stan wrote:

I assume -1 is to make sure they are always at the top of the list? I would have recommended static_cast but it seems consistent with the lines below. Maybe a comment?

It's a fairly standard c pattern -> with unsigned types, "-1" is the max positive value.
The correct change is actually to use std::numeric_limits<size_t>::max() though, so I could change it to that.

Any reason why it wasn't made a priority at first ?

You mean in past diffs? This comes straight from rP13167, I assume it wasn't deemed necessary or was simply forgotten.

Wed, Jun 3, 8:50 AM
Stan added inline comments to D576: Fix VisualActor tech changes not being applied in some cases.
Wed, Jun 3, 8:50 AM
Stan added a comment to D2781: Maintain the highest-priority real directory when loading mods.

I assume -1 is to make sure they are always at the top of the list? I would have recommended static_cast but it seems consistent with the lines below. Maybe a comment?
Any reason why it wasn't made a priority at first ?

Wed, Jun 3, 8:42 AM
wraitii updated the summary of D2781: Maintain the highest-priority real directory when loading mods.
Wed, Jun 3, 8:05 AM
wraitii added a comment to D2782: [Gameplay] make ram less effective.

Seems like a change for the better.

Wed, Jun 3, 8:04 AM · Restricted Project, Contributors
wraitii added a comment to D1931: Refactor HWDetect and rendering options setup.

I think I'll proceed with this over the WE perhaps, need to remind myself to update the wiki pages and do the few useful changes pointed out.

Wed, Jun 3, 7:50 AM
Angen added a comment to D2782: [Gameplay] make ram less effective.

garrisoned aura

Wed, Jun 3, 7:50 AM · Restricted Project, Contributors
autobuild committed rP23732: [Windows] Automated build..
[Windows] Automated build.
Wed, Jun 3, 7:25 AM
borg- added a comment to D2782: [Gameplay] make ram less effective.

If we garrison units inside, it should increase the speed of the battering ram.

Wed, Jun 3, 1:35 AM · Restricted Project, Contributors
Lionkanzen added a comment to D2782: [Gameplay] make ram less effective.

If we garrison units inside, it should increase the speed of the battering ram.

Wed, Jun 3, 1:24 AM · Restricted Project, Contributors
borg- updated the summary of D2782: [Gameplay] make ram less effective.
Wed, Jun 3, 12:59 AM · Restricted Project, Contributors
borg- created D2782: [Gameplay] make ram less effective.
Wed, Jun 3, 12:49 AM · Restricted Project, Contributors
Lionkanzen accepted D1218: New formation bonus.

ok I will test this.

Wed, Jun 3, 12:41 AM
borg- requested changes to D2533: [gameplay] train war dogs at barracks.

The best idea is train dogs in the houses. This does not the game unbalanced because they are training only in era 2. Can also put a limit of 1 or 2 dogs per house.

Wed, Jun 3, 12:12 AM

Tue, Jun 2

Nescio added a comment to rP23730: Remove inaccurate howdah from the carthaginian champion elephant..

(Strictly speaking it's a tower, not an howdah.)

Tue, Jun 2, 11:40 PM
elexis added inline comments to D1931: Refactor HWDetect and rendering options setup.
Tue, Jun 2, 11:23 PM
wraitii added inline comments to D576: Fix VisualActor tech changes not being applied in some cases.
Tue, Jun 2, 10:55 PM
Vulcan added a comment to D576: Fix VisualActor tech changes not being applied in some cases.

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

Tue, Jun 2, 10:55 PM
wraitii added inline comments to D576: Fix VisualActor tech changes not being applied in some cases.
Tue, Jun 2, 10:53 PM
wraitii updated the diff for D576: Fix VisualActor tech changes not being applied in some cases.

(commandeering, this isn't just a rebase).

Tue, Jun 2, 10:49 PM