Page MenuHomeWildfire Games
Feed All Stories

Wed, Jan 13

Nescio updated the diff for D3350: [gameplay] tweak mace, ptol, sele specific structures.
  • and lighthouse a vision range of 200
Wed, Jan 13, 3:16 PM
Vulcan added a comment to D3350: [gameplay] tweak mace, ptol, sele specific structures.

Build has FAILED

Wed, Jan 13, 3:15 PM
Nescio updated the diff for D3350: [gameplay] tweak mace, ptol, sele specific structures.
  • improve library aura a bit, per @borg-
Wed, Jan 13, 3:14 PM
Nescio added a comment to D2494: [gameplay] overhaul artillery attacks.

Actually I really like the splash concept. However, it has several issues (see summary and earlier posts), hence this patch.
I'd welcome giving e.g. the fireraiser or dragon linear splash and perhaps the plane a circular splash (something for a future patch), but it simply doesn't work that well in practice for trainable entities, which is why I believe this change is a real improvement.

I would say the strongest argument in favour of linear splash damage is realism, since most likely Roman scorpio were used to pierce through several ranks of enemy troops.

Source, please? I suspect that was more the exception than the rule. (Also, several means more than two but not many.)

Wed, Jan 13, 3:09 PM
Angen removed a reviewer for D3350: [gameplay] tweak mace, ptol, sele specific structures: Angen.
Wed, Jan 13, 3:08 PM
borg- added a comment to D3350: [gameplay] tweak mace, ptol, sele specific structures.

I guess the tiny map has a radius of 256. So what do you think is best?

200.

Wed, Jan 13, 3:04 PM
Nescio added a comment to D3350: [gameplay] tweak mace, ptol, sele specific structures.

Maybe increase the range of the lighthouse a little more, like 210?

So I tested how much of a tiny map a lighthouse could reveal.


With a vision range of 200:

With a vision range of 210:

I guess the tiny map has a radius of 256. So what do you think is best?

Since Hellenistic Metropolis is removed of library, what do you think about improving the aura a litle, or move to p2?

No, let's keep it in the city phase. As for the aura, I'm open to suggestions.

Wed, Jan 13, 2:59 PM
borg- added a comment to D3350: [gameplay] tweak mace, ptol, sele specific structures.

Since Hellenistic Metropolis is removed of library, what do you think about improving the aura a litle?

Wed, Jan 13, 2:47 PM
Vulcan added a comment to D3350: [gameplay] tweak mace, ptol, sele specific structures.

Build has FAILED

Wed, Jan 13, 2:47 PM
Nescio added a comment to D3350: [gameplay] tweak mace, ptol, sele specific structures.

I agree with everything

Great!

but I don't think it's logical that "Hellenistic Metropolis" doubles health. Hellenic constructions have +10% health, so why does tech double the value? I don't know if this change should be made in this patch.

Yes, I fully agree with you, the technology seems far to strong. However, I would prefer to see it changed in a separate patch, partially because I'm unsure what would be appropiate values.

Maybe increase the range of the lighthouse a little more, like 210?

That would mean a 36% larger area than it currently has.

Wed, Jan 13, 2:47 PM
borg- added a comment to D3350: [gameplay] tweak mace, ptol, sele specific structures.

I agree with everything, but I don't think it's logical that "Hellenistic Metropolis" doubles health. Hellenic constructions have +10% health, so why does tech double the value? I don't know if this change should be made in this patch.
Maybe increase the range of the lighthouse a little more, like 210?

Wed, Jan 13, 2:41 PM
Stan added a comment to D3320: [gameplay] Add a carnyx player for the gauls.

Assembly changes are just to make it easier to rebase, I just took your file +borg-'s comments

Wed, Jan 13, 2:41 PM
wraitii added a comment to D2494: [gameplay] overhaul artillery attacks.

Seems the splash damage removal is very contentious for the bolt shooters. I'm not a fan on several grounds, technical outlined above, but I'll add that it's easily countered by putting units in 'skirmish' formations when walking.
It's also really hard to balance correctly, and the unit will tend to be either useless or overpowered, which isn't great.
I would say the strongest argument in favour of linear splash damage is realism, since most likely Roman scorpio were used to pierce through several ranks of enemy troops. But our units don't really fight in formation, and walking in formation is also somewhat defeated by the skirmish formation.
Overall, I think it's better for gameplay if those units are treated as snipers than as mass-control, both being, I think, rather equally viable usage in ancient times.

