Page MenuHomeWildfire Games
Feed All Stories

Today

Stan added inline comments to D3013: Disallow gaia to change state.
Mon, Sep 21, 9:45 PM
bb requested review of D3013: Disallow gaia to change state.
Mon, Sep 21, 9:39 PM
Stan added inline comments to rP24060: Safeguard QueryPlayerIDInterface calls.
Mon, Sep 21, 9:19 PM
bb accepted D2810: Don't process commands sent by observers..

Note that the reveal map cheat does not work anymore for observers after this. Is that bad?

Observers can already see the whole map, slightly bad that observers don't have the retroMe cheat anymore, but well, meh.

Mon, Sep 21, 9:16 PM
bb committed rP24060: Safeguard QueryPlayerIDInterface calls.
Safeguard QueryPlayerIDInterface calls
Mon, Sep 21, 8:55 PM
bb accepted D2654: Allow free Upgrades and/or Technologies..
Mon, Sep 21, 6:36 PM
bb added a comment to D2995: Properly internationalize attack names/Generlize to arbitrary names in the GUI.

Then there is the problem of translating. For instance, whilst ‘sword’ can refer to both single- and double-edged blades in English, this is not true for all languages. In Chinese, jiàn is the word for double-edged swords and dāo the term for single-edged swords (and knives). Thus in those languages “sword” would be wrong in about half of the cases.

This is actually easy to fix, we can add a context tag in the template, which will be extracted already. So I just added the code to pipe the context through and use it for translations. In example: one can put <AttackName context="Double-edged">Sword</AttackName> in the template. On this specific topic: I also added context for all sword wielding units.

Mon, Sep 21, 6:35 PM
bb updated the diff for D2995: Properly internationalize attack names/Generlize to arbitrary names in the GUI.

Back to attackName
add context support
add context for swords
improve some names

Mon, Sep 21, 6:33 PM
Angen updated subscribers of D3009: Refactor UpdateMessageSubscriptions in CCmpRangeOverlayRenderer.
Mon, Sep 21, 2:16 PM
bb accepted D3009: Refactor UpdateMessageSubscriptions in CCmpRangeOverlayRenderer.

Digging back to D555, this code must be ccmpSelectable copy-pasta

Mon, Sep 21, 2:12 PM
Nescio requested review of D3012: rename some fauna templates.
Mon, Sep 21, 2:11 PM
Nescio added a comment to D3011: [art/actors] remove Babylonian lion from Persian barracks.

The barracks already has quite a few props to make it interesting. Besides, the Persian arsenal, range, stable don't have it either.

Mon, Sep 21, 1:23 PM
Stan added a comment to D3011: [art/actors] remove Babylonian lion from Persian barracks.

It's important in the sense that it removes decoration as such makes buildings more historically accurate, but also more bland and less interesting to look at. So when I need to edit buildings I have to take it into account.

Mon, Sep 21, 1:01 PM
Nescio added a comment to D3011: [art/actors] remove Babylonian lion from Persian barracks.

Yes, but that's a minor point. Besides, those don't belong on generic structures either: reliefs of Persians in ceremonial robes are found on several of the monumental structures of the Persepolis palace terrace, as well as on the tomb of Darius and some other rock inscriptions (i.e. all Wonder-quality), but to the best of my knowledge not on any of the buildings found in the lower city of Persepolis or elsewhere.

Mon, Sep 21, 12:39 PM
Nescio added a comment to D2993: [gameplay] add visible garrison points to rome siege wall tower.

To be able to use the lower floor (also for siege wall gate) the actor needs to be edited to raise the platform, because currently the garrison heads go through the roof: https://wildfiregames.com/forum/topic/22779-0abc-mod/page/17/?tab=comments#comment-391719

Mon, Sep 21, 12:33 PM
Nescio added a comment to D1010: introduce subfolders for geology templates.

Good points! I wrote and ran the script, but didn't validate it afterwards to verify the replacements were correct; I should have.
Anyway, I've now updated the sed script. Could you try and check again?

Mon, Sep 21, 12:30 PM
Nescio updated the summary of D1010: introduce subfolders for geology templates.
Mon, Sep 21, 12:27 PM
Freagarach requested changes to D1010: introduce subfolders for geology templates.

Your script needs some fixes.
E.g.

-e 's,geology_metal_alpine,alpine_small,g' \
-e 's,geology_metal_alpine_slabs,alpine_large,g' \

means that first geology_metal_alpine_slabs is replaced with alpine_small_slabs.
Also the gaia-folder is missing in the replacement strings.

