Page MenuHomeWildfire Games
Feed Advanced Search

Today

Nescio added a comment to D2056: Support for more garrisoned slots per entity..

What does gui/common/tooltips.js then?

Mon, Jul 22, 6:27 PM
Nescio added a comment to D2056: Support for more garrisoned slots per entity..

Apparently I'm doing something wrong then. I gave cavalry a garrisonable size of 3, but it doesn't show up in the techtree:


nor in the in-game extended tooltip (opened when clicking on icon):

Mon, Jul 22, 6:12 PM
Nescio added a comment to D2056: Support for more garrisoned slots per entity..

Could you show a screenshot of the garrisonable size tooltip?

Mon, Jul 22, 4:15 PM
Nescio added a comment to D2106: Move hardcoded gain per garrisoned trader on ship to template.

By the way, do fishing boats get a gather bonus from garrisoning a worker inside? I vaguely recall reading that somewhere (might have been a previous alpha, non-implemented feature, or mod), but if it's implemented, it should probably be moved to its xml template as well.

Mon, Jul 22, 1:39 PM
Nescio added a comment to D2110: manual/intro.txt corrections.

Not implemented, but worth considering: use (space en-dash space) instead of : (colon space) to separate hotkey and description, e.g.

Mon, Jul 22, 1:33 PM
Nescio updated the diff for D2110: manual/intro.txt corrections.

Display backslash (\ won't show up, \\ does) and add names of other punctuation symbols for better visibility.

Mon, Jul 22, 1:26 PM
Nescio created D2110: manual/intro.txt corrections.
Mon, Jul 22, 1:09 PM

Yesterday

Nescio updated the diff for D2083: territory pull again.

Per @elexis' earlier remarks.

Sun, Jul 21, 9:32 PM
Nescio added a comment to D2106: Move hardcoded gain per garrisoned trader on ship to template.

Yes, a merchant ship with 15 traders inside generates four times as much resources as an empty one. Or to put it differently, garrisoning traders in merchant ships is only profitable if an empty merchant ship generates more than five times as much as a trader walking its own route.
A much cheaper option is to simply construct additional merchant ships. (Four empty merchant ships cost 4×100=400 metal in total; one merchant ship with 15 traders inside costs 100+15×80=1300 metal and 15×100=1500 food in total.)

Sun, Jul 21, 9:24 PM
Nescio added a comment to D2107: Clean var values from Trader.js.

Out of curiosity, what's the difference between var and let? And if it should be let, then why do so many files have var? (Try running grep -cr 'var ' in the ai, components, and helpers folders.)

Sun, Jul 21, 4:09 PM
Nescio added a comment to D1808: standardized hero aura descriptions.

The aura description formats should be clear and unambiguous.

garrisoned type: Garrisoned [class] [change] [attributes]

Agreed.

garrison type: Hosting [class] [change] [attributes]

E.g. “Hosting warship +10% health.” Would the uninformed player understand that means the hero is to be garrisoned?

formation type: Formation [class] [change] [attributes] or Accompanying [class] [change] [attributes]

E.g. “Formation spearmen +10% health.” Wouldn't that imply all spearmen in any formation, rather than only spearmen in the formation containing the aura-entity (e.g. hero)?
E.g. “Accompanying spearmen +10% health.” Again, how is the player supposed to know that means only spearmen in the hero's formation?

Sun, Jul 21, 3:49 PM
Nescio added a comment to D2106: Move hardcoded gain per garrisoned trader on ship to template.

True, neither traders nor merchant ships are capturable in 0 A.D.
I'm wondering though what would be the most logical place to define this bonus: the entity template that receives it or the entity template that gives it?

Sun, Jul 21, 3:36 PM
Nescio added a comment to D1808: standardized hero aura descriptions.

I think we should avoid trailing zeroes on user-visible text for now.

Agreed. In that case the GUI should probably be edited to not add any trailing zeroes for e.g. armour either (in a different patch, of course).

should classes be capitalized or not?

I would postpone this decision as well.

I don't think postponing that decision is a good idea. Everytime a string is updated, the entire string has to be re-translated again for all languages, which means that changing capitalization later doubles the work-load for our translators. Besides, the purpose of this patch is to improve consistency, and a policy to keep it uncapitalized in some data files while demanding it to be capitalized elsewhere doesn't seem very consistent.

Sun, Jul 21, 3:32 PM
Nescio added a comment to D2106: Move hardcoded gain per garrisoned trader on ship to template.

Shouldn't it be Trader-based instead of Merchant-based? That would allow e.g. gaul trader adds +0.15, mace trader +0.20, pers trader +0.25.

Sun, Jul 21, 3:05 PM
Nescio added a comment to D2083: territory pull again.

rP16022 broke the pull templates? And the wall templates fixed by rP22070 and the trigger templates fixed by rP17270? But then why does Josh say in #3613 that it was working in Alpha 18 (released march 2015 after rP16022, see Changelogs)?

No idea when exactly it was broken, but try placing other/territory_pull.xml in A23 in Atlas and you get these errors:

ERROR: RelaxNGValidator: Validation error: preview%7Cother/territory_pull:1: Expecting an element RetainInFog, got nothing
Sun, Jul 21, 10:33 AM
Nescio added a comment to D1808: standardized hero aura descriptions.

Things to be decided upon before I'll update this patch:

  • +1 or +1.0 armour?
  • should classes be capitalized or not?
  • formation, garrison, and garrisoned aura description format.
Sun, Jul 21, 10:27 AM
Nescio added a comment to D1807: standardized catafalque aura descriptions.

Let's keep those sentences in the catafalque aura descriptions then for now.

Sun, Jul 21, 10:25 AM
Nescio added inline comments to D1806: standardized structure aura descriptions.
Sun, Jul 21, 10:24 AM

Sat, Jul 20

Nescio added a comment to D1807: standardized catafalque aura descriptions.

Also, the question remains whether it's right to remove the history flavour sentence from these catafalque aura descriptions (other aura types don't have that).