Wed, Jan 13, 2:33 PM
Nescio requested review of D3350: [gameplay] tweak mace, ptol, sele specific structures.
Wed, Jan 13, 2:32 PM
borg- added a comment to D3344: [gameplay] enable gaul assembly.

I would like to see the three infantry champions trained on assembly and removal of unlock tech.

Wed, Jan 13, 2:32 PM
borg- added a comment to D3344: [gameplay] enable gaul assembly.

Why? It is a hall for princes, so it needs some blingbling ;)

Standard cost for gaul buildings.

Wed, Jan 13, 2:30 PM
borg- added a comment to D3345: [gameplay] adjust some siege engine stats.

And what values would you recommend for the fireraiser's <xp> and health?

Wed, Jan 13, 2:27 PM
Nescio added a comment to D3345: [gameplay] adjust some siege engine stats.

And what values would you recommend for the fireraiser's <xp> and health?

Wed, Jan 13, 2:23 PM
Vulcan added a comment to D2528: Performance improvements to VertexBuffer.

Build has FAILED

Wed, Jan 13, 2:23 PM
borg- added a comment to D3345: [gameplay] adjust some siege engine stats.

By the way, the siege tower is quite costly (500 wood, 300 metal, 40 seconds); what do you think of giving it a bit more health (e.g. 500→600)?

No, it is preferable to decrease cost.

Wed, Jan 13, 2:16 PM
Stan added a comment to D3346: Removes CPU lighting after no FFP.

Will it make a profileable difference?

Wed, Jan 13, 2:15 PM
Angen added a comment to D3347: Fix structure tree parsing wrong templates.

first, thank you for feedback :)

Wed, Jan 13, 2:07 PM
wraitii added a comment to D3287: [gameplay] Rework forge armor techs..

Elephants are very large animals, I'm not sure how giving them a shield would work. Anyway, that's just nitpicking, not a major objection against this patch.

It could be argued that this applies to the elephant's rider, which is also somewhat vulnerable. I think unless we have a strong reason not to include them (such as specific techs for elephants), it's probably better to.

Wed, Jan 13, 2:04 PM
wraitii updated the diff for D3278: UnitAI walk orders: stop trying only when very obstructed..

Rebased. Planning to merge still as the current behaviour can look buggy, even if it's not really problematic.

Wed, Jan 13, 2:01 PM
Freagarach updated the Trac tickets for D2171: Avoid GUI errors for captured buildings with paired techs of a different civ.
Wed, Jan 13, 1:33 PM
vladislavbelov resigned from rP24228: Allow registering hooks that trigger on config changes..
Wed, Jan 13, 1:27 PM
vladislavbelov resigned from rP17737: init configDB with renderer initial values, refs #3737.
Wed, Jan 13, 1:27 PM
Vulcan added a comment to D2171: Avoid GUI errors for captured buildings with paired techs of a different civ.

Build has FAILED

Wed, Jan 13, 1:18 PM
Freagarach updated the diff for D2171: Avoid GUI errors for captured buildings with paired techs of a different civ.

A bit more elaborate fix, though still a tad ugly.

Wed, Jan 13, 1:15 PM
Nescio added a comment to rP24591: Fix typo in r24589 / rP24589 after rebasing..

It was already used by another technology available at the same structure.

Wed, Jan 13, 12:39 PM
Stan added a comment to D3302: Upgrade ship_movement_speed / tower_health icons.

Icons should be good now, this needs a rebase.

Wed, Jan 13, 12:38 PM
Stan updated subscribers of rP24591: Fix typo in r24589 / rP24589 after rebasing..

Just for the record @borg- @Nescio what was wrong with the original icon?

Wed, Jan 13, 12:38 PM
Freagarach added a comment to D3329: [gameplay] alter ptol structure civ bonus.

No hard feelings. We all make mistakes, especially under pressure. As long as we can point them out to eachother respectfully it's good :) (For me at least.)

Wed, Jan 13, 12:28 PM
s0600204 added a comment to D3347: Fix structure tree parsing wrong templates.

check i did not break something else