Mon, Sep 21, 12:09 PM
Freagarach added a comment to D2993: [gameplay] add visible garrison points to rome siege wall tower.

I guess we need only some opinions from the balancing department.
Unless this:

In D2993#131488, @Stan wrote:

The top of the tower looks very crowded though.

is sufficient reason to hold this off?

Mon, Sep 21, 11:54 AM
Stan added a comment to D3011: [art/actors] remove Babylonian lion from Persian barracks.

That also removes the two characters from the walls

Mon, Sep 21, 11:44 AM
Nescio requested review of D3011: [art/actors] remove Babylonian lion from Persian barracks.
Mon, Sep 21, 11:34 AM
Nescio added a comment to D2993: [gameplay] add visible garrison points to rome siege wall tower.

Anything that needs changing?

Mon, Sep 21, 11:22 AM
Nescio planned changes to D2841: [gameplay] split Celtic civ bonuses.

See also https://wildfiregames.com/forum/topic/24495-celtic-reference/page/8/

Mon, Sep 21, 11:21 AM
Nescio added a comment to D2950: [gameplay] keep population for houses.

This (D2950) and the other (D2848) are two separate patches, with different aims, that can be reviewed independently. If and when one is committed, the other will have to be rebased, though that's unimportant for the discussion here.
This patch initially kept the population for brit and gaul structures, but that exception was later removed, per @borg-.
As for D2848, the purpose of which is to differentiate brit and gaul, it's unclear if and in what form it will end up. There was some more discussion on the forums last month, which indicate that perhaps the brit might keep the structure build time discount and the gauls would get one or more new bonuses (still to be determined). Anyway, that's outside the scope of this patch, and belongs in D2848.

Mon, Sep 21, 11:17 AM
Stan committed rP24059: Add missing credit in 3D work. I missed the fact he was in 2D.
Add missing credit in 3D work. I missed the fact he was in 2D
Mon, Sep 21, 11:05 AM
Nescio added a comment to rP24058: New Kushite Range by @m7600. Small Edits by @Stan.

gui/credits/texts/art.json?

Mon, Sep 21, 11:03 AM
Stan committed rP24058: New Kushite Range by @m7600. Small Edits by @Stan.
New Kushite Range by @m7600. Small Edits by @Stan
Mon, Sep 21, 10:40 AM
Stan committed rP24057: Optimise some textures. Reduce the noise and the size to 512x512, and while at….
Optimise some textures. Reduce the noise and the size to 512x512, and while at…
Mon, Sep 21, 10:02 AM
autobuild committed rP24056: [i18n] Updated POT and PO files..
[i18n] Updated POT and PO files.
Mon, Sep 21, 9:38 AM
Freagarach added a comment to D2382: [WIP] - Building Sockets..

@gameboy No sir, this patch is not ready. In the title it says [WIP] -- meaning that it is a work in progress. Moreover the status of the patch is Changes Planned meaning there is stuff that needs to be done but time (and motivation) to be found.

Mon, Sep 21, 7:33 AM
autobuild committed rP24055: [Windows] Automated build..
[Windows] Automated build.
Mon, Sep 21, 7:31 AM

Yesterday

autobuild committed rP24054: [Windows] Automated build..
[Windows] Automated build.
Sun, Sep 20, 6:21 PM
Stan added a comment to D2967: Cache the model-animation maximum bounds..

Will that break Atlas animation hotloading?

Sun, Sep 20, 12:55 PM
Angen added a comment to rP24053: [Petra/AI]Handle 0 case for template parsing.

Fixes: #5823

Sun, Sep 20, 11:50 AM
Angen committed rP24053: [Petra/AI]Handle 0 case for template parsing.
[Petra/AI]Handle 0 case for template parsing
Sun, Sep 20, 11:49 AM
Angen closed D3010: Handle 0 case for template parsing.
Sun, Sep 20, 11:49 AM
Angen accepted rP23847: Use victory conditions json's in Atlas.
Sun, Sep 20, 11:41 AM
Stan committed rP24052: Fix non power of two texture. .
Fix non power of two texture.
Sun, Sep 20, 11:37 AM
Stan committed rP24051: Fix map serialization bug in rP23847 and rP23881..
Fix map serialization bug in rP23847 and rP23881.
Sun, Sep 20, 11:35 AM
Stan closed D2974: Fix map serialization bug.
Sun, Sep 20, 11:34 AM
badosu added a comment to D2950: [gameplay] keep population for houses.

Given we won't have the building time bonus as per https://code.wildfiregames.com/D2841, what is the justification for removing from britons?