Sat, Jul 20, 8:42 PM
Nescio added a comment to D1808: standardized hero aura descriptions.

Looking at the April 1 version:
formation type: [class] in his formation [change] [attributes]
garrison type: [class] he is garrison in [change] [attributes]
garrisoned type: Garrisoned [class] [change] [attributes]

Sat, Jul 20, 8:21 PM
Nescio added a comment to D1808: standardized hero aura descriptions.

As for +3.0 vs +3, I have no strong preference, provided it is done consistently. The gui displays armour with one decimal, though:

Sat, Jul 20, 8:06 PM
Nescio added a comment to D1807: standardized catafalque aura descriptions.

Thanks for the notification, I missed that one; I'll reply over there.

Sat, Jul 20, 7:54 PM
Nescio updated the diff for D2083: territory pull again.

template_territory_pull.xml

Sat, Jul 20, 6:54 PM
Nescio added a comment to D1042: rename maps/ file names to use underscores (instead of spaces and capitalization).

Those height maps aren't used, their copyright is unclear, and none of their commit messages says anything about them.
I just opened the three maps/scenarios/Height Map Import - Demo * files and they're all three blank maps without any entities. I'm not sure what they demonstrate; if one wants to know how to import height maps I suppose they could have a look at e.g. maps/random/hellas.js. So perhaps delete all those Height Map scenario files (png, xml, and pmp)?

Sat, Jul 20, 6:47 PM
Nescio added a comment to D1042: rename maps/ file names to use underscores (instead of spaces and capitalization).

If I understand you correctly, you suggest those heightmaps to be deleted, but want to know on what grounds exactly?

Sat, Jul 20, 6:10 PM
Nescio added a comment to D2083: territory pull again.

template_pull.xml -> template_territory_pull.xml and pulls/territory_60.xml -> territory_pulls/territory_pull_60.xml?

The parent template doesn't actually have a territory component, because that's specified in its children, so wouldn't it be a bit weird to name it template_territory_pull.xml? Otherwise I have no objections.

Sat, Jul 20, 6:08 PM

Fri, Jul 19

Nescio added a comment to D2100: Make cart team bonus actually be +10% instead + 0.1.