Wed, Jan 13, 12:15 PM
Freagarach added inline comments to rP24589: [Gameplay A24] - Change nisean_horses technology..
Wed, Jan 13, 12:13 PM
Freagarach added a comment to rP24589: [Gameplay A24] - Change nisean_horses technology..

Differential revision: D2856.

Wed, Jan 13, 12:11 PM
Vulcan added a comment to D2528: Performance improvements to VertexBuffer.

Build has FAILED

Wed, Jan 13, 12:05 PM
Nescio added a comment to D3287: [gameplay] Rework forge armor techs..

Elephants are very large animals, I'm not sure how giving them a shield would work. Anyway, that's just nitpicking, not a major objection against this patch.
The technology description strings need to be corrected, though.

Wed, Jan 13, 12:02 PM
Stan updated the diff for D2528: Performance improvements to VertexBuffer.

Fix condition merging

Wed, Jan 13, 12:02 PM
Nescio added a comment to D2978: [WIP] - Update tooltips after rP24001..

Technology tooltips were checked and corrected in D3338/rP24564, so you should rebase this and focus on doing the armour→resistance correction in aura descriptions.

Wed, Jan 13, 12:00 PM
Vulcan added a comment to D2528: Performance improvements to VertexBuffer.

Build is green

Wed, Jan 13, 11:58 AM
borg- added a comment to D3287: [gameplay] Rework forge armor techs..

I don't see any major problems. like any other champion, it seems fair to me to be affected by techs.

Wed, Jan 13, 11:52 AM
Nescio added a comment to D3345: [gameplay] adjust some siege engine stats.

The fireraiser also needs health and xp values.
By the way, the siege tower is quite costly (500 wood, 300 metal, 40 seconds); what do you think of giving it a bit more health (e.g. 500→600)?
And perhaps ram metal cost can be lowered a bit (e.g. 200→150)? A tower costs only 100 wood + 100 metal.

Wed, Jan 13, 11:50 AM
Vulcan added a comment to D2674: [gameplay] make foundations not ConquestCritical.

Build has FAILED

Wed, Jan 13, 11:47 AM
Nescio added a comment to D2494: [gameplay] overhaul artillery attacks.

Maybe one thing I would like to see -> your call @Nescio , is to add tooltips saying "good vs Buildings" / "good vs Units" ? Maybe too obvious?

If that's done for these two, it should also be done for other units and structures. I vaguely recall having had this discussion perhaps two years ago, but I forgot where and with whom. My main objection is that displaying more text in tooltips in game makes it less likely people will read everything, and also risks drowning more important information.
In my opinion loading screen tips are a better place for discussing the functions of structures and unit types; I'd say it's outside the scope of this patch, though.

Wed, Jan 13, 11:45 AM
Vulcan added a comment to D2528: Performance improvements to VertexBuffer.

Build has FAILED

Wed, Jan 13, 11:39 AM
wraitii updated the diff for D2674: [gameplay] make foundations not ConquestCritical.

Rebased for commit.

Wed, Jan 13, 11:38 AM
Vulcan added a comment to D3344: [gameplay] enable gaul assembly.

Build has FAILED

Wed, Jan 13, 11:36 AM
wraitii committed rP24594: Fix xmlvalidator following rP24215.
Fix xmlvalidator following rP24215
Wed, Jan 13, 11:33 AM
wraitii closed D3349: Fix xmlvalidator gollowing rP24215.
Wed, Jan 13, 11:33 AM
wraitii accepted D2494: [gameplay] overhaul artillery attacks.

To be honest, I would like to remove linear splash damage. It's complex to implement (particularly in JS), gives no obvious feedback to the user graphically, and it's kind of broken (the ranges are very approximate right now). For stuff like bolt shooters, we should instead let the projectile continue its course and hit units along the way.
Of course, that would require a few other changes in general.

Wed, Jan 13, 11:31 AM
borg- added a comment to D3345: [gameplay] adjust some siege engine stats.

300 wood, 150 metal 30 secs?

Wed, Jan 13, 11:27 AM
Nescio removed a reviewer for D3344: [gameplay] enable gaul assembly: Angen.
Wed, Jan 13, 11:19 AM
Nescio closed D2856: [gameplay] redo nisean_horses technology.

Committed as rP24589 by @Freagarach.

Wed, Jan 13, 11:18 AM
Vulcan added a comment to D3326: Make WxWidgets High-DPI aware / Upgrade WXWidgets to 3.1.4 on MacOS.