Sun, Sep 20, 11:18 AM
badosu added a comment to D2972: [gameplay] Increase hack damage of pikemen.

Anything that increases the role of melee is a plus for me. My concerns:

Sun, Sep 20, 11:07 AM
gameboy added a comment to D2382: [WIP] - Building Sockets..

@Freagarach Is this patch ready to be released?

Sun, Sep 20, 10:36 AM
Freagarach edited reviewers for D1010: introduce subfolders for geology templates, added: Freagarach; removed: Restricted Owners Package.
Sun, Sep 20, 7:55 AM

Sat, Sep 19

Harbormaster failed to build B13199: rP24050: Add safeguards for all queryOwnerInterface calls for rP24050: Add safeguards for all queryOwnerInterface calls!
Sat, Sep 19, 8:01 PM
bb committed rP24050: Add safeguards for all queryOwnerInterface calls.
Add safeguards for all queryOwnerInterface calls
Sat, Sep 19, 7:56 PM
bb closed D2973: Add safeguards for all queryOwnerInterface calls.
Sat, Sep 19, 7:56 PM
asterix added a comment to D2812: Adds MSAA to anti-aliasing techniques.

Just found this nice post about MSAA that might come in handy here @vladislavbelov https://mynameismjp.wordpress.com/2012/10/24/msaa-overview/

Sat, Sep 19, 12:12 PM
Nescio updated the summary of D1010: introduce subfolders for geology templates.
Sat, Sep 19, 11:01 AM
Nescio added inline comments to rP23890: Enable Macedonian gastraphetēs (crossbowmen) at the siege workshop..
Sat, Sep 19, 10:46 AM
Nescio updated the summary of D1009: introduce subfolders for flora templates.
Sat, Sep 19, 10:43 AM
Nescio updated the diff for D1010: introduce subfolders for geology templates.
  • redone from scratch
Sat, Sep 19, 10:41 AM
Krinkle added a comment to D2827: build-osx-libs: Update SDL to 2.0.12 (fix macOS 10.14+).

@Itms ping :)

Sat, Sep 19, 3:52 AM
Krinkle awarded D3004: Do not display right click tooltip when it is not supported right now a Orange Medal token.
Sat, Sep 19, 3:51 AM
Krinkle awarded rP24024: Clean Formation.js a Love token.
Sat, Sep 19, 3:48 AM
Krinkle abandoned D1617: implement Travis code integration testing.
Sat, Sep 19, 3:47 AM
borg- updated the diff for D2996: [gameplay] Move bireme to village phase and balance..

DefaultArrowCount 1.

Sat, Sep 19, 12:36 AM

Fri, Sep 18

Freagarach added inline comments to rP23890: Enable Macedonian gastraphetēs (crossbowmen) at the siege workshop..
Fri, Sep 18, 10:07 PM
Freagarach accepted D2973: Add safeguards for all queryOwnerInterface calls.
Fri, Sep 18, 3:02 PM
Freagarach added a comment to D2973: Add safeguards for all queryOwnerInterface calls.

This is complete and correct.

Fri, Sep 18, 2:48 PM
Freagarach commandeered D1323: Add Upkeep component..

Forgot,,,

Fri, Sep 18, 1:36 PM
Nescio added a comment to D2953: move various values to specific fauna files.

without [...] this patch makes little sense IMHO.