Yes, I'd like to have one; no idea how to do it myself, though. Perhaps you could help?

Fri, Jul 19, 11:26 AM
Nescio added a comment to D2100: Make cart team bonus actually be +10% instead + 0.1.

Thanks! Found it, simulation/components/Trader.js line 5:

// Additional gain for ships for each garrisoned trader, in percents
const GARRISONED_TRADER_ADDITION = 20;

Wouldn't it be better to define that value inside the trader template, rather than hard-code it in this javascript component?

Fri, Jul 19, 11:15 AM
Nescio added a comment to D2100: Make cart team bonus actually be +10% instead + 0.1.

Currently I'm looking at the template_unit_ship_merchant.xml's tooltip. Is the “Garrison a Trader aboard for additional profit (+20% for each garrisoned).” part actually implemented? If so, where? If not, wouldn't it be better to remove that sentence then?

Fri, Jul 19, 11:00 AM
Nescio added a comment to D2100: Make cart team bonus actually be +10% instead + 0.1.

Please have a careful look before changing this.
This aura affects entities with the “Market” class; doing a grep -r Market reveals we should look at the following templates:

campaigns/campaign_city_minor_test.xml
campaigns/campaign_city_test.xml
template_structure_economic_market.xml
template_structure_military_dock.xml
structures/cart_dock.xml
structures/cart_market.xml

The first and second don't actually have a <Market> node, the third and fourth have:

<Market>
  <TradeType>land naval</TradeType>
  <InternationalBonus>0.2</InternationalBonus>
</Market>

to which the fifth and sixth add:

<Market>
  <InternationalBonus op="add">0.1</InternationalBonus>
</Market>

Then we also might want to have a look at simulation/data/technologies/trade_commercial_treaty.json, which has:

	"tooltip": "Market +10% International Bonus.",
	"modifications": [
		{ "value": "Market/InternationalBonus", "add": 0.1 }
	],
	"affects": ["Market"],

As far as I understand it—please correct me if I'm mistaken—the international bonus is a bonus based on what you'd get when trading with your own markets. In other words, 0.1 means 10% more than non-international trade.
What this proposed patch actually does is increase the bonus by 0.02 (= 10% of 0.2), i.e. 2% of normal gain, rather than the 10% it does now.
So if something is to be changed, then probably the tooltips, not the values.

Fri, Jul 19, 10:45 AM

Thu, Jul 18

Nescio added a comment to D2044: Allow immunity to certain damage types..

[EDIT]: Seems to me we need a definition of restrictedClasses ;)

No, not really; e.g. spearmen inflict both hack and pierce damage; they should be able to attack structures even if structures would be immune to pierce damage.

Thu, Jul 18, 5:28 PM
Nescio added a comment to D2051: Display wonders alongside heroes and relics.

See also D1397.

Thu, Jul 18, 1:58 PM
Nescio added a comment to D2007: standardize formation tooltips.

Thanks! D2006 is a similar patch, perhaps you're interested in reviewing that one as well?

It's a bit unfortunate that tooltips speaking of classes are in formation templates, whereas it's actually unit templates that decide what formations they can have.

Is it a good idea to change that? Not sure.
Sounds like that should be changed, formation templates should imo define what units can go in (as the formation parents the units).

Yes, that would probably an improvement.

Thu, Jul 18, 9:19 AM

Wed, Jul 17

Nescio updated the diff for D2007: standardize formation tooltips.

Rephrased skirmish formation tooltip, per @bb.

Wed, Jul 17, 9:50 PM
Nescio added a comment to D2022: enable archery tradition for kush.

Perhaps you can add a reference to the source? For future reference and may help to persuade people ;)

Herodotus Histories book II is about Egypt and Kush (“Aithiopia”).
For the Ethiopians in Xerxes' army, see Herodotus VII.69-70.

Wed, Jul 17, 7:13 PM
Nescio added a comment to D1904: Move fauna_arctic_wolf_attack.xml from trigger/ to gaia/.