Build has FAILED

Wed, Jan 13, 11:16 AM
Nescio added a comment to D3287: [gameplay] Rework forge armor techs..

Elephants don't benefit from resistance technologies currently, but will with this patch. Is that good or bad?

Wed, Jan 13, 11:15 AM
Stan updated the diff for D2528: Performance improvements to VertexBuffer.

Rebase, add missing includes/reorder them { on new line

Wed, Jan 13, 11:13 AM
Nescio added a comment to D3329: [gameplay] alter ptol structure civ bonus.

Well, I'm glad someone looks critically at the actual modifications! I noticed I'm making quite a few mistakes lately, probably the result of writing many patches in quick succession. Rushing things is risky, I know.

Wed, Jan 13, 11:11 AM
Nescio updated the diff for D3344: [gameplay] enable gaul assembly.
Wed, Jan 13, 11:09 AM
Nescio added a comment to D3345: [gameplay] adjust some siege engine stats.

Maybe something like 50%?

That would mean 200 for the ram and 250 for the siege tower. Fine by me, but in keep in mind that's enough for a basic soldier to promote to elite rank.

fireraise appears on the "survival" map, so it seems interesting to include.

It needs a cost then (resources and time). Any recommendations?

Wed, Jan 13, 11:05 AM
wraitii committed rP24593: Fix shadows-on-water option..
Fix shadows-on-water option.
Wed, Jan 13, 11:04 AM
wraitii closed D3348: Fix shadows-on-water option..
Wed, Jan 13, 11:04 AM
wraitii requested verification of rP24228: Allow registering hooks that trigger on config changes..
Wed, Jan 13, 11:04 AM
Nescio requested changes to D3320: [gameplay] Add a carnyx player for the gauls.

Let's do the assembly changes in D3344, not here.

Wed, Jan 13, 11:03 AM
wraitii retitled D3326: Make WxWidgets High-DPI aware / Upgrade WXWidgets to 3.1.4 on MacOS from Upgrade WXWidgets to 3.1.4 on MacOS to Make WxWidgets High-DPI aware / Upgrade WXWidgets to 3.1.4 on MacOS.
Wed, Jan 13, 11:03 AM
borg- added a comment to D3345: [gameplay] adjust some siege engine stats.

Maybe something like 50%?
fireraise appears on the "survival" map, so it seems interesting to include.

Wed, Jan 13, 11:03 AM
wraitii updated the diff for D3326: Make WxWidgets High-DPI aware / Upgrade WXWidgets to 3.1.4 on MacOS.

This fixes the scaling, per https://wildfiregames.com/forum/topic/28249-trunk23678-atlas-editor-the-map-simulation-displayed-at-14-bottom-left-corner-of-its-canvas-osx/?tab=comments#comment-398428 .

Wed, Jan 13, 11:01 AM
Nescio added a comment to D3345: [gameplay] adjust some siege engine stats.

Shouldn't the xp loot of ram and siege_tower be bigger since in theory you need fight with the soldiers inside?

Perhaps. I assume <xp> depends on how dangerous an unit is, e.g. ranged soldiers are much easier to kill than melee soldiers, but have the same <xp>, presumably because their long attack range makes them more effective when massed; the same applies to artillery.
I have no objections to giving rams and siege towers more <xp>, but it shouldn't be 100% of health. What value would you recommend for them?

Fireraiser should be included?

The fireraiser doesn't have a cost, so I don't think giving it a loot is appropiate.

Wed, Jan 13, 10:56 AM
vladislavbelov accepted D3348: Fix shadows-on-water option..
Wed, Jan 13, 10:50 AM
Freagarach added a comment to D2171: Avoid GUI errors for captured buildings with paired techs of a different civ.

@Imarok's finding can probably be fixed by not showing techs from another civ.

Wed, Jan 13, 10:46 AM
OptimusShepard added a comment to D2528: Performance improvements to VertexBuffer.

@vladislavbelov any plans to commit this improvement?

Wed, Jan 13, 10:28 AM
Vulcan added a comment to D3225: Stop dodging arrows by spamclicking or patrol: Change the targetting code to account for "end of path".

Build is green

Wed, Jan 13, 10:21 AM
Stan committed rP24592: Fix issue with duplicate tags. Perl script doesn't like it..
Fix issue with duplicate tags. Perl script doesn't like it.
Wed, Jan 13, 10:15 AM
wraitii requested review of D3349: Fix xmlvalidator gollowing rP24215.
Wed, Jan 13, 10:14 AM
wraitii updated the diff for D3225: Stop dodging arrows by spamclicking or patrol: Change the targetting code to account for "end of path".

Angen's note.

Wed, Jan 13, 10:13 AM
Harbormaster failed to build B14982: rP24591: Fix typo in r24589 / rP24589 after rebasing. for rP24591: Fix typo in r24589 / rP24589 after rebasing.!
Wed, Jan 13, 10:06 AM
Freagarach added inline comments to D3335: Do not set formation members idle when their controller is not..
Wed, Jan 13, 10:06 AM
Freagarach committed rP24591: Fix typo in r24589 / rP24589 after rebasing..
Fix typo in r24589 / rP24589 after rebasing.
Wed, Jan 13, 10:01 AM
Vulcan added a comment to D3335: Do not set formation members idle when their controller is not..

Build is green

Wed, Jan 13, 9:57 AM
wraitii retitled D3335: Do not set formation members idle when their controller is not. from [WIP] - Do not set formation members idle when their controller is not. to Do not set formation members idle when their controller is not..
Wed, Jan 13, 9:52 AM
wraitii updated the diff for D3335: Do not set formation members idle when their controller is not..

Slight tweak.
I think this works well enough for A24

Wed, Jan 13, 9:52 AM
wraitii requested review of D3348: Fix shadows-on-water option..
Wed, Jan 13, 9:51 AM
Freagarach added a reviewer for D3342: simulation templates: purge unnecessary attack lines: Freagarach.
Wed, Jan 13, 9:33 AM
Vulcan added a comment to D3307: Allow hotkeys to be undefined..

Build is green

Wed, Jan 13, 9:30 AM
wraitii requested verification of rP24462: Correctly serialize/deserialize user-defined JS objects..
Wed, Jan 13, 9:30 AM
wraitii updated the diff for D3307: Allow hotkeys to be undefined..

Address comments.

Wed, Jan 13, 9:22 AM
wraitii added inline comments to D3307: Allow hotkeys to be undefined..
Wed, Jan 13, 9:16 AM
Vulcan added a comment to D3347: Fix structure tree parsing wrong templates.

Build is green

Wed, Jan 13, 9:10 AM
Angen updated the summary of D3347: Fix structure tree parsing wrong templates.
Wed, Jan 13, 9:06 AM
Angen published D3347: Fix structure tree parsing wrong templates for review.
Wed, Jan 13, 9:06 AM
Angen added inline comments to rP23808: Rewrite Structure Tree and Template Viewer to use OOP principles.
Wed, Jan 13, 8:59 AM
vladislavbelov added inline comments to D3346: Removes CPU lighting after no FFP.
Wed, Jan 13, 8:47 AM
Stan added inline comments to D3346: Removes CPU lighting after no FFP.
Wed, Jan 13, 8:33 AM
vladislavbelov added a comment to D3340: Removes GLSL shaders code duplication for fog and shadows.
In D3340#148586, @Stan wrote:

Should h files get copyright headers?

I'm not sure, shaders don't have copyright headers.

Wed, Jan 13, 8:30 AM
vladislavbelov added inline comments to D3346: Removes CPU lighting after no FFP.
Wed, Jan 13, 8:25 AM
Freagarach added a comment to D2674: [gameplay] make foundations not ConquestCritical.
In D2674#118627, @Stan wrote:

arc diff --update a diff you don't own :)

Or change the URL when updating via the web interface.

Wed, Jan 13, 8:02 AM
Freagarach committed rP24590: [Gameplay A24] - Alter ptol structure bonus..
[Gameplay A24] - Alter ptol structure bonus.
Wed, Jan 13, 8:01 AM
Freagarach closed D3329: [gameplay] alter ptol structure civ bonus.
Wed, Jan 13, 8:01 AM
Freagarach added inline comments to D3329: [gameplay] alter ptol structure civ bonus.
Wed, Jan 13, 7:59 AM
Freagarach committed rP24589: [Gameplay A24] - Change nisean_horses technology..
[Gameplay A24] - Change nisean_horses technology.
Wed, Jan 13, 7:54 AM