That's a fair point. This patch does mean more duplication. When writing it, I envisioned the following:

  • move various values to specific fauna files (this patch)
  • move <UnitAI/NaturalBehaviour> and the related <Vision/Range> to specific fauna files
  • reduce the number of generic template_unit_fauna* from the current 25 to perhaps three or four that are meaningful
  • differentiate animal stats (cf. D497, #3778, #3918), affecting gameplay

The first and second are prerequisites of the third and also make the fourth easier. This patch is already quite large, however, if you believe it's better, I suppose I could merge the first three and do all those things here.

Fri, Sep 18, 12:50 PM
Nescio added a comment to D1323: Add Upkeep component..

@Freagarach, since this patch now is completely different from the initial version, you should just commandeer.

Fri, Sep 18, 12:37 PM
Stan added a comment to D1323: Add Upkeep component..

Maybe we should allow specific resources to go negative, eg if one wants to use coin.

Fri, Sep 18, 12:26 PM
Herald added a reviewer for D2917: [gameplay] allow upgrading cart_house to cart_apartment: Balancing.

One could use a technology to swap the house with the apartment in the build list? Meaning the player would have houses and can build apartments thereafter.
E.g. the player builds one house, researches the technology and can only build apartments next.

Fri, Sep 18, 12:22 PM
Freagarach updated the diff for D1323: Add Upkeep component..

Component.

Fri, Sep 18, 10:58 AM
wraitii added a comment to D2922: Fix a compartment mismatch in XmppClient / Possibly fix semi-random lobby / in game JS crash.

Just tested -> couldn't reproduce the crash via launching a map or triggering a shrinking GC on the lobby page.

Fri, Sep 18, 10:55 AM
autobuild committed rP24049: [i18n] Updated POT and PO files..
[i18n] Updated POT and PO files.
Fri, Sep 18, 9:29 AM
Freagarach added a comment to D2953: move various values to specific fauna files.

To me this adds merely a lot of duplication actually. While I am not opposed to moving values to specific templates, without the number of template_unit_fauna_* files can be greatly reduced, keeping only those that are meaningful (e.g. elephant, whale). this patch makes little sense IMHO.
E.g. there are three bears, nicely inheriting from a *bear template, which now all three have the exact same values added. I know that in reality those three bears are vastly different and should have different stats, but until they do I do no think it is necessary to duplicate those values.

Fri, Sep 18, 9:00 AM

Thu, Sep 17

Freagarach added a comment to D1009: introduce subfolders for flora templates.

I'm hoping someone knows a clever way to move and rename all of them at once with a single command while preserving their svn file history.

I would be interested in that as well. In the meantime one could write a bash script like used for the civ-moving. (Notice the first part of each sentence in my itemisation was a command one could use :) )

Thu, Sep 17, 6:48 PM
Nescio added a comment to D1009: introduce subfolders for flora templates.

(svn rename is just an alias to svn mv according to its help function, whereas the rename program can do more.)

Yeah, I figured that out already.

My knowledge is limited, but I would do something along the lines of:

  • find files with name matching flora_tree_*
  • cut that part from the name
  • svn mv the matching files to tree/ with their new name

That's what I want to do, however, there are over 70 such files and I don't really want to svn mv them one by one, so I'm hoping someone knows a clever way to move and rename all of them at once with a single command while preserving their svn file history.

Thu, Sep 17, 6:43 PM
Freagarach edited reviewers for D1009: introduce subfolders for flora templates, added: Freagarach; removed: Restricted Owners Package.
Thu, Sep 17, 6:25 PM
Freagarach added a comment to D1009: introduce subfolders for flora templates.

My knowledge is limited, but I would do something along the lines of:

  • find files with name matching flora_tree_*
  • cut that part from the name
  • svn mv the matching files to tree/ with their new name

(svn rename is just an alias to svn mv according to its help function, whereas the rename program can do more.)

Thu, Sep 17, 6:21 PM
Stan added a comment to D1009: introduce subfolders for flora templates.

Did you svn add tree/ ?

Thu, Sep 17, 6:14 PM
Nescio added a comment to D1009: introduce subfolders for flora templates.