(If you're interested, D1881 is similar, since all other wallsets are inside structures/.)

Wed, Jul 17, 4:08 PM
Nescio updated the diff for D2083: territory pull again.

Now without vision files (adding those is perhaps not a good idea).

Wed, Jul 17, 4:00 PM
Nescio added a comment to D1904: Move fauna_arctic_wolf_attack.xml from trigger/ to gaia/.

No idea; I stumbled upon this when viewing triggers/ files because of territory pull discussion.

Wed, Jul 17, 3:59 PM
Nescio updated the diff for D1904: Move fauna_arctic_wolf_attack.xml from trigger/ to gaia/.

Moved to gaia/fauna_arctic_wolf_domestic.xml

Wed, Jul 17, 3:42 PM
Nescio updated the summary of D1042: rename maps/ file names to use underscores (instead of spaces and capitalization).
Wed, Jul 17, 3:35 PM
Nescio updated the test plan for D1042: rename maps/ file names to use underscores (instead of spaces and capitalization).
Wed, Jul 17, 3:35 PM
Nescio added a comment to D1042: rename maps/ file names to use underscores (instead of spaces and capitalization).

Agreed, let's delete those pngs then; updated script:

Wed, Jul 17, 3:30 PM
Nescio updated the summary of D1042: rename maps/ file names to use underscores (instead of spaces and capitalization).
Wed, Jul 17, 10:55 AM
Nescio updated the test plan for D1042: rename maps/ file names to use underscores (instead of spaces and capitalization).
Wed, Jul 17, 10:54 AM
Nescio updated the summary of D1042: rename maps/ file names to use underscores (instead of spaces and capitalization).
Wed, Jul 17, 10:53 AM
Nescio added a comment to D1042: rename maps/ file names to use underscores (instead of spaces and capitalization).

Updated script:


Which removes Flight_demo.* and Sandbox - Ptolemies.* (because their second version looks much better), takes care of Azure Coast* scenarios, then performs the earlier agreed-upon map moves (removes hyphens and parentheses, replaces spaces with underscores, converts upper case into lower case).

what to do with the scenarios/Height Map *.png files

The question is which purpose do they serve.
If they're in the public.zip, I suppose players dont actually see the files?
Can post copies of the files in some place that people look up, wiki page, forum post, trac ticket, and or phabricator page and or commit to art repository.

I don't know what their purpose is. Where should they be moved to exactly? Or do you think they should simply be deleted?

Wed, Jul 17, 10:47 AM
Nescio added a comment to D1904: Move fauna_arctic_wolf_attack.xml from trigger/ to gaia/.

Or Or fauna_arctic_wolf_violent.xml? (Other wolves are merely aggressive.)

Wed, Jul 17, 10:18 AM
Nescio added a comment to D2083: territory pull again.

Thank you for your comment.
Yes, these pulls become permanent when connected with a territory root; I suppose their usefulness is limited to scenarios where players can't build new structures (e.g. maps/scenarios/Campaign Test Map.xml).

Wed, Jul 17, 10:17 AM

Mon, Jul 15

Nescio added a comment to D2083: territory pull again.

It seems Jenkins is failing the build of the map file again, presumably because of spaces in file name (see also D1042).

Mon, Jul 15, 8:36 PM
Nescio updated the diff for D2083: territory pull again.
Mon, Jul 15, 8:33 PM
Nescio created D2083: territory pull again.
Mon, Jul 15, 8:31 PM
Nescio added inline comments to D2079: Adds snapping to edges for buildings.
Mon, Jul 15, 8:10 PM

Sun, Jul 14

Gallaecio awarded D1808: standardized hero aura descriptions a Love token.
Sun, Jul 14, 1:59 PM
Nescio added a comment to rP22266: moves territory_pull.xml from other/ to special/ and updates the only file in….

Thanks! However, template_*.xml files don't show up in Atlas either, so it wouldn't actually solve the issue? Or do you mean creating additional files?

template_territory_pull.xml
territory_pulls/32.xml
Sun, Jul 14, 9:45 AM

Sat, Jul 13

Nescio updated the diff for D2049: Add four default skirm structures.

Updated, now with footprint and obstruction.
Also minor workshop templates corrections.

Sat, Jul 13, 11:19 PM
Nescio updated subscribers of D2069: Cleanup of GUI-interface and test..

@Nescio, you are my superior in this

Am I?
Because it involves javascript, you actually need someone else; perhaps @bb is willing to.

Sat, Jul 13, 10:42 PM
Nescio added a comment to D2065: Rename "CauseSplashDamage" to the more generic "CauseDamageOverArea"..

Actually “splash” implies a liquid (the word itself is an onomatopoeia), so I'm not sure it is the most appropiate term for anything we use in 0 A.D.; anyway, that's probably outside the scope of this patch.
I merely chimed in here because I'm curious as to why you think this particular occurrence has to be renamed.

Sat, Jul 13, 10:18 PM
Nescio updated the diff for D2000: aura style corrections.

Updated

Sat, Jul 13, 10:14 PM
Nescio added a comment to D2065: Rename "CauseSplashDamage" to the more generic "CauseDamageOverArea"..

If it is to be changed, shouldn't *all* occurrences of “splash” then be replaced with “area”?

Sat, Jul 13, 11:19 AM
Nescio added a comment to D2049: Add four default skirm structures.

Good point! Not all skirmish templates specify a footprint, though; should those (barracks, blacksmith, civic centre, etc) be updated here as well?

Sat, Jul 13, 11:05 AM

Thu, Jul 11

Nescio added inline comments to D1846: Adds a "properties"-property to resources and let mods be able to prevent resources from being bartered, traded and/or tributed..
Thu, Jul 11, 8:16 PM
Nescio added a comment to D2052: remove unnecessary selection panels object.

however it seems like you are fixing a non-existing problem here

Possibly. I fully admit this patch isn't of vital importance, but I do believe it's a minor improvement; otherwise I wouldn't have bothered proposing it.

The GUI objects incorporate their size from their parent [...]
actually the 50%'s in the inner objects refer to the 50% of the object starting in L117

Yes, I fully understand that. However, *effectively*

<object size="50%-512 0 50%+512 100%">
  <object size="50%-304 100%-170 50%-110 100%"/>
  <object size="50%-114 100%-205 50%+114 100%"/>
  <object size="50%+110 100%-170 50%+512 100%"/>
</object>

is equivalent to

<object size="50%-304 100%-170 50%-110 100%"/>
<object size="50%-114 100%-205 50%+114 100%"/>
<object size="50%+110 100%-170 50%+512 100%"/>

One could wrap it into as many objects one likes, e.g.

Thu, Jul 11, 6:52 PM
Nescio added a comment to D1846: Adds a "properties"-property to resources and let mods be able to prevent resources from being bartered, traded and/or tributed..

Still getting this:

WARNING: JavaScript warning: simulation/ai/petra/tradeManager.js line 204 reference to undefined property mostNeeded[1]
Thu, Jul 11, 6:46 PM

Tue, Jul 9

Nescio added a comment to D1846: Adds a "properties"-property to resources and let mods be able to prevent resources from being bartered, traded and/or tributed..

New errors encountered:

WARNING: JavaScript warning: simulation/ai/petra/tradeManager.js line 179 reference to undefined property stocks[res]
Tue, Jul 9, 10:19 PM
Nescio added a comment to D2054: Apply modifications to template values based on entity owner instead watching player.

That's quick! And yes, I can confirm the errors have now gone, thank you.

Tue, Jul 9, 9:05 PM
Nescio added a comment to D2054: Apply modifications to template values based on entity owner instead watching player.

However, I'm getting the following recurring errors when attempting to build palisades (or another wallset):

WARNING: JavaScript warning: simulation/components/GuiInterface.js line 540 Script value conversion check failed: v.isString() || v.isNumber() (got type undefined)
Tue, Jul 9, 8:49 PM
Nescio added a comment to D2054: Apply modifications to template values based on entity owner instead watching player.

Oops, I did, you're right, sorry for the false alarm.

Tue, Jul 9, 8:47 PM
Nescio added a comment to D2054: Apply modifications to template values based on entity owner instead watching player.

Not quite sure what causes the duplication here:

Tue, Jul 9, 8:10 PM
Nescio added a comment to D1846: Adds a "properties"-property to resources and let mods be able to prevent resources from being bartered, traded and/or tributed..

So glad you're testing it already!

Actually I incorporated this in my mod two or three weeks ago, but I forgot to report these errors earlier; today someone else reminded me.
In 0abc silver, food, iron, wood, stone are all barterable and tributable, but only silver is tradable. The error might have something to do with that.

I'm starting to get a love-hate relationship with PetraAI ;) (The implementation of the garrison slots took me about half an hour, fixing Petra took the rest of the day,,,)

Yeah, occassionally Petra is surprisingly flexible, but more often it is rather time-consuming to figure out how to tweak it; there are plenty of things I'd like the AI to be able to do, but haven't figured out how to.

Tue, Jul 9, 7:45 PM
Nescio added a comment to D1846: Adds a "properties"-property to resources and let mods be able to prevent resources from being bartered, traded and/or tributed..

Occassionally I get the following warning and error:

WARNING: JavaScript warning: simulation/ai/petra/tradeManager.js line 193 reference to undefined property tradingGoods[res]
Tue, Jul 9, 7:17 PM
Nescio updated subscribers of D2056: Support for more garrisoned slots per entity..

Does it work as expected when you garrison infantry inside a siege tower and subsequently garrison the siege tower in a ship?

Tue, Jul 9, 6:58 PM

Sun, Jul 7

Nescio added a comment to D2051: Display wonders alongside heroes and relics.

Indeed, victory timers are something else and are already shown separately to all players (top centre, below faction emblem).

Sun, Jul 7, 5:09 PM
Nescio added a comment to D2054: Apply modifications to template values based on entity owner instead watching player.

This is great, many thanks! I just tried it out and it solves something that has bothered me for a long time.
The question to be asked is is it complete, does it correctly give all stats displayed everywhere? Hopefully someone more familiar with js could verify.

Sun, Jul 7, 4:53 PM
Nescio added a comment to D2051: Display wonders alongside heroes and relics.

Suppose you haven't scouted the entire map. Wouldn't it be weird if you nonetheless could see the health of the wonders of other players, while you don't even know their location?
To me, the gui is to display things the player has access to, not reveal information they don't have.

Sun, Jul 7, 2:50 PM
Nescio added a comment to D2051: Display wonders alongside heroes and relics.

Players can't train more than one hero nor build more than one wonder (see simulation/templates/special/player/player.xml). Although wonders could be captured in theory, in practice I don't expect any player to get more than twenty.

And conceptually: If an ally has a wonder shouldn't that be shown as well?

Why? Catafalques and heroes of other players aren't currently shown either, not even in regicide games. Only entities owned by the player are displayed.

Sun, Jul 7, 1:48 PM
Nescio added a comment to D2044: Allow immunity to certain damage types..

Than again this patch also should have a real reason to be considered adding - I can't see that.

Increased mod support?
Also, currently in 0 A.D. siege engines have a pierce armour of 50 (i.e. 1-0.9^50, which is over 99%), so they're practically invulnerable already; to me that seems to be a workaround. This patch would allow making them completely invulnerable, so e.g. archers won't waste their arrows on attacking rams, but instead attack other enemy units around.

Sun, Jul 7, 1:39 PM
Nescio added a comment to D2051: Display wonders alongside heroes and relics.

Still we have to check it this leads to bugs (including non-visible entities in the list that are "supposed" to be visible).

What do you mean exactly? template_structures_wonder.xml is the only file with the "Wonder" class.

Sun, Jul 7, 1:25 PM
Nescio accepted D2047: Show capture bar of panel entities..
  • It is a nice addition to the game, both informative and clear
  • Fixes a potential error currently present (in case someone disables health for e.g. catafalques)
  • The changes look good
  • In game it works as it should, without errors, including proper player colours:


Therefore I think this patch ought to be accepted. Thank you.

Sun, Jul 7, 12:50 PM
Nescio added a comment to D2051: Display wonders alongside heroes and relics.

To be able to have access to more items (hero, relic and wonder icons) the way to go is IMO to make the "sidebar" scrollable if it extends the height of the available area on the screen.
That would also allow mods have any number of heroes.
Fiddling with pixels rarely really fixes space issues.

Currently the number of items displayed are hardcoded, both in js and xml.
Ideally a new row would start when the current one reaches the right end of the screen. Unfortunately my understanding of the gui is rather limited; I have no difficulty with the xml files, but the js code is a different cup of tea.
De-hardcoding displayed item numbers (e.g. 10 researches on right, 24 structures in selection panel, etc.) is outside the scope of this patch, though.

Sun, Jul 7, 12:27 PM
Nescio added a comment to D2051: Display wonders alongside heroes and relics.

Opinions on the desired order of these entities are welcome!

  • hero, relic, wonder
  • hero, wonder, relic
  • relic, hero, wonder
  • relic, wonder, hero
  • wonder, hero, relic
  • wonder, relic, hero

(Entity panels are placed from top left corner to the right, which means the last item is closer to the centre of the screen than the first.)

Sun, Jul 7, 10:21 AM
Nescio added a comment to D2047: Show capture bar of panel entities..

Owner player colours in capture bar?

  • No overwriting of health bar.
  • Undependant of whether capture or health are present.

Thanks, tried it out, and indeed it seems to work as it ought to:

Sun, Jul 7, 10:16 AM

Sat, Jul 6

Nescio added a reviewer for D2025: Technology specific names: Restricted Owners Package.
Sat, Jul 6, 5:47 PM
Nescio added a reviewer for D2022: enable archery tradition for kush: Restricted Owners Package.
Sat, Jul 6, 5:46 PM
Nescio added a reviewer for D1863: reduce Blemmye and Nuba camp building time: Restricted Owners Package.
Sat, Jul 6, 5:37 PM
Nescio added a reviewer for D1762: Give Fortress a territory root: Restricted Owners Package.
Sat, Jul 6, 5:37 PM
Nescio added a reviewer for D1761: Enable workshops: Restricted Owners Package.
Sat, Jul 6, 5:36 PM
Nescio added a reviewer for D1765: Gaia templates' <Selectable> clean-up: bb.
Sat, Jul 6, 5:35 PM
Nescio added a reviewer for D1711: map preview naming consistency: Restricted Owners Package.
Sat, Jul 6, 5:33 PM
Nescio added a reviewer for D1881: move wallset_palisade from other/ to structures/: Restricted Owners Package.
Sat, Jul 6, 5:32 PM
Nescio added a reviewer for D2036: hellenic_royal_stoa → stoa: Restricted Owners Package.
Sat, Jul 6, 5:32 PM
Nescio updated the summary of D2052: remove unnecessary selection panels object.
Sat, Jul 6, 5:30 PM
Nescio updated the summary of D2052: remove unnecessary selection panels object.
Sat, Jul 6, 5:29 PM
Nescio created D2052: remove unnecessary selection panels object.
Sat, Jul 6, 5:26 PM
Nescio created D2051: Display wonders alongside heroes and relics.
Sat, Jul 6, 4:56 PM
Nescio added a reviewer for D1778: More aura icons: Restricted Owners Package.
Sat, Jul 6, 4:36 PM
Nescio added a reviewer for D1042: rename maps/ file names to use underscores (instead of spaces and capitalization): Restricted Owners Package.
Sat, Jul 6, 4:36 PM
Nescio edited reviewers for D2033: Correct sword prop for three units, added: Restricted Owners Package; removed: Stan.
Sat, Jul 6, 4:26 PM
Nescio created D2050: Move six actors to correct location.
Sat, Jul 6, 4:24 PM
Nescio created D2049: Add four default skirm structures.
Sat, Jul 6, 3:43 PM
Nescio added a reviewer for D2047: Show capture bar of panel entities.: Restricted Owners Package.

This patch seems to work fine with catafalques, but when I have a hero, I get the following errors:

WARNING: JavaScript warning: gui/session/session.js line 1050 reference to undefined property panelEntState.capturePoints
Sat, Jul 6, 2:42 PM