TODO: flora_tree_* → tree/*

The following does what I want:

svn mv flora_tree_* tree/
cd tree/
rename 'flora_tree_' '' flora_tree_*

However, replacing rename with svn rename doesn't work on my end, and without it the svn history is lost. Any suggestions?

Thu, Sep 17, 6:12 PM
Nescio updated the diff for D1009: introduce subfolders for flora templates.
  • redone from scratch
Thu, Sep 17, 5:43 PM
Angen added a comment to D2494: [gameplay] overhaul artillery attacks.

Addition of splash damage story
http://irclogs.wildfiregames.com/2012-02/2012-02-06-QuakeNet-%230ad-dev.log

Thu, Sep 17, 5:20 PM
Itms added a comment to D2922: Fix a compartment mismatch in XmppClient / Possibly fix semi-random lobby / in game JS crash.

Hi there! I rebased https://github.com/na-Itms/0ad/tree/spidermonkey upon this. Please test it, along SM52 🙂 I'll update the forum post whenever I have the chance.

Thu, Sep 17, 2:39 PM
Stan added a comment to D2913: Stop dodging arrows by patrol: Wait a few seconds between patrol waypoints..

http://irclogs.wildfiregames.com/2020-09/2020-09-11-QuakeNet-%230ad-dev.log

Thu, Sep 17, 1:34 PM
Angen added a comment to D2985: [gui] omit resource trickle and loot from attack & resistance tooltip.

instead of removing this information, maybe it could be moved to extended version of tooltip

Thu, Sep 17, 1:27 PM
Nescio added a comment to D2985: [gui] omit resource trickle and loot from attack & resistance tooltip.

That varies from case to case. For attack & resistance specifically, I suppose only combat and action (e.g. healer, gatherer) stats.

Thu, Sep 17, 12:46 PM
Nescio added a comment to D2950: [gameplay] keep population for houses.

Yes, the extra population is removed from the brit, gaul, and iber structures; brit and gaul structures already are built more quickly, whereas iber fortress and tower have significantly more health; see also the D2841 discussion.

Thu, Sep 17, 12:39 PM
Freagarach added a comment to D2985: [gui] omit resource trickle and loot from attack & resistance tooltip.

Agreed, but then there is the question what is ,,only the most important and relevant information'' ;)

Thu, Sep 17, 12:38 PM
Nescio added a comment to D2985: [gui] omit resource trickle and loot from attack & resistance tooltip.

In my opinion the tooltip that opens when right-clicking on a entity icon should list everything (as is already the case), but the smaller ones that pop up when hovering over an icon ought to display only the most important and relevant information, hence this patch.

Thu, Sep 17, 12:35 PM
Nescio updated the diff for D2987: [gameplay] change ship technologies.
  • replace armor_ship_* technologies with different ship_* technologies, as discussed with @borg-
  • delete special_exploration.json and vision_sibylline.json, per @borg-
Thu, Sep 17, 12:32 PM
Freagarach requested changes to D2913: Stop dodging arrows by patrol: Wait a few seconds between patrol waypoints..

See IRC on acceptance-day.

Thu, Sep 17, 12:28 PM
Freagarach added inline comments to D2940: Move some position-related functions to PositionHelper..
Thu, Sep 17, 12:25 PM
Freagarach added a comment to D2985: [gui] omit resource trickle and loot from attack & resistance tooltip.

in the one that opens when right-clicking on an entity icon.

Hmm, maybe that is indeed enough,,,

Thu, Sep 17, 12:08 PM
Stan added a comment to D2950: [gameplay] keep population for houses.

The Brit one did ?

Thu, Sep 17, 11:49 AM
Stan added inline comments to D2940: Move some position-related functions to PositionHelper..
Thu, Sep 17, 11:32 AM
Nescio added a comment to D2985: [gui] omit resource trickle and loot from attack & resistance tooltip.

Perhaps this could be moved to the tooltip of the entity icon?

Which tooltip do you mean exactly?
Loot and resource trickle are already displayed in the one that pops up when hovering over an icon in the structure tree, as well as in the one that opens when right-clicking on an entity icon.

Thu, Sep 17, 11:26 AM
Nescio added a comment to D2802: [gameplay] make fireships ignore anything but ships.

A fireship is “a ship loaded with burning material and explosives and set adrift to ignite and blow up an enemy's ships”. It should not be confused with a warship armed with flamethrowers and Greek fire, as used by the Byzantines in the 7th C AD. Age of Empires II erroneously calls the latter a “Fire Ship”, while their “Demolition Ship” actually resembles a real fireship.

Thu, Sep 17, 11:23 AM
Freagarach added a comment to D2985: [gui] omit resource trickle and loot from attack & resistance tooltip.

Perhaps this should be moved to the tooltip of the entity icon?

Thu, Sep 17, 11:20 AM
Freagarach added a comment to D2551: rename some structure icons.

@Stan did the discussion help you any further?

Thu, Sep 17, 11:18 AM
Nescio updated the diff for D2507: [gameplay] allow building palisades in neutral territory.
  • rebased
Thu, Sep 17, 11:12 AM
Freagarach added a comment to D2477: [gameplay] garrison units on short wall segments.

It isn't pretty, but it works.

Since a key of 0 A.D. is prettiness, I don't think this should be committed.

Thu, Sep 17, 11:11 AM
Nescio added a comment to D2995: Properly internationalize attack names/Generlize to arbitrary names in the GUI.

For simplicity and consistency it might be best if the name node takes exactly the name of its parent, i.e. Capture, Melee, Ranged, or Slaughter. If and when multiple attacks are enabled and can have any name, then the translatable string can be renamed too.

Thu, Sep 17, 11:10 AM
Nescio added inline comments to rP23553: Move Persian cavalry to the persian folder..
Thu, Sep 17, 11:07 AM
Freagarach updated subscribers of D2802: [gameplay] make fireships ignore anything but ships.
In D2802#119810, @borg- wrote:

As far as I know about them, they were Greek and not Iberian ships, but I must be mistaken. It would be interesting to have a historical expert here.

@genava55

Thu, Sep 17, 11:01 AM
Vulcan added a comment to D2940: Move some position-related functions to PositionHelper..

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

Thu, Sep 17, 10:58 